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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting jeff.mcgee@xxxxxxxxx (2017-10-13 21:10:32)
> From: Jeff McGee <jeff.mcgee@xxxxxxxxx>
> 
> 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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux