amdgpu: remove invalid check in amdgpu_bo_alloc

The heap is checked by the kernel and not libdrm, to make it even worse
it prevented allocating resources other than VRAM and GTT.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Junwei Zhang <Jerry.Zhang@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c
index 6a95929c5d6ddf8dcf70828e8f087bb2e5c3ade5..34904e382dc490204e8cce98dc7cd72f1b571520 100644 (file)
--- a/amdgpu/amdgpu_bo.c
+++ b/amdgpu/amdgpu_bo.c
@@ -74,19 +74,14 @@ int amdgpu_bo_alloc(amdgpu_device_handle dev,
                    amdgpu_bo_handle *buf_handle)
 {
        union drm_amdgpu_gem_create args;
-       unsigned heap = alloc_buffer->preferred_heap;
-       int r = 0;
-
-       /* It's an error if the heap is not specified */
-       if (!(heap & (AMDGPU_GEM_DOMAIN_GTT | AMDGPU_GEM_DOMAIN_VRAM)))
-               return -EINVAL;
+       int r;
 
        memset(&args, 0, sizeof(args));
        args.in.bo_size = alloc_buffer->alloc_size;
        args.in.alignment = alloc_buffer->phys_alignment;
 
        /* Set the placement. */
-       args.in.domains = heap;
+       args.in.domains = alloc_buffer->preferred_heap;
        args.in.domain_flags = alloc_buffer->flags;
 
        /* Allocate the buffer with the preferred heap. */