AW: [PATCH 1/2] drm/amdgpu: Set GTT_USWC flag to enable freesync

[Koenig, Christian]

Well that's unfortunately a NAK.

We currently can't communicate by DMA-buf if USWC is possible or not.

For the short term we could add something like a special handling for A+A configurations here. E.g. check if the imported BO is also an andgpu BO and set the flag if it is also set on the exported BO.

Regards,
Christian.

________________________________
Von: Sharma, Shashank <Shashank.Sharma at amd.com>
Gesendet: Samstag, 13. Februar 2021 17:37
An: amd-gfx at lists.freedesktop.org <amd-gfx at lists.freedesktop.org>
Cc: Sharma, Shashank <Shashank.Sharma at amd.com>; Koenig, Christian <Christian.Koenig at amd.com>; Deucher, Alexander <Alexander.Deucher at amd.com>
Betreff: [PATCH 1/2] drm/amdgpu: Set GTT_USWC flag to enable freesync

This patch sets 'AMDGPU_GEM_CREATE_CPU_GTT_USWC' as input
parameter flag, during object creation of an imported DMA
buffer.

In absence of this flag:
1. Function amdgpu_display_supported_domains() doesn't add
   AMDGPU_GEM_DOMAIN_GTT as supported domain.
2. Due to which, Function amdgpu_display_user_framebuffer_create()
   refuses to create framebuffer for imported DMA buffers.
3. Due to which, AddFB() IOCTL fails.
4. Due to which, amdgpu_present_check_flip() check fails in DDX
5. Due to which DDX driver doesn't allow flips (goes to blitting)
6. Due to which setting Freesync/VRR property fails for PRIME buffers.

So, this patch finally enables Freesync with PRIME buffer offloading.

Cc: Koenig Christian <christian.koenig at amd.com>
Cc: Deucher Alexander <alexander.deucher at amd.com>
Signed-off-by: Shashank Sharma <shashank.sharma at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
index 47e0b48dc26f..db62f3c9d6a5 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
@@ -448,8 +448,8 @@ amdgpu_dma_buf_create_obj(struct drm_device *dev, struct dma_buf *dma_buf)
         bp.resv = resv;
         dma_resv_lock(resv, NULL);
         ret = amdgpu_gem_object_create(adev, dma_buf->size, PAGE_SIZE,
-                       AMDGPU_GEM_DOMAIN_CPU,
-                       0, ttm_bo_type_sg, resv, &gobj);
+                       AMDGPU_GEM_DOMAIN_CPU, AMDGPU_GEM_CREATE_CPU_GTT_USWC,
+                       ttm_bo_type_sg, resv, &gobj);
         if (ret)
                 goto error;

--
2.25.1

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20210213/40f3a811/attachment-0001.htm>