Hi Chris, On Fri, Dec 13, 2019 at 10:31:40PM +0000, Chris Wilson wrote: > Wait for the object to be idle before changing its cache-level and > unbinding. This was dropped as supposedly superfluous from commit > 8b1c78e06e61 ("drm/i915: Avoid calling i915_gem_object_unbind holding > object lock"), but it turns out to prevent some cache dirt escaping. > Smells like papering over a race... > > Closes: https://gitlab.freedesktop.org/drm/intel/issues/820 > Fixes: 8b1c78e06e61 ("drm/i915: Avoid calling i915_gem_object_unbind holding object lock") > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Andi Shyti <andi.shyti@xxxxxxxxx> Acked-by: Andi Shyti <andi.shyti@xxxxxxxxx> Thanks, Andi _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx