diff options
author | Chunming Zhou <David1.Zhou@amd.com> | 2015-11-26 16:33:58 +0800 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2015-12-02 15:03:54 -0500 |
commit | e2f784fa8a3b1fbf6999ba44f7327bd9d29f2c5b (patch) | |
tree | a04b42503f09aa74b4f3c7052712c589a2e48e17 /drivers/gpu | |
parent | 0c62c6599b28b8f9e5a1822668325083598579da (diff) | |
download | op-kernel-dev-e2f784fa8a3b1fbf6999ba44f7327bd9d29f2c5b.zip op-kernel-dev-e2f784fa8a3b1fbf6999ba44f7327bd9d29f2c5b.tar.gz |
drm/amdgpu: add err check for pin userptr
Missing error check if the operation failed.
Signed-off-by: Chunming Zhou <David1.Zhou@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Diffstat (limited to 'drivers/gpu')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c index d4bac5f..8051cb9 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c @@ -587,9 +587,13 @@ static int amdgpu_ttm_backend_bind(struct ttm_tt *ttm, uint32_t flags = amdgpu_ttm_tt_pte_flags(gtt->adev, ttm, bo_mem); int r; - if (gtt->userptr) - amdgpu_ttm_tt_pin_userptr(ttm); - + if (gtt->userptr) { + r = amdgpu_ttm_tt_pin_userptr(ttm); + if (r) { + DRM_ERROR("failed to pin userptr\n"); + return r; + } + } gtt->offset = (unsigned long)(bo_mem->start << PAGE_SHIFT); if (!ttm->num_pages) { WARN(1, "nothing to bind %lu pages for mreg %p back %p!\n", |