[PATCH 2/2] drm/amdgpu: Fix use of interruptible waiting 1. The signal interrupt can affect the expected behaviour. 2. There is no good mechanism to handle the corresponding error. When signal interrupt happens, unpin is not called. As a result, inside AMDGPU, the statistic of pin size will be wrong.

[Christian König]

You somehow messed up the commit message. Everything got mangled into 
the subject line while sending the mails.

Apart from that the change looks good to me and with the commit message 
fixed is Reviewed-by: Christian König <christian.koenig at amd.com>

Regards,
Christian.

Am 24.04.2017 um 21:34 schrieb Alex Xie:
> Change-Id: I6889a4d9dd2703bcf5d448d18f6af51c496a93c9
> Signed-off-by: Alex Xie <AlexBin.Xie at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c
> index 76be2d2..75bd76f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c
> @@ -123,7 +123,7 @@ static void amdgpu_benchmark_move(struct amdgpu_device *adev, unsigned size,
>   	}
>   
>   	if (sobj) {
> -		r = amdgpu_bo_reserve(sobj, false);
> +		r = amdgpu_bo_reserve(sobj, true);
>   		if (likely(r == 0)) {
>   			amdgpu_bo_unpin(sobj);
>   			amdgpu_bo_unreserve(sobj);
> @@ -131,7 +131,7 @@ static void amdgpu_benchmark_move(struct amdgpu_device *adev, unsigned size,
>   		amdgpu_bo_unref(&sobj);
>   	}
>   	if (dobj) {
> -		r = amdgpu_bo_reserve(dobj, false);
> +		r = amdgpu_bo_reserve(dobj, true);
>   		if (likely(r == 0)) {
>   			amdgpu_bo_unpin(dobj);
>   			amdgpu_bo_unreserve(dobj);