aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2019-03-02 17:16:58 -0500
committerAlex Deucher <alexander.deucher@amd.com>2019-03-19 16:04:04 -0400
commitd38ca8f0f1a44c0ffe1cc746edf16b245e188ea5 (patch)
tree01fa01700f6bb83ce77f77dbff8f77c302532c2c /drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
parentb849aaa41c914a0fd88003f88cb04420a873c624 (diff)
drm/amdgpu: fix missing assignment of error return code to variable ret
An earlier commit replaced ttm_bo_wait with amdgpu_bo_sync_wait and removed the error return assignment to variable ret. Fix this by adding the assignment back. Also break line to clean up checkpatch overly long line warning. Detected by CoverityScan, CID#1477327 ("Logically dead code") Fixes: c60cd590cb7d ("drm/amdgpu: Replace ttm_bo_wait with amdgpu_bo_sync_wait") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c')
-rw-r--r--drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
index f8104760f1e6..4e96ad84efaa 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
@@ -878,7 +878,8 @@ static int init_kfd_vm(struct amdgpu_vm *vm, void **process_info,
878 pr_err("validate_pt_pd_bos() failed\n"); 878 pr_err("validate_pt_pd_bos() failed\n");
879 goto validate_pd_fail; 879 goto validate_pd_fail;
880 } 880 }
881 amdgpu_bo_sync_wait(vm->root.base.bo, AMDGPU_FENCE_OWNER_KFD, false); 881 ret = amdgpu_bo_sync_wait(vm->root.base.bo,
882 AMDGPU_FENCE_OWNER_KFD, false);
882 if (ret) 883 if (ret)
883 goto wait_pd_fail; 884 goto wait_pd_fail;
884 amdgpu_bo_fence(vm->root.base.bo, 885 amdgpu_bo_fence(vm->root.base.bo,