[PATCH 2/4] drm/amdgpu: set the evf name to identify the userq case

[Liang, Prike]

[Public]

> From: Koenig, Christian <Christian.Koenig at amd.com>
> Sent: Wednesday, April 16, 2025 7:02 PM
> To: Liang, Prike <Prike.Liang at amd.com>; amd-gfx at lists.freedesktop.org
> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>
> Subject: Re: [PATCH 2/4] drm/amdgpu: set the evf name to identify the userq
> case
>
> Am 16.04.25 um 10:50 schrieb Prike Liang:
> > The evf fence name can clearly identify the userq usage.
> >
> > Signed-off-by: Prike Liang <Prike.Liang at amd.com>
> > ---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_eviction_fence.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_eviction_fence.c
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_eviction_fence.c
> > index 7030d721196b..b34225bbd85d 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_eviction_fence.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_eviction_fence.c
> > @@ -31,7 +31,7 @@
> >  static const char *
> >  amdgpu_eviction_fence_get_driver_name(struct dma_fence *fence)  {
> > -   return "amdgpu";
> > +   return "amdgpu_userq_evf";
>
> Please use amdgpu_eviction as name here.
> Apart from that looks good to me.

Thanks for the suggestion. I will update in the following later version.

>
> Regards,
> Christian.
>
> >  }
> >
> >  static const char *