Am 17.04.25 um 13:15 schrieb Sunil Khatri:
> Add context and seqno of the fence in error logging
> rather than printing fence ptr.
>
> Suggested-by: Pierre-Eric Pelloux-Prayer <[email protected]>
> Suggested-by: Tvrtko Ursulin <[email protected]>
> Signed-off-by: Sunil Khatri <[email protected]>

Reviewed-by: Christian König <[email protected]>

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c
> index e944d05685dd..cd9dc0018ee6 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_userqueue.c
> @@ -43,7 +43,8 @@ amdgpu_userqueue_cleanup(struct amdgpu_userq_mgr *uq_mgr,
>       if (f && !dma_fence_is_signaled(f)) {
>               ret = dma_fence_wait_timeout(f, true, msecs_to_jiffies(100));
>               if (ret <= 0) {
> -                     DRM_ERROR("Timed out waiting for fence f=%p\n", f);
> +                     DRM_ERROR("Timed out waiting for fence=%llu:%llu\n",
> +                               f->context, f->seqno);
>                       return;
>               }
>       }
> @@ -630,7 +631,8 @@ amdgpu_userqueue_wait_for_signal(struct amdgpu_userq_mgr 
> *uq_mgr)
>                       continue;
>               ret = dma_fence_wait_timeout(f, true, msecs_to_jiffies(100));
>               if (ret <= 0) {
> -                     DRM_ERROR("Timed out waiting for fence f=%p\n", f);
> +                     DRM_ERROR("Timed out waiting for fence=%llu:%llu\n",
> +                               f->context, f->seqno);
>                       return -ETIMEDOUT;
>               }
>       }

Reply via email to