diff options
author | Terje Bergstrom <tbergstrom@nvidia.com> | 2016-11-08 16:10:43 -0500 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2016-11-11 11:20:31 -0500 |
commit | d09d259d74d229bb22e3440ab0660ecd66f94250 (patch) | |
tree | 20857c2af0811df6b1ee3af1993059662154364e /drivers | |
parent | 0f58a469042a1934ae6c97dee8532df64e171a48 (diff) |
gpu: nvgpu: vgpu: Do not overwrite err code on fail
vgpu_vm_alloc_share() wants to return -EINVAL if VMA areas requested
do not fulfill the criteria. The error code gets overwritten by a
call to vgpu_comm_sendrecv(), which makes vgpu_vm_alloc_share() always
return 0.
Change-Id: I93f56025f963d1d4ad2f9b06139fce742d3be41b
Signed-off-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-on: http://git-master/r/1249961
GVS: Gerrit_Virtual_Submit
Reviewed-by: Aingara Paramakuru <aparamakuru@nvidia.com>
Reviewed-by: Seema Khowala <seemaj@nvidia.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/gpu/nvgpu/vgpu/mm_vgpu.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/nvgpu/vgpu/mm_vgpu.c b/drivers/gpu/nvgpu/vgpu/mm_vgpu.c index 9a305fd5..69f6fcaf 100644 --- a/drivers/gpu/nvgpu/vgpu/mm_vgpu.c +++ b/drivers/gpu/nvgpu/vgpu/mm_vgpu.c | |||
@@ -436,8 +436,7 @@ clean_up_share: | |||
436 | msg.cmd = TEGRA_VGPU_CMD_AS_FREE_SHARE; | 436 | msg.cmd = TEGRA_VGPU_CMD_AS_FREE_SHARE; |
437 | msg.handle = vgpu_get_handle(g); | 437 | msg.handle = vgpu_get_handle(g); |
438 | p->handle = vm->handle; | 438 | p->handle = vm->handle; |
439 | err = vgpu_comm_sendrecv(&msg, sizeof(msg), sizeof(msg)); | 439 | WARN_ON(vgpu_comm_sendrecv(&msg, sizeof(msg), sizeof(msg)) || msg.ret); |
440 | WARN_ON(err || msg.ret); | ||
441 | clean_up: | 440 | clean_up: |
442 | kfree(vm); | 441 | kfree(vm); |
443 | as_share->vm = NULL; | 442 | as_share->vm = NULL; |