[PATCH v3 1/4] drm/amdgpu: Use allowed_domains for pinning dmabufs

[Felix Kuehling]

When determining the domains for pinning DMABufs, filter allowed_domains
and fail with a warning if VRAM is forbidden and GTT is not an allowed
domain.

Fixes: b0dd8c4879ef ("drm/amdgpu: allow pinning DMA-bufs into VRAM if all 
importers can do P2P")
Suggested-by: Christian König <christian.koe...@amd.com>
Signed-off-by: Felix Kuehling <felix.kuehl...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
index 667080cc9ae1..0446586bd5a7 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
@@ -77,7 +77,7 @@ static int amdgpu_dma_buf_pin(struct dma_buf_attachment 
*attach)
 {
        struct dma_buf *dmabuf = attach->dmabuf;
        struct amdgpu_bo *bo = gem_to_amdgpu_bo(dmabuf->priv);
-       u32 domains = bo->preferred_domains;
+       u32 domains = bo->allowed_domains;
 
        dma_resv_assert_held(dmabuf->resv);
 
@@ -93,6 +93,9 @@ static int amdgpu_dma_buf_pin(struct dma_buf_attachment 
*attach)
        if (domains & AMDGPU_GEM_DOMAIN_VRAM)
                bo->flags |= AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED;
 
+       if (WARN_ON(!domains))
+               return -EINVAL;
+
        return amdgpu_bo_pin(bo, domains);
 }
 
-- 
2.34.1