drm/i915: Ironlake do_idle_maps w/a may be called w/o struct_mutex
authorChris Wilson <[email protected]>
Thu, 30 Mar 2017 08:53:41 +0000 (09:53 +0100)
committerChris Wilson <[email protected]>
Thu, 30 Mar 2017 10:17:43 +0000 (11:17 +0100)
commit228ec87ccd040b620c467cd61d594bfaa4f8a12e
treecf3c15c2839e62c07e01e40720a2d2d8132fd7d0
parentb8991403ea0f3e5b59e08e37a7ac0af8ef4264e3
drm/i915: Ironlake do_idle_maps w/a may be called w/o struct_mutex

Since commit 1233e2db199d ("drm/i915: Move object backing storage
manipulation to its own locking"), i915_gem_object_put_pages() and
specifically the i915_gem_gtt_finish_pages() may be called from outside
of the struct_mutex and so we can no longer pass I915_WAIT_LOCKED to
i915_gem_wait_for_idle.

Fixes: 1233e2db199d ("drm/i915: Move object backing storage manipulation to its own locking")
Signed-off-by: Chris Wilson <[email protected]>
Cc: Chris Wilson <[email protected]>
Cc: Joonas Lahtinen <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: Jani Nikula <[email protected]>
Cc: [email protected]
Cc: <[email protected]> # v4.10+
Link: http://patchwork.freedesktop.org/patch/msgid/[email protected]
Reviewed-by: Joonas Lahtinen <[email protected]>
drivers/gpu/drm/i915/i915_gem_gtt.c