Re: [PATCH 2/8] drm/i915: Migrate to safe iterators in error state capture

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

 



On 09/10/2015 08:49, Chris Wilson wrote:
On Thu, Oct 08, 2015 at 07:31:34PM +0100, Tomas Elf wrote:
Using safe list iterators alleviates the problem of unsynchronized driver list
manipulations while error state capture is in the process of traversing lists.

Wrong, this does not make anything safe.
-Chris


I know, I didn't say it would make it safe. As I said in an earlier mail: I use the word "alleviate" to indicate that we're reducing the problem scope and make fatal crashes less likely to occur but the underlying problem is still there. Since my focus is on TDR I didn't set out to solve the underlying problem, I only needed the error state capture code to not break my tests, which is why I set out to solve the crashes - not the underlying problem. If you have a solution for the underlying problem then that's another story.

Thanks,
Tomas
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://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