[PATCH 1/3] drm/radeon: move ring locking out of reset path

Christian König deathsimple at vodafone.de
Fri Jun 29 08:18:52 PDT 2012


On 29.06.2012 17:09, Michel Dänzer wrote:
> On Fre, 2012-06-29 at 16:45 +0200, Christian König wrote:
>> Hold the ring lock the whole time the reset is in progress,
>> otherwise another process can submit new jobs.
> Sounds good, but doesn't this create other paths (e.g. initialization,
> resume) where the ring is being accessed without holding the lock? Isn't
> that a problem?

Thought about that also.

For init I'm pretty sure that no application can submit commands before 
we are done, otherwise we are doomed anyway.

For resume I'm not really sure, but I think that applications are 
resumed after the hardware driver had a chance of doing so.

Christian.



More information about the dri-devel mailing list