[PATCH v7 2/2] drm/i915/guc: Close deregister-context race against CT-loss

Daniele Ceraolo Spurio daniele.ceraolospurio at intel.com
Wed Dec 6 21:47:27 UTC 2023



On 11/30/2023 4:10 PM, Teres Alexis, Alan Previn wrote:
>> As far as i can tell, its only if we started resetting / wedging right after this
>> queued worker got started.
> alan: hope Daniele can proof read my tracing and confirm if got it right.

Yup, we don't flush the worker in reset prepare, so there is a chance 
that it might run parallel to the reset/wedge code, which we handle by 
checking the submission status. The list manipulation is protected by 
spinlock so we're safe on that side. The rest of the approach also LGTM:

Acked-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>

Daniele


More information about the dri-devel mailing list