From: Apoorva Singh <apoorva1.si...@intel.com>

Blitter commands which do not have MOCS fields rely on
cacheability of BlitterCacheControlRegister which was mapped
to index 0 by default.Once we changed the MOCS value of
index 0 to L3 WB, tests like gem_linear_blits started failing
due to a change in cacheability from UC to WB.

Program and place the BlitterCacheControlRegister in
build_aux_regs().

Signed-off-by: Apoorva Singh <apoorva1.si...@intel.com>
Signed-off-by: Ayaz A Siddiqui <ayaz.siddi...@intel.com>
---
 drivers/gpu/drm/i915/gt/intel_mocs.c | 13 +++++++++++++
 drivers/gpu/drm/i915/i915_reg.h      |  7 +++++++
 2 files changed, 20 insertions(+)

diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c 
b/drivers/gpu/drm/i915/gt/intel_mocs.c
index 403bd48362b19..82eafa8d22453 100644
--- a/drivers/gpu/drm/i915/gt/intel_mocs.c
+++ b/drivers/gpu/drm/i915/gt/intel_mocs.c
@@ -386,6 +386,17 @@ add_aux_reg(struct drm_i915_aux_table *aux,
        return x;
 }
 
+static struct drm_i915_aux_table *
+add_blit_cctl_override(struct drm_i915_aux_table *aux, u8 idx)
+{
+       return add_aux_reg(aux,
+                          REG_ENGINE_CONTEXT,
+                          "BLIT_CCTL",
+                          BLIT_CCTL(0),
+                          BLIT_CCTL_MOCS(idx, idx),
+                          0);
+}
+
 static struct drm_i915_aux_table *
 add_cmd_cctl_override(struct drm_i915_aux_table *aux, u8 idx)
 {
@@ -412,6 +423,8 @@ build_aux_regs(const struct intel_engine_cs *engine,
                 * a entry in drm_i915_aux_table link list.
                 */
                aux = add_cmd_cctl_override(aux, mocs->uc_index);
+               if (engine->class == COPY_ENGINE_CLASS)
+                       aux = add_blit_cctl_override(aux, mocs->uc_index);
        }
        return aux;
 }
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index df7a4550fb50f..207e0ada179b2 100644
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -2567,6 +2567,13 @@ static inline bool i915_mmio_reg_valid(i915_reg_t reg)
                      REG_FIELD_PREP(CMD_CCTL_WRITE_OVERRIDE_MASK, (write) << 
1) | \
                      REG_FIELD_PREP(CMD_CCTL_READ_OVERRIDE_MASK, (read) << 1))
 
+#define BLIT_CCTL(base)        _MMIO((base) + 0x204)
+#define   BLIT_CCTL_DST_MOCS_MASK      REG_GENMASK(14, 8)
+#define   BLIT_CCTL_SRC_MOCS_MASK      REG_GENMASK(6, 0)
+#define   BLIT_CCTL_DST_MOCS_SHIFT     8
+#define   BLIT_CCTL_MOCS(dst, src)                                             
        \
+       ((((dst) << 1) << BLIT_CCTL_DST_MOCS_SHIFT) | ((src) << 1))
+
 #define RING_RESET_CTL(base)   _MMIO((base) + 0xd0)
 #define   RESET_CTL_CAT_ERROR     REG_BIT(2)
 #define   RESET_CTL_READY_TO_RESET REG_BIT(1)
-- 
2.26.2

Reply via email to