diff options
| author | Alex Xie <[email protected]> | 2017-04-24 19:26:57 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2017-04-28 21:33:07 +0000 |
| commit | d159f26caaa91c88e9078c69fd159be9cec3a005 (patch) | |
| tree | 58f5d60ff2d31999a1832aaef92d262f39467cc0 /drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c | |
| parent | drm/amdgpu: Fix use of interruptible waiting (diff) | |
| download | kernel-d159f26caaa91c88e9078c69fd159be9cec3a005.tar.gz kernel-d159f26caaa91c88e9078c69fd159be9cec3a005.zip | |
drm/amdgpu: Real return value can be over-written when clean up
Signed-off-by: Alex Xie <[email protected]>
Reviewed-by: Christian König <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c index cc97eee93226..726b3e018eeb 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_benchmark.c @@ -117,6 +117,11 @@ static void amdgpu_benchmark_move(struct amdgpu_device *adev, unsigned size, } out_cleanup: + /* Check error value now. The value can be overwritten when clean up.*/ + if (r) { + DRM_ERROR("Error while benchmarking BO move.\n"); + } + if (sobj) { r = amdgpu_bo_reserve(sobj, false); if (likely(r == 0)) { @@ -133,10 +138,6 @@ out_cleanup: } amdgpu_bo_unref(&dobj); } - - if (r) { - DRM_ERROR("Error while benchmarking BO move.\n"); - } } void amdgpu_benchmark(struct amdgpu_device *adev, int test_number) |