We can't explicitly control the mmap caching type for discrete, but
using mmap_device_coherent should be good enough here on such devices.

Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/4842
Signed-off-by: Matthew Auld <matthew.a...@intel.com>
Cc: Gwan-gyeong Mun <gwan-gyeong....@intel.com>
---
 lib/intel_bufops.c | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/lib/intel_bufops.c b/lib/intel_bufops.c
index 05c0b0d4..c63a5760 100644
--- a/lib/intel_bufops.c
+++ b/lib/intel_bufops.c
@@ -451,11 +451,16 @@ static void __copy_ccs(struct buf_ops *bops, struct 
intel_buf *buf,
        ccs_size = CCS_SIZE(gen, buf);
        size = offset + ccs_size;
 
-       map = __gem_mmap_offset__wc(bops->fd, buf->handle, 0, size,
-                                   PROT_READ | PROT_WRITE);
-       if (!map)
-               map = gem_mmap__wc(bops->fd, buf->handle, 0, size,
-                                  PROT_READ | PROT_WRITE);
+       if (gem_has_lmem(bops->fd)) {
+               map = gem_mmap__device_coherent(bops->fd, buf->handle, 0, size,
+                                               PROT_READ | PROT_WRITE);
+       } else {
+               map = __gem_mmap_offset__wc(bops->fd, buf->handle, 0, size,
+                                           PROT_READ | PROT_WRITE);
+               if (!map)
+                       map = gem_mmap__wc(bops->fd, buf->handle, 0, size,
+                                          PROT_READ | PROT_WRITE);
+       }
 
        switch (dir) {
        case CCS_LINEAR_TO_BUF:
-- 
2.36.1

Reply via email to