[AMD Official Use Only - AMD Internal Distribution Only]
There is currently a bug in CI (requiring Intel Graphic Card which is totally
nonsense ) blocking this merge request.
Thanks
Lingshan
-----Original Message-----
From: Liang, Prike <prike.li...@amd.com>
Sent: Thursday, January 2, 2025 4:04 PM
To: Zhu, Lingshan <lingshan....@amd.com>; amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Kuehling, Felix
<felix.kuehl...@amd.com>; Koenig, Christian <christian.koe...@amd.com>;
Kasiviswanathan, Harish <harish.kasiviswanat...@amd.com>
Subject: RE: [PATCH] drm/amdkfd: test release process eviction fence before
signal
[AMD Official Use Only - AMD Internal Distribution Only]
Thanks for the information. I draft this patch to resolve the HIP stream test
that complained about the KFD process signals an invalidate fence on the latest
drm-next branch. BTW, it looks like your patch still hasn't landed in the
drm-next branch.
Regards,
Prike
> -----Original Message-----
> From: Zhu, Lingshan <lingshan....@amd.com>
> Sent: Thursday, January 2, 2025 3:13 PM
> To: Liang, Prike <prike.li...@amd.com>; amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Kuehling, Felix
> <felix.kuehl...@amd.com>; Koenig, Christian
> <christian.koe...@amd.com>; Kasiviswanathan, Harish
> <harish.kasiviswanat...@amd.com>
> Subject: Re: [PATCH] drm/amdkfd: test release process eviction fence
> before signal
>
> Hello Prike
>
> This issue has already been fixed, please see:
> https://www.spinics.net/lists/amd-gfx/msg117071.html
>
> Thanks
> Lingshan
> On 1/2/2025 2:30 PM, Prike Liang wrote:
> > It requires to validate the release process eviction fence before
> > signal the fence.
> >
> > Signed-off-by: Prike Liang <prike.li...@amd.com>
> > ---
> > drivers/gpu/drm/amd/amdkfd/kfd_process.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> > b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> > index 0976b5b0e8e8..083f83c94531 100644
> > --- a/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> > +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> > @@ -1160,7 +1160,8 @@ static void kfd_process_wq_release(struct
> > work_struct
> *work)
> > */
> > synchronize_rcu();
> > ef = rcu_access_pointer(p->ef);
> > - dma_fence_signal(ef);
> > + if (ef)
> > + dma_fence_signal(ef);
> >
> > kfd_process_remove_sysfs(p);
> >
