drm/i915/gvt: Fix workload status after wait

From commit 6c5bae6713acf691de21ab7e19c40ac801e7639d, workload status setting
was changed to only capture on error path, but we need to set it properly in
normal path too, otherwise we'll fail to complete workload which could lead
guest VM vGPU reset.

v2: uses braces and add Fixes tag.

Fixes: 6c5bae6713ac ("drm/i915: Rearrange i915_wait_request() accounting with callers")
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>

diff --git a/drivers/gpu/drm/i915/gvt/scheduler.c b/drivers/gpu/drm/i915/gvt/scheduler.c
index 18acb45dd14dedc99f0c663561697ac94e4d3d6f..843a5de4300d16653f6b0606fc981f8fa37ee04c 100644 (file)
--- a/drivers/gpu/drm/i915/gvt/scheduler.c
+++ b/drivers/gpu/drm/i915/gvt/scheduler.c
@@ -455,6 +455,8 @@ static int workload_thread(void *priv)
                if (lret < 0) {
                        workload->status = lret;
                        gvt_err("fail to wait workload, skip\n");
+               } else {
+                       workload->status = 0;
                }
 
 complete: