[PATCH] drm/amdgpu/benchmark: fix error handling

Christian König ckoenig.leichtzumerken at gmail.com
Wed Feb 23 15:12:36 UTC 2022


Am 23.02.22 um 15:23 schrieb Alex Deucher:
> Forgot to assign the return value here.
>
> Reported-by: kernel test robot <lkp at intel.com>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>

Reviewed-by: Christian König <christian.koenig 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 3136a9ad2d54..edc6377ec5ff 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c
> @@ -163,8 +163,8 @@ int amdgpu_benchmark(struct amdgpu_device *adev, int test_number)
>   			 "benchmark test: %d (simple test, VRAM to VRAM)\n",
>   			 test_number);
>   		/* simple test, VRAM to VRAM */
> -		amdgpu_benchmark_move(adev, 1024*1024, AMDGPU_GEM_DOMAIN_VRAM,
> -				      AMDGPU_GEM_DOMAIN_VRAM);
> +		r = amdgpu_benchmark_move(adev, 1024*1024, AMDGPU_GEM_DOMAIN_VRAM,
> +					  AMDGPU_GEM_DOMAIN_VRAM);
>   		if (r)
>   			goto done;
>   		break;



More information about the amd-gfx mailing list