Re: [PATCH] drm/sched: avoid killing parent entity on child SIGKILL v3

["Christian König" <christian.koenig@amd.com>]

On 10/28/25 14:13, Philipp Stanner wrote:
> On Tue, 2025-10-28 at 09:07 -0400, Alex Deucher wrote:
>> Looks like this hasn't landed yet.  Can someone push this?
> 
> Pushed it (with fixed commit message) to drm-misc-fixes.
> 
> I had Acked it for Christian but it seems that was lost.

Sorry, been on vacation last week and it took me till today to get through all my mails and back to this issue.

Thanks for taking care of it.

Christian.

> 
> Thx Alex
> 
> P.
> 
>>
>> Alex
>>
>> On Fri, Oct 17, 2025 at 2:18 AM Philipp Stanner <phasta@mailbox.org> wrote:
>>>
>>> On Wed, 2025-10-15 at 16:01 +0200, Christian König wrote:
>>>> From: David Rosca <david.rosca@amd.com>
>>>>
>>>> The DRM scheduler tracks who last uses an entity and when that process
>>>> is killed blocks all further submissions to that entity.
>>>>
>>>> The problem is that we didn't track who initially created an entity, so
>>>> when a process accidently leaked its file descriptor to a child and
>>>> that child got killed, we killed the parent's entities.
>>>>
>>>> Avoid that and instead initialize the entities last user on entity
>>>> creation. This also allows to drop the extra NULL check.
>>>>
>>>> v2: still use cmpxchg
>>>> v3: improve the commit message
>>>>
>>>> Signed-off-by: David Rosca <david.rosca@amd.com>
>>>> Signed-off-by: Christian König <christian.koenig@amd.com>
>>>> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4568
>>>> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
>>>> CC: stable@vger.kernel.org
>>>
>>> Acked-by: Philipp Stanner <phasta@kernel.org>
>>>
>>>
>>> Fire at will, Christian. Maybe optionally with the commit message nits
>>> twirked in we discussed before.
>>>
>>>
>>> P.