Skip to content

Commit 883bd89

Browse files
prliangpubalexdeucher
authored andcommitted
drm/amdgpu/userq: assign an error code for invalid userq va
It should return an error code if userq VA validation fails. Fixes: 9e46b8b ("drm/amdgpu: validate userq buffer virtual address and size") Signed-off-by: Prike Liang <Prike.Liang@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
1 parent 90e09ea commit 883bd89

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

drivers/gpu/drm/amd/amdgpu/amdgpu_userq.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ int amdgpu_userq_input_va_validate(struct amdgpu_vm *vm, u64 addr,
7171
return 0;
7272
}
7373

74+
r = -EINVAL;
7475
out_err:
7576
amdgpu_bo_unreserve(vm->root.bo);
7677
return r;
@@ -508,6 +509,7 @@ amdgpu_userq_create(struct drm_file *filp, union drm_amdgpu_userq *args)
508509
if (amdgpu_userq_input_va_validate(&fpriv->vm, args->in.queue_va, args->in.queue_size) ||
509510
amdgpu_userq_input_va_validate(&fpriv->vm, args->in.rptr_va, AMDGPU_GPU_PAGE_SIZE) ||
510511
amdgpu_userq_input_va_validate(&fpriv->vm, args->in.wptr_va, AMDGPU_GPU_PAGE_SIZE)) {
512+
r = -EINVAL;
511513
kfree(queue);
512514
goto unlock;
513515
}

0 commit comments

Comments
 (0)