diff options
| author |   Zhenyu Wang <zhenyuw@linux.intel.com> | 2016-11-02 15:00:15 +0800 |
|---|---|---|
| committer | Zhenyu Wang <zhenyuw@linux.intel.com> | 2016-11-07 14:17:00 +0800 |
| commit | 9b172345caa4e21423c649c4b98b32f53b6c89cb (patch) | |
| tree | 4ebc72797c1890906a4179c6d6b8dd08ef7b719c /drivers/gpu/drm/i915/gvt/scheduler.c | |
| parent | drm/i915/gvt: update misc ctl regs base on stepping info (diff) | |
| download | linux-dev-9b172345caa4e21423c649c4b98b32f53b6c89cb.tar.xz linux-dev-9b172345caa4e21423c649c4b98b32f53b6c89cb.zip | |
drm/i915/gvt: Fix workload status after wait
From commit e95433c73a11759203af1cae5958f998c9673370, 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: e95433c73a11 ("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>
Diffstat (limited to 'drivers/gpu/drm/i915/gvt/scheduler.c')
| -rw-r--r-- | drivers/gpu/drm/i915/gvt/scheduler.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/gvt/scheduler.c b/drivers/gpu/drm/i915/gvt/scheduler.c index 18acb45dd14d..843a5de4300d 100644 --- 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: |