xf86-video-intel: src/sna/gen6_common.h src/sna/gen8_render.c

Chris Wilson ickle at kemper.freedesktop.org
Fri Jun 13 00:28:50 PDT 2014


 src/sna/gen6_common.h |    2 +-
 src/sna/gen8_render.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 8e743ccef8f19b1226ceb8d6a5ceab65584a620d
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Fri Jun 13 08:27:05 2014 +0100

    sna/gen8: Check for WT support before attempting to use the WT mocs
    
    It appears the kernel has a better idea when WT is supported on which
    gen8 parts. I assumed it was always supported, but experience suggests
    otherwise, so only use WT support when advertised.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79967
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/src/sna/gen6_common.h b/src/sna/gen6_common.h
index 8b60ef6..6668620 100644
--- a/src/sna/gen6_common.h
+++ b/src/sna/gen6_common.h
@@ -36,7 +36,7 @@
 static inline bool is_uncached(struct sna *sna,
 			       struct kgem_bo *bo)
 {
-	return bo->scanout && !sna->kgem.has_wt;
+	return bo->io || (bo->scanout && !sna->kgem.has_wt);
 }
 
 inline static bool can_switch_to_blt(struct sna *sna,
diff --git a/src/sna/gen8_render.c b/src/sna/gen8_render.c
index 466fe01..a48ccf4 100644
--- a/src/sna/gen8_render.c
+++ b/src/sna/gen8_render.c
@@ -1377,7 +1377,7 @@ gen8_bind_bo(struct sna *sna,
 		domains = I915_GEM_DOMAIN_RENDER << 16 |I915_GEM_DOMAIN_RENDER;
 	} else
 		domains = I915_GEM_DOMAIN_SAMPLER << 16;
-	ss[1] = bo->io ? 0 : is_scanout ? (MOCS_WT | MOCS_ALL_CACHES) << 24 : (MOCS_WB | MOCS_ALL_CACHES) << 24;
+	ss[1] = is_uncached(sna, bo) ? 0 : is_scanout ? (MOCS_WT | MOCS_ALL_CACHES) << 24 : (MOCS_WB | MOCS_ALL_CACHES) << 24;
 	ss[2] = ((width - 1)  << SURFACE_WIDTH_SHIFT |
 		 (height - 1) << SURFACE_HEIGHT_SHIFT);
 	ss[3] = (bo->pitch - 1) << SURFACE_PITCH_SHIFT;


More information about the xorg-commit mailing list