diff options
| author | Maarten Lankhorst <[email protected]> | 2020-08-19 14:08:46 +0000 |
|---|---|---|
| committer | Joonas Lahtinen <[email protected]> | 2020-09-07 11:29:53 +0000 |
| commit | 1af343cdc1971ff953f08f9dcf0efad464cf057f (patch) | |
| tree | b660b134644d6ef433326260a64229082dcdfaea /drivers/gpu/drm/i915/gvt/cmd_parser.c | |
| parent | drm/i915: Add an implementation for i915_gem_ww_ctx locking, v2. (diff) | |
| download | kernel-1af343cdc1971ff953f08f9dcf0efad464cf057f.tar.gz kernel-1af343cdc1971ff953f08f9dcf0efad464cf057f.zip | |
drm/i915: Remove locking from i915_gem_object_prepare_read/write
Execbuffer submission will perform its own WW locking, and we
cannot rely on the implicit lock there.
This also makes it clear that the GVT code will get a lockdep splat when
multiple batchbuffer shadows need to be performed in the same instance,
fix that up.
Signed-off-by: Maarten Lankhorst <[email protected]>
Reviewed-by: Tvrtko Ursulin <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Signed-off-by: Joonas Lahtinen <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/i915/gvt/cmd_parser.c')
| -rw-r--r-- | drivers/gpu/drm/i915/gvt/cmd_parser.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c b/drivers/gpu/drm/i915/gvt/cmd_parser.c index 943c8d232703..d0a599b51bfe 100644 --- a/drivers/gpu/drm/i915/gvt/cmd_parser.c +++ b/drivers/gpu/drm/i915/gvt/cmd_parser.c @@ -1923,6 +1923,7 @@ static int perform_bb_shadow(struct parser_exec_state *s) if (ret) goto err_unmap; + i915_gem_object_unlock(bb->obj); INIT_LIST_HEAD(&bb->list); list_add(&bb->list, &s->workload->shadow_bb); |