[PATCH 4/7] drm/amdgpu:return ETIME if really time out

Liu, Monk Monk.Liu at amd.com
Tue Dec 12 10:23:37 UTC 2017


Without this patch a timed out submit will never be caught, is it really good ?

For the backward compatible why it's applied on incorrect logic ?

-----Original Message-----
From: Christian König [mailto:ckoenig.leichtzumerken at gmail.com] 
Sent: 2017年12月12日 18:20
To: Liu, Monk <Monk.Liu at amd.com>; amd-gfx at lists.freedesktop.org
Subject: Re: [PATCH 4/7] drm/amdgpu:return ETIME if really time out

NAK, it does make sense but that would change the user space interface in a non backward compatible way, wouldn't it?

Christian.

Am 12.12.2017 um 10:46 schrieb Monk Liu:
> Change-Id: I7304577a500fc2d41482f2fe3f1692ae3797f576
> Signed-off-by: Monk Liu <Monk.Liu at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> index 44523a8..f62b07a 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> @@ -1506,8 +1506,8 @@ static int amdgpu_cs_wait_any_fence(struct 
> amdgpu_device *adev,
>   
>   	if (first < fence_count && array[first])
>   		r = array[first]->error;
> -	else
> -		r = 0;
> +	else if (r == 0)
> +		r = -ETIME;
>   
>   err_free_fence_array:
>   	for (i = 0; i < fence_count; i++)



More information about the amd-gfx mailing list