diff options
author | Zhenyu Wang <zhenyuw@linux.intel.com> | 2019-10-23 17:43:27 +0800 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2019-10-23 23:55:29 +0100 |
commit | 8eb8e322ec07392e8c8008437216c38c310ff6c7 (patch) | |
tree | b0fad4a9f12c52fb919b64fa3cf7afaf18147195 /drivers/gpu/drm/i915/gvt | |
parent | 058179e72e0956a2dfe4927db6cbe5fbfb2406aa (diff) | |
download | linux-8eb8e322ec07392e8c8008437216c38c310ff6c7.tar.gz linux-8eb8e322ec07392e8c8008437216c38c310ff6c7.tar.bz2 linux-8eb8e322ec07392e8c8008437216c38c310ff6c7.zip |
drm/i915/gvt: fix dead locking in early workload shadow
As early workload scan and shadow happens in execlist mmio handler,
which has already taken vgpu_lock. So remove extra lock taking here.
Fixes: 952f89f098c7 ("drm/i915/gvt: Wean off struct_mutex")
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'drivers/gpu/drm/i915/gvt')
-rw-r--r-- | drivers/gpu/drm/i915/gvt/scheduler.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/gvt/scheduler.c b/drivers/gpu/drm/i915/gvt/scheduler.c index 36bb7639e82f..a5b942ee3ceb 100644 --- a/drivers/gpu/drm/i915/gvt/scheduler.c +++ b/drivers/gpu/drm/i915/gvt/scheduler.c @@ -1584,9 +1584,7 @@ intel_vgpu_create_workload(struct intel_vgpu *vgpu, int ring_id, */ if (list_empty(workload_q_head(vgpu, ring_id))) { intel_runtime_pm_get(&dev_priv->runtime_pm); - mutex_lock(&vgpu->vgpu_lock); ret = intel_gvt_scan_and_shadow_workload(workload); - mutex_unlock(&vgpu->vgpu_lock); intel_runtime_pm_put_unchecked(&dev_priv->runtime_pm); } |