On to, 2017-03-30 at 09:53 +0100, Chris Wilson wrote: > 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 <chris@xxxxxxxxxxxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: <stable@xxxxxxxxxxxxxxx> # v4.10+ Is there Bugzilla or did you just hit the assert on your own? Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation