On 24.04.2023 18:09, Andi Shyti wrote:
This reverts commit b76c0deef6273609c02ed5053209f6397cd1b0fb.

This patch is causing boot failures for MTL.

Revert it.

Signed-off-by: Andi Shyti <andi.sh...@linux.intel.com>
Cc: Matt Roper <matthew.d.ro...@intel.com>
Cc: Lucas De Marchi <lucas.demar...@intel.com>
Cc: Aravind Iddamsetty <aravind.iddamse...@intel.com>
Cc: Nirmoy Das <nirmoy....@intel.com>
Cc: Fei Yang <fei.y...@intel.com>
Cc: Andrzej Hajda <andrzej.ha...@intel.com>

Reviewed-by: Andrzej Hajda <andrzej.ha...@intel.com>

Regards
Andrzej

---
  drivers/gpu/drm/i915/gt/intel_gt_regs.h |  6 +--
  drivers/gpu/drm/i915/gt/intel_gtt.c     | 47 +----------------
  drivers/gpu/drm/i915/gt/intel_gtt.h     |  8 ---
  drivers/gpu/drm/i915/gt/intel_mocs.c    | 70 +------------------------
  4 files changed, 3 insertions(+), 128 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_gt_regs.h 
b/drivers/gpu/drm/i915/gt/intel_gt_regs.h
index e8c3b762a92a3..fd1f9cd35e9d7 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt_regs.h
+++ b/drivers/gpu/drm/i915/gt/intel_gt_regs.h
@@ -356,11 +356,7 @@
  #define GEN7_TLB_RD_ADDR                      _MMIO(0x4700)
#define GEN12_PAT_INDEX(index) _MMIO(0x4800 + (index) * 4)
-#define _PAT_INDEX(index)                      _PICK_EVEN_2RANGES(index, 8, \
-                                                                  0x4800, 
0x4804, \
-                                                                  0x4848, 
0x484c)
-#define XEHP_PAT_INDEX(index)                  MCR_REG(_PAT_INDEX(index))
-#define XELPMP_PAT_INDEX(index)                        _MMIO(_PAT_INDEX(index))
+#define XEHP_PAT_INDEX(index)                  MCR_REG(0x4800 + (index) * 4)
#define XEHP_TILE0_ADDR_RANGE MCR_REG(0x4900)
  #define   XEHP_TILE_LMEM_RANGE_SHIFT          8
diff --git a/drivers/gpu/drm/i915/gt/intel_gtt.c 
b/drivers/gpu/drm/i915/gt/intel_gtt.c
index 2f6a9be0ffe61..4f436ba7a3c83 100644
--- a/drivers/gpu/drm/i915/gt/intel_gtt.c
+++ b/drivers/gpu/drm/i915/gt/intel_gtt.c
@@ -468,44 +468,6 @@ void gtt_write_workarounds(struct intel_gt *gt)
        }
  }
-static void xelpmp_setup_private_ppat(struct intel_uncore *uncore)
-{
-       intel_uncore_write(uncore, XELPMP_PAT_INDEX(0),
-                          MTL_PPAT_L4_0_WB);
-       intel_uncore_write(uncore, XELPMP_PAT_INDEX(1),
-                          MTL_PPAT_L4_1_WT);
-       intel_uncore_write(uncore, XELPMP_PAT_INDEX(2),
-                          MTL_PPAT_L4_3_UC);
-       intel_uncore_write(uncore, XELPMP_PAT_INDEX(3),
-                          MTL_PPAT_L4_0_WB | MTL_2_COH_1W);
-       intel_uncore_write(uncore, XELPMP_PAT_INDEX(4),
-                          MTL_PPAT_L4_0_WB | MTL_3_COH_2W);
-
-       /*
-        * Remaining PAT entries are left at the hardware-default
-        * fully-cached setting
-        */
-}
-
-static void xelpg_setup_private_ppat(struct intel_gt *gt)
-{
-       intel_gt_mcr_multicast_write(gt, XEHP_PAT_INDEX(0),
-                                    MTL_PPAT_L4_0_WB);
-       intel_gt_mcr_multicast_write(gt, XEHP_PAT_INDEX(1),
-                                    MTL_PPAT_L4_1_WT);
-       intel_gt_mcr_multicast_write(gt, XEHP_PAT_INDEX(2),
-                                    MTL_PPAT_L4_3_UC);
-       intel_gt_mcr_multicast_write(gt, XEHP_PAT_INDEX(3),
-                                    MTL_PPAT_L4_0_WB | MTL_2_COH_1W);
-       intel_gt_mcr_multicast_write(gt, XEHP_PAT_INDEX(4),
-                                    MTL_PPAT_L4_0_WB | MTL_3_COH_2W);
-
-       /*
-        * Remaining PAT entries are left at the hardware-default
-        * fully-cached setting
-        */
-}
-
  static void tgl_setup_private_ppat(struct intel_uncore *uncore)
  {
        /* TGL doesn't support LLC or AGE settings */
@@ -641,14 +603,7 @@ void setup_private_pat(struct intel_gt *gt)
GEM_BUG_ON(GRAPHICS_VER(i915) < 8); - if (gt->type == GT_MEDIA) {
-               xelpmp_setup_private_ppat(gt->uncore);
-               return;
-       }
-
-       if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 70))
-               xelpg_setup_private_ppat(gt);
-       else if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 50))
+       if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 50))
                xehp_setup_private_ppat(gt);
        else if (GRAPHICS_VER(i915) >= 12)
                tgl_setup_private_ppat(uncore);
diff --git a/drivers/gpu/drm/i915/gt/intel_gtt.h 
b/drivers/gpu/drm/i915/gt/intel_gtt.h
index ea17849e7a5c8..69ce55f517f56 100644
--- a/drivers/gpu/drm/i915/gt/intel_gtt.h
+++ b/drivers/gpu/drm/i915/gt/intel_gtt.h
@@ -147,14 +147,6 @@ typedef u64 gen8_pte_t;
  #define GEN8_PDE_IPS_64K BIT(11)
  #define GEN8_PDE_PS_2M   BIT(7)
-#define MTL_PPAT_L4_CACHE_POLICY_MASK REG_GENMASK(3, 2)
-#define MTL_PAT_INDEX_COH_MODE_MASK    REG_GENMASK(1, 0)
-#define MTL_PPAT_L4_3_UC       REG_FIELD_PREP(MTL_PPAT_L4_CACHE_POLICY_MASK, 3)
-#define MTL_PPAT_L4_1_WT       REG_FIELD_PREP(MTL_PPAT_L4_CACHE_POLICY_MASK, 1)
-#define MTL_PPAT_L4_0_WB       REG_FIELD_PREP(MTL_PPAT_L4_CACHE_POLICY_MASK, 0)
-#define MTL_3_COH_2W   REG_FIELD_PREP(MTL_PAT_INDEX_COH_MODE_MASK, 3)
-#define MTL_2_COH_1W   REG_FIELD_PREP(MTL_PAT_INDEX_COH_MODE_MASK, 2)
-
  enum i915_cache_level;
struct drm_i915_gem_object;
diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c 
b/drivers/gpu/drm/i915/gt/intel_mocs.c
index 2c014407225cc..69b489e8dfeda 100644
--- a/drivers/gpu/drm/i915/gt/intel_mocs.c
+++ b/drivers/gpu/drm/i915/gt/intel_mocs.c
@@ -40,10 +40,6 @@ struct drm_i915_mocs_table {
  #define LE_COS(value)         ((value) << 15)
  #define LE_SSE(value)         ((value) << 17)
-/* Defines for the tables (GLOB_MOCS_0 - GLOB_MOCS_16) */
-#define _L4_CACHEABILITY(value)        ((value) << 2)
-#define IG_PAT(value)          ((value) << 8)
-
  /* Defines for the tables (LNCFMOCS0 - LNCFMOCS31) - two entries per word */
  #define L3_ESC(value)         ((value) << 0)
  #define L3_SCC(value)         ((value) << 1)
@@ -54,7 +50,6 @@ struct drm_i915_mocs_table {
  /* Helper defines */
  #define GEN9_NUM_MOCS_ENTRIES 64  /* 63-64 are reserved, but configured. */
  #define PVC_NUM_MOCS_ENTRIES  3
-#define MTL_NUM_MOCS_ENTRIES   16
/* (e)LLC caching options */
  /*
@@ -78,12 +73,6 @@ struct drm_i915_mocs_table {
  #define L3_2_RESERVED         _L3_CACHEABILITY(2)
  #define L3_3_WB                       _L3_CACHEABILITY(3)
-/* L4 caching options */
-#define L4_0_WB                        _L4_CACHEABILITY(0)
-#define L4_1_WT                        _L4_CACHEABILITY(1)
-#define L4_2_RESERVED          _L4_CACHEABILITY(2)
-#define L4_3_UC                        _L4_CACHEABILITY(3)
-
  #define MOCS_ENTRY(__idx, __control_value, __l3cc_value) \
        [__idx] = { \
                .control_value = __control_value, \
@@ -427,57 +416,6 @@ static const struct drm_i915_mocs_entry pvc_mocs_table[] = 
{
        MOCS_ENTRY(2, 0, L3_3_WB),
  };
-static const struct drm_i915_mocs_entry mtl_mocs_table[] = {
-       /* Error - Reserved for Non-Use */
-       MOCS_ENTRY(0,
-                  IG_PAT(0),
-                  L3_LKUP(1) | L3_3_WB),
-       /* Cached - L3 + L4 */
-       MOCS_ENTRY(1,
-                  IG_PAT(1),
-                  L3_LKUP(1) | L3_3_WB),
-       /* L4 - GO:L3 */
-       MOCS_ENTRY(2,
-                  IG_PAT(1),
-                  L3_LKUP(1) | L3_1_UC),
-       /* Uncached - GO:L3 */
-       MOCS_ENTRY(3,
-                  IG_PAT(1) | L4_3_UC,
-                  L3_LKUP(1) | L3_1_UC),
-       /* L4 - GO:Mem */
-       MOCS_ENTRY(4,
-                  IG_PAT(1),
-                  L3_LKUP(1) | L3_GLBGO(1) | L3_1_UC),
-       /* Uncached - GO:Mem */
-       MOCS_ENTRY(5,
-                  IG_PAT(1) | L4_3_UC,
-                  L3_LKUP(1) | L3_GLBGO(1) | L3_1_UC),
-       /* L4 - L3:NoLKUP; GO:L3 */
-       MOCS_ENTRY(6,
-                  IG_PAT(1),
-                  L3_1_UC),
-       /* Uncached - L3:NoLKUP; GO:L3 */
-       MOCS_ENTRY(7,
-                  IG_PAT(1) | L4_3_UC,
-                  L3_1_UC),
-       /* L4 - L3:NoLKUP; GO:Mem */
-       MOCS_ENTRY(8,
-                  IG_PAT(1),
-                  L3_GLBGO(1) | L3_1_UC),
-       /* Uncached - L3:NoLKUP; GO:Mem */
-       MOCS_ENTRY(9,
-                  IG_PAT(1) | L4_3_UC,
-                  L3_GLBGO(1) | L3_1_UC),
-       /* Display - L3; L4:WT */
-       MOCS_ENTRY(14,
-                  IG_PAT(1) | L4_1_WT,
-                  L3_LKUP(1) | L3_3_WB),
-       /* CCS - Non-Displayable */
-       MOCS_ENTRY(15,
-                  IG_PAT(1),
-                  L3_GLBGO(1) | L3_1_UC),
-};
-
  enum {
        HAS_GLOBAL_MOCS = BIT(0),
        HAS_ENGINE_MOCS = BIT(1),
@@ -507,13 +445,7 @@ static unsigned int get_mocs_settings(const struct 
drm_i915_private *i915,
        memset(table, 0, sizeof(struct drm_i915_mocs_table));
table->unused_entries_index = I915_MOCS_PTE;
-       if (IS_METEORLAKE(i915)) {
-               table->size = ARRAY_SIZE(mtl_mocs_table);
-               table->table = mtl_mocs_table;
-               table->n_entries = MTL_NUM_MOCS_ENTRIES;
-               table->uc_index = 9;
-               table->unused_entries_index = 1;
-       } else if (IS_PONTEVECCHIO(i915)) {
+       if (IS_PONTEVECCHIO(i915)) {
                table->size = ARRAY_SIZE(pvc_mocs_table);
                table->table = pvc_mocs_table;
                table->n_entries = PVC_NUM_MOCS_ENTRIES;

Reply via email to