[Intel-gfx] [PATCH 1/2] drm/i915: Properly lock the engine timeline in debugfs i915_gem_request

Chris Wilson chris at chris-wilson.co.uk
Fri Oct 13 20:22:29 UTC 2017


Quoting jeff.mcgee at intel.com (2017-10-13 21:10:32)
> From: Jeff McGee <jeff.mcgee at intel.com>
> 
> We are racing with updates to the timeline. This can cause an inconsistent
> snapshot to be dumped, or even worse a NULL pointer dereference.

So no one's being using this, lets delete it.

If the execution list is useful in anyway, add it to i915_engine_info so
you also have the execution queue.

Patch itself looks fine, but as you can see from the bitrot I haven't
been using for debugging for a few years.
-Chris


More information about the Intel-gfx mailing list