[PATCH V1] drm/amdgpu/userq: add context and seqno of the fence

[Christian König]

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 <pierre-eric.pelloux-prayer at amd.com>
> Suggested-by: Tvrtko Ursulin <tvrtko.ursulin at igalia.com>
> Signed-off-by: Sunil Khatri <sunil.khatri at amd.com>

Reviewed-by: Christian König <christian.koenig at amd.com>

> ---
>  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;
>  		}
>  	}