diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-08-07 11:27:37 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2014-08-15 00:58:17 -0400 |
| commit | 68b1ea30e4b64c6c5a1ed7c1970062d61c1a892c (patch) | |
| tree | 71a6ee059681061434df34f99633e46d43b6275c | |
| parent | 4fb0bbd5d0f13f39a2fe5e092a64cece3a4afcb9 (diff) | |
drm/radeon: add a check for allocation failure (v2)
We can easily return -ENOMEM here if kzalloc() fails.
v2: agd5f: drop the vm mutex
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| -rw-r--r-- | drivers/gpu/drm/radeon/radeon_vm.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_vm.c b/drivers/gpu/drm/radeon/radeon_vm.c index ccae4d9dc3de..9c8358f6f130 100644 --- a/drivers/gpu/drm/radeon/radeon_vm.c +++ b/drivers/gpu/drm/radeon/radeon_vm.c | |||
| @@ -483,6 +483,10 @@ int radeon_vm_bo_set_addr(struct radeon_device *rdev, | |||
| 483 | /* add a clone of the bo_va to clear the old address */ | 483 | /* add a clone of the bo_va to clear the old address */ |
| 484 | struct radeon_bo_va *tmp; | 484 | struct radeon_bo_va *tmp; |
| 485 | tmp = kzalloc(sizeof(struct radeon_bo_va), GFP_KERNEL); | 485 | tmp = kzalloc(sizeof(struct radeon_bo_va), GFP_KERNEL); |
| 486 | if (!tmp) { | ||
| 487 | mutex_unlock(&vm->mutex); | ||
| 488 | return -ENOMEM; | ||
| 489 | } | ||
| 486 | tmp->it.start = bo_va->it.start; | 490 | tmp->it.start = bo_va->it.start; |
| 487 | tmp->it.last = bo_va->it.last; | 491 | tmp->it.last = bo_va->it.last; |
| 488 | tmp->vm = vm; | 492 | tmp->vm = vm; |
