This issue was not catched by CI, because of series of unfortunate events.
Before, CI has rebooted without module blocklist, and CI catched boot-time
dmesg correctly and marked it as 'ci@boot' test with failure if there was a taint.
I've been doing changes to make blocklisting i915 possible and load it as
the first test of IGT: that'd make possible to remove some workarounds
and integrate the result better on our framework.
The test to decide if i915 should be modprobed was slightly off, and
on these runs where i915 failed to load in boot, it was modprobed again,
and modprobe hanged because of existing i915. Results were not collected.
I've added the condition to the conditional modprobe, and the results
from failed boot-time modprobe should be soon available as before,
eg. CI_DRM_10873 later shards with SNB.
Regards,
Tomi
> From: Latvala, Petri <petri.latvala@xxxxxxxxx>
> On Tue, Nov 02, 2021 at 03:25:10PM -0700, Matt Roper wrote:
> > Bspec: 54077,68173,54833
> > Cc: Anusha Srivatsa <anusha.srivatsa@xxxxxxxxx>
> > Signed-off-by: Matt Roper <matthew.d.roper@xxxxxxxxx>
> > ---
> > drivers/gpu/drm/i915/gt/intel_workarounds.c | 278
> +++++++++++++++++++-
> > drivers/gpu/drm/i915/i915_reg.h | 94 +++++--
> > drivers/gpu/drm/i915/intel_pm.c | 21 +-
> > 3 files changed, 372 insertions(+), 21 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c
> b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > index 4aaa210fc003..37fd541a9719 100644
> > --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > @@ -644,6 +644,42 @@ static void dg1_ctx_workarounds_init(struct
> intel_engine_cs *engine,
> >
> DG1_HZ_READ_SUPPRESSION_OPTIMIZATION_DISABLE);
> > }
> >
> > +static void dg2_ctx_workarounds_init(struct intel_engine_cs *engine,
> > + struct
> i915_wa_list *wal)
> > +{
> > + gen12_ctx_gt_tuning_init(engine, wal);
> > +
> > + /* Wa_16011186671:dg2_g11 */
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G11, STEP_A0,
> STEP_B0)) {
> > + wa_masked_dis(wal, VFLSKPD,
> DIS_MULT_MISS_RD_SQUASH);
> > + wa_masked_en(wal, VFLSKPD,
> DIS_OVER_FETCH_CACHE);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_A0,
> STEP_B0)) {
> > + /* Wa_14010469329:dg2_g10 */
> > + wa_masked_en(wal,
> GEN11_COMMON_SLICE_CHICKEN3,
> > +
> XEHP_DUAL_SIMD8_SEQ_MERGE_DISABLE);
> > +
> > + /*
> > + * Wa_22010465075:dg2_g10
> > + * Wa_22010613112:dg2_g10
> > + * Wa_14010698770:dg2_g10
> > + */
> > + wa_masked_en(wal,
> GEN11_COMMON_SLICE_CHICKEN3,
> > +
> GEN12_DISABLE_CPS_AWARE_COLOR_PIPE);
> > + }
> > +
> > + /* Wa_16013271637:dg2 */
> > + wa_masked_en(wal, SLICE_COMMON_ECO_CHICKEN1,
> > +
> MSC_MSAA_REODER_BUF_BYPASS_DISABLE);
> > +
> > + /* Wa_22012532006:dg2 */
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_A0,
> STEP_C0) ||
> > + IS_DG2_GRAPHICS_STEP(engine->i915, G11, STEP_A0,
> STEP_B0))
> > + wa_masked_en(wal,
> GEN9_HALF_SLICE_CHICKEN7,
> > +
> DG2_DISABLE_ROUND_ENABLE_ALLOW_FOR_SSLA);
> > +}
> > +
> > static void fakewa_disable_nestedbb_mode(struct intel_engine_cs
> *engine,
> >
> struct i915_wa_list *wal)
> > {
> > @@ -730,7 +766,9 @@ __intel_engine_init_ctx_wa(struct intel_engine_cs
> *engine,
> > if (engine->class != RENDER_CLASS)
> > goto done;
> >
> > - if (IS_XEHPSDV(i915))
> > + if (IS_DG2(i915))
> > + dg2_ctx_workarounds_init(engine, wal);
> > + else if (IS_XEHPSDV(i915))
> > ; /* noop; none at this time */
> > else if (IS_DG1(i915))
> > dg1_ctx_workarounds_init(engine, wal);
> > @@ -1343,12 +1381,117 @@ xehpsdv_gt_workarounds_init(struct intel_gt
> *gt, struct i915_wa_list *wal)
> > GLOBAL_INVALIDATION_MODE);
> > }
> >
> > +static void
> > +dg2_gt_workarounds_init(struct intel_gt *gt, struct i915_wa_list *wal)
> > +{
> > + struct intel_engine_cs *engine;
> > + int id;
> > +
> > + xehp_init_mcr(gt, wal);
> > +
> > + /* Wa_14011060649:dg2 */
> > + wa_14011060649(gt, wal);
> > +
> > + /*
> > + * Although there are per-engine instances of these registers,
> > + * they technically exist outside the engine itself and are not
> > + * impacted by engine resets. Furthermore, they're part of the
> > + * GuC blacklist so trying to treat them as engine workarounds
> > + * will result in GuC initialization failure and a wedged GPU.
> > + */
> > + for_each_engine(engine, gt, id) {
> > + if (engine->class != VIDEO_DECODE_CLASS)
> > + continue;
> > +
> > + /* Wa_16010515920:dg2_g10 */
> > + if (IS_DG2_GRAPHICS_STEP(gt->i915, G10,
> STEP_A0, STEP_B0))
> > + wa_write_or(wal,
> VDBOX_CGCTL3F18(engine->mmio_base),
> > +
> ALNUNIT_CLKGATE_DIS);
> > + }
> > +
> > + if (IS_DG2_G10(gt->i915)) {
> > + /* Wa_22010523718:dg2 */
> > + wa_write_or(wal,
> UNSLICE_UNIT_LEVEL_CLKGATE,
> > + CG3DDISCFEG_CLKGATE_DIS);
> > +
> > + /* Wa_14011006942:dg2 */
> > + wa_write_or(wal,
> SUBSLICE_UNIT_LEVEL_CLKGATE,
> > + DSS_ROUTER_CLKGATE_DIS);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(gt->i915, G10, STEP_A0, STEP_B0))
> {
> > + /* Wa_14010680813:dg2_g10 */
> > + wa_write_or(wal, GEN12_GAMSTLB_CTRL,
> CONTROL_BLOCK_CLKGATE_DIS |
> > + EGRESS_BLOCK_CLKGATE_DIS |
> TAG_BLOCK_CLKGATE_DIS);
> > +
> > + /* Wa_14010948348:dg2_g10 */
> > + wa_write_or(wal, UNSLCGCTL9430,
> MSQDUNIT_CLKGATE_DIS);
> > +
> > + /* Wa_14011037102:dg2_g10 */
> > + wa_write_or(wal, UNSLCGCTL9444,
> LTCDD_CLKGATE_DIS);
> > +
> > + /* Wa_14011371254:dg2_g10 */
> > + wa_write_or(wal, SLICE_UNIT_LEVEL_CLKGATE,
> NODEDSS_CLKGATE_DIS);
> > +
> > + /* Wa_14011431319:dg2_g10 */
> > + wa_write_or(wal, UNSLCGCTL9440,
> GAMTLBOACS_CLKGATE_DIS |
> > + GAMTLBVDBOX7_CLKGATE_DIS
> |
> > + GAMTLBVDBOX6_CLKGATE_DIS
> |
> > + GAMTLBVDBOX5_CLKGATE_DIS
> |
> > + GAMTLBVDBOX4_CLKGATE_DIS
> |
> > + GAMTLBVDBOX3_CLKGATE_DIS
> |
> > + GAMTLBVDBOX2_CLKGATE_DIS
> |
> > + GAMTLBVDBOX1_CLKGATE_DIS
> |
> > + GAMTLBVDBOX0_CLKGATE_DIS
> |
> > + GAMTLBKCR_CLKGATE_DIS |
> > + GAMTLBGUC_CLKGATE_DIS |
> > + GAMTLBBLT_CLKGATE_DIS);
> > + wa_write_or(wal, UNSLCGCTL9444,
> GAMTLBGFXA0_CLKGATE_DIS |
> > + GAMTLBGFXA1_CLKGATE_DIS |
> > +
> GAMTLBCOMPA0_CLKGATE_DIS |
> > +
> GAMTLBCOMPA1_CLKGATE_DIS |
> > +
> GAMTLBCOMPB0_CLKGATE_DIS |
> > +
> GAMTLBCOMPB1_CLKGATE_DIS |
> > +
> GAMTLBCOMPC0_CLKGATE_DIS |
> > +
> GAMTLBCOMPC1_CLKGATE_DIS |
> > +
> GAMTLBCOMPD0_CLKGATE_DIS |
> > +
> GAMTLBCOMPD1_CLKGATE_DIS |
> > + GAMTLBMERT_CLKGATE_DIS |
> > + GAMTLBVEBOX3_CLKGATE_DIS
> |
> > + GAMTLBVEBOX2_CLKGATE_DIS
> |
> > + GAMTLBVEBOX1_CLKGATE_DIS
> |
> > +
> GAMTLBVEBOX0_CLKGATE_DIS);
> > +
> > + /* Wa_14010569222:dg2_g10 */
> > + wa_write_or(wal,
> UNSLICE_UNIT_LEVEL_CLKGATE,
> > + GAMEDIA_CLKGATE_DIS);
> > +
> > + /* Wa_14011028019:dg2_g10 */
> > + wa_write_or(wal, SSMCGCTL9530,
> RTFUNIT_CLKGATE_DIS);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(gt->i915, G10, STEP_A0, STEP_B0)
> ||
> > + IS_DG2_GRAPHICS_STEP(gt->i915, G11, STEP_A0, STEP_B0))
> {
> > + /* Wa_14012362059:dg2 */
> > + wa_write_or(wal, GEN12_MERT_MOD_CTRL,
> FORCE_MISS_FTLB);
> > + }
> > +
> > + /* Wa_1509235366:dg2 */
> > + wa_write_or(wal, GEN12_GAMCNTRL_CTRL,
> INVALIDATION_BROADCAST_MODE_DIS |
> > + GLOBAL_INVALIDATION_MODE);
> > +
> > + /* Wa_14014830051:dg2 */
> > + wa_write_clr(wal, SARB_CHICKEN1, COMP_CKN_IN);
> > +}
> > +
> > static void
> > gt_init_workarounds(struct intel_gt *gt, struct i915_wa_list *wal)
> > {
> > struct drm_i915_private *i915 = gt->i915;
> >
> > - if (IS_XEHPSDV(i915))
> > + if (IS_DG2(i915))
> > + dg2_gt_workarounds_init(gt, wal);
> > + else if (IS_XEHPSDV(i915))
> > xehpsdv_gt_workarounds_init(gt, wal);
> > else if (IS_DG1(i915))
> > dg1_gt_workarounds_init(gt, wal);
> > @@ -1739,6 +1882,34 @@ static void xehpsdv_whitelist_build(struct
> intel_engine_cs *engine)
> > allow_read_ctx_timestamp(engine);
> > }
> >
> > +static void dg2_whitelist_build(struct intel_engine_cs *engine)
> > +{
> > + struct i915_wa_list *w = &engine->whitelist;
> > +
> > + allow_read_ctx_timestamp(engine);
> > +
> > + switch (engine->class) {
> > + case RENDER_CLASS:
> > + /*
> > + * Wa_1507100340:dg2_g10
> > + *
> > + * This covers 4 registers which are next to one
> another :
> > + * - PS_INVOCATION_COUNT
> > + * - PS_INVOCATION_COUNT_UDW
> > + * - PS_DEPTH_COUNT
> > + * - PS_DEPTH_COUNT_UDW
> > + */
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10,
> STEP_A0, STEP_B0))
> > + whitelist_reg_ext(w,
> PS_INVOCATION_COUNT,
> > +
> RING_FORCE_TO_NONPRIV_ACCESS_RD |
> > +
> RING_FORCE_TO_NONPRIV_RANGE_4);
> > +
> > + break;
> > + default:
> > + break;
> > + }
> > +}
> > +
> > void intel_engine_init_whitelist(struct intel_engine_cs *engine)
> > {
> > struct drm_i915_private *i915 = engine->i915;
> > @@ -1746,7 +1917,9 @@ void intel_engine_init_whitelist(struct
> intel_engine_cs *engine)
> >
> > wa_init_start(w, "whitelist", engine->name);
> >
> > - if (IS_XEHPSDV(i915))
> > + if (IS_DG2(i915))
> > + dg2_whitelist_build(engine);
> > + else if (IS_XEHPSDV(i915))
> > xehpsdv_whitelist_build(engine);
> > else if (IS_DG1(i915))
> > dg1_whitelist_build(engine);
> > @@ -1826,6 +1999,105 @@ static void
> > rcs_engine_wa_init(struct intel_engine_cs *engine, struct i915_wa_list
> *wal)
> > {
> > struct drm_i915_private *i915 = engine->i915;
> > + u64 dss_mask = intel_sseu_get_subslices(&engine->gt-
> >info.sseu, 0);
>
> fi-snb-2600:
>
> https://paste.debian.net/1219275
>
> [ 4.348876] intel_sseu_get_subslices:39 GEM_BUG_ON(slice >= sseu-
> >max_slices)
>
>
>
> --
> Petri Latvala
>
>
>
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G11, STEP_A0,
> STEP_B0)) {
> > + /* Wa_14013392000:dg2_g11 */
> > + wa_masked_en(wal, GEN7_ROW_CHICKEN2,
> GEN12_ENABLE_LARGE_GRF_MODE);
> > +
> > + /* Wa_16011620976:dg2_g11 */
> > + wa_write_or(wal, LSC_CHICKEN_BIT_0_UDW,
> DIS_CHAIN_2XSIMD8);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_A0,
> STEP_B0) ||
> > + IS_DG2_GRAPHICS_STEP(engine->i915, G11, STEP_A0,
> STEP_B0)) {
> > + /* Wa_14012419201:dg2 */
> > + wa_masked_en(wal, GEN9_ROW_CHICKEN4,
> > +
> GEN12_DISABLE_HDR_PAST_PAYLOAD_HOLD_FIX);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_B0,
> STEP_C0) ||
> > + IS_DG2_G11(engine->i915)) {
> > + /*
> > + * Wa_22012826095:dg2
> > + * Wa_22013059131:dg2
> > + */
> > + wa_write_clr_set(wal,
> LSC_CHICKEN_BIT_0_UDW,
> > +
> MAXREQS_PER_BANK,
> > +
> REG_FIELD_PREP(MAXREQS_PER_BANK, 2));
> > +
> > + /* Wa_22013059131:dg2 */
> > + wa_write_or(wal, LSC_CHICKEN_BIT_0,
> > +
> FORCE_1_SUB_MESSAGE_PER_FRAGMENT);
> > + }
> > +
> > + /* Wa_1308578152:dg2_g10 when first gslice is fused off */
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_B0,
> STEP_C0) &&
> > + (dss_mask & GENMASK(GEN_DSS_PER_GSLICE - 1, 0)) == 0)
> {
> > + wa_masked_dis(wal,
> GEN12_CS_DEBUG_MODE1_CCCSUNIT_BE_COMMON,
> > +
> GEN12_REPLAY_MODE_GRANULARITY);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_B0,
> STEP_FOREVER) ||
> > + IS_DG2_G11(engine->i915)) {
> > + /* Wa_22013037850:dg2 */
> > + wa_write_or(wal, LSC_CHICKEN_BIT_0_UDW,
> > +
> DISABLE_128B_EVICTION_COMMAND_UDW);
> > +
> > + /* Wa_22012856258:dg2 */
> > + wa_masked_en(wal, GEN7_ROW_CHICKEN2,
> > +
> GEN12_DISABLE_READ_SUPPRESSION);
> > +
> > + /*
> > + * Wa_22010960976:dg2
> > + * Wa_14013347512:dg2
> > + */
> > + wa_masked_dis(wal, GEN12_HDC_CHICKEN0,
> > +
> LSC_L1_FLUSH_CTL_3D_DATAPORT_FLUSH_EVENTS_MASK);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_A0,
> STEP_B0)) {
> > + /*
> > + * Wa_1608949956:dg2_g10
> > + * Wa_14010198302:dg2_g10
> > + */
> > + wa_masked_en(wal, GEN8_ROW_CHICKEN,
> > + MDQ_ARBITRATION_MODE |
> UGM_BACKUP_MODE);
> > +
> > + /*
> > + * Wa_14010918519:dg2_g10
> > + *
> > + * LSC_CHICKEN_BIT_0 always reads back as 0 is
> this stepping,
> > + * so ignoring verification.
> > + */
> > + wa_add(wal, LSC_CHICKEN_BIT_0_UDW, 0,
> > + FORCE_SLM_FENCE_SCOPE_TO_TILE |
> FORCE_UGM_FENCE_SCOPE_TO_TILE,
> > + 0, false);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_A0,
> STEP_B0)) {
> > + /* Wa_22010430635:dg2 */
> > + wa_masked_en(wal,
> > + GEN9_ROW_CHICKEN4,
> > + GEN12_DISABLE_GRF_CLEAR);
> > +
> > + /* Wa_14010648519:dg2 */
> > + wa_write_or(wal, XEHP_L3NODEARBCFG,
> XEHP_LNESPARE);
> > + }
> > +
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_A0,
> STEP_C0) ||
> > + IS_DG2_G11(engine->i915)) {
> > + /* Wa_22012654132:dg2 */
> > + wa_add(wal, GEN10_CACHE_MODE_SS, 0,
> > +
> _MASKED_BIT_ENABLE(ENABLE_PREFETCH_INTO_IC),
> > + 0 /* write-only, so skip validation */,
> > + true);
> > + }
> > +
> > + /* Wa_14013202645:dg2 */
> > + if (IS_DG2_GRAPHICS_STEP(engine->i915, G10, STEP_B0,
> STEP_C0) ||
> > + IS_DG2_GRAPHICS_STEP(engine->i915, G11, STEP_A0,
> STEP_B0))
> > + wa_write_or(wal, RT_CTRL, DIS_NULL_QUERY);
> >
> > if (IS_DG1_GRAPHICS_STEP(i915, STEP_A0, STEP_B0) ||
> > IS_TGL_UY_GRAPHICS_STEP(i915, STEP_A0, STEP_B0)) {
> > diff --git a/drivers/gpu/drm/i915/i915_reg.h
> b/drivers/gpu/drm/i915/i915_reg.h
> > index b806ad4bdeca..ee39d6bd0f3c 100644
> > --- a/drivers/gpu/drm/i915/i915_reg.h
> > +++ b/drivers/gpu/drm/i915/i915_reg.h
> > @@ -502,6 +502,11 @@ static inline bool i915_mmio_reg_valid(i915_reg_t
> reg)
> > #define INVALIDATION_BROADCAST_MODE_DIS REG_BIT(12)
> > #define GLOBAL_INVALIDATION_MODE
> REG_BIT(2)
> >
> > +#define GEN12_GAMSTLB_CTRL _MMIO(0xcf4c)
> > +#define CONTROL_BLOCK_CLKGATE_DIS REG_BIT(12)
> > +#define EGRESS_BLOCK_CLKGATE_DIS REG_BIT(11)
> > +#define TAG_BLOCK_CLKGATE_DIS REG_BIT(7)
> > +
> > #define GEN12_MERT_MOD_CTRL _MMIO(0xcf28)
> > #define FORCE_MISS_FTLB REG_BIT(3)
> >
> > @@ -777,6 +782,9 @@ static inline bool i915_mmio_reg_valid(i915_reg_t
> reg)
> > #define EU_PERF_CNTL5 _MMIO(0xe55c)
> > #define EU_PERF_CNTL6 _MMIO(0xe65c)
> >
> > +#define RT_CTRL _MMIO(0xe530)
> > +#define DIS_NULL_QUERY REG_BIT(10)
> > +
> > /*
> > * OA Boolean state
> > */
> > @@ -2781,6 +2789,9 @@ static inline bool i915_mmio_reg_valid(i915_reg_t
> reg)
> > #define VDBOX_CGCTL3F10(base) _MMIO((base) +
> 0x3f10)
> > #define IECPUNIT_CLKGATE_DIS REG_BIT(22)
> >
> > +#define VDBOX_CGCTL3F18(base) _MMIO((base) +
> 0x3f18)
> > +#define ALNUNIT_CLKGATE_DIS REG_BIT(13)
> > +
> > #define ERROR_GEN6 _MMIO(0x40a0)
> > #define GEN7_ERR_INT _MMIO(0x44040)
> > #define ERR_INT_POISON (1 << 31)
> > @@ -3124,7 +3135,8 @@ static inline bool i915_mmio_reg_valid(i915_reg_t
> reg)
> > #define GEN9_RCS_FE_FSM2 _MMIO(0x22a4)
> >
> > #define GEN10_CACHE_MODE_SS
> _MMIO(0xe420)
> > -#define FLOAT_BLEND_OPTIMIZATION_ENABLE (1 << 4)
> > +#define ENABLE_PREFETCH_INTO_IC REG_BIT(3)
> > +#define FLOAT_BLEND_OPTIMIZATION_ENABLE REG_BIT(4)
> >
> > /* Fuse readout registers for GT */
> > #define HSW_PAVP_FUSE1
> _MMIO(0x911C)
> > @@ -4333,18 +4345,25 @@ enum {
> > #define SARBUNIT_CLKGATE_DIS (1 << 5)
> > #define RCCUNIT_CLKGATE_DIS (1 << 7)
> > #define MSCUNIT_CLKGATE_DIS (1 << 10)
> > +#define NODEDSS_CLKGATE_DIS REG_BIT(12)
> > #define L3_CLKGATE_DIS REG_BIT(16)
> > #define L3_CR2X_CLKGATE_DIS REG_BIT(17)
> >
> > #define SUBSLICE_UNIT_LEVEL_CLKGATE _MMIO(0x9524)
> > -#define GWUNIT_CLKGATE_DIS (1 << 16)
> > +#define DSS_ROUTER_CLKGATE_DIS REG_BIT(28)
> > +#define GWUNIT_CLKGATE_DIS REG_BIT(16)
> >
> > #define SUBSLICE_UNIT_LEVEL_CLKGATE2 _MMIO(0x9528)
> > #define CPSSUNIT_CLKGATE_DIS REG_BIT(9)
> >
> > +#define SSMCGCTL9530 _MMIO(0x9530)
> > +#define RTFUNIT_CLKGATE_DIS REG_BIT(18)
> > +
> > #define UNSLICE_UNIT_LEVEL_CLKGATE _MMIO(0x9434)
> > #define VFUNIT_CLKGATE_DIS REG_BIT(20)
> > -#define TSGUNIT_CLKGATE_DIS REG_BIT(17)
> > +#define TSGUNIT_CLKGATE_DIS REG_BIT(17) /*
> XEHPSDV */
> > +#define CG3DDISCFEG_CLKGATE_DIS REG_BIT(17) /* DG2 */
> > +#define GAMEDIA_CLKGATE_DIS REG_BIT(11)
> > #define HSUNIT_CLKGATE_DIS REG_BIT(8)
> > #define VSUNIT_CLKGATE_DIS REG_BIT(3)
> >
> > @@ -8404,6 +8423,9 @@ enum {
> > #define GEN9_CTX_PREEMPT_REG _MMIO(0x2248)
> > #define GEN12_DISABLE_POSH_BUSY_FF_DOP_CG REG_BIT(11)
> >
> > +#define GEN12_CS_DEBUG_MODE1_CCCSUNIT_BE_COMMON
> _MMIO(0x20EC)
> > +#define GEN12_REPLAY_MODE_GRANULARITY
> REG_BIT(0)
> > +
> > #define GEN8_CS_CHICKEN1 _MMIO(0x2580)
> > #define GEN9_PREEMPT_3D_OBJECT_LEVEL (1 <<
> 0)
> > #define GEN9_PREEMPT_GPGPU_LEVEL(hi, lo) (((hi) << 2) | ((lo)
> << 1))
> > @@ -8427,9 +8449,10 @@ enum {
> > #define GEN8_ERRDETBCTRL (1 << 9)
> >
> > #define GEN11_COMMON_SLICE_CHICKEN3
> _MMIO(0x7304)
> > - #define DG1_FLOAT_POINT_BLEND_OPT_STRICT_MODE_EN
> REG_BIT(12)
> > - #define GEN11_BLEND_EMB_FIX_DISABLE_IN_RCC
> REG_BIT(11)
> > - #define GEN12_DISABLE_CPS_AWARE_COLOR_PIPE
> REG_BIT(9)
> > +#define DG1_FLOAT_POINT_BLEND_OPT_STRICT_MODE_EN
> REG_BIT(12)
> > +#define XEHP_DUAL_SIMD8_SEQ_MERGE_DISABLE
> REG_BIT(12)
> > +#define GEN11_BLEND_EMB_FIX_DISABLE_IN_RCC
> REG_BIT(11)
> > +#define GEN12_DISABLE_CPS_AWARE_COLOR_PIPE
> REG_BIT(9)
> >
> > #define HIZ_CHICKEN
> _MMIO(0x7018)
> > # define CHV_HZ_8X8_MODE_IN_1X
> REG_BIT(15)
> > @@ -8493,6 +8516,12 @@ enum {
> > #define HDC_FORCE_NON_COHERENT (1 <<
> 4)
> > #define HDC_BARRIER_PERFORMANCE_DISABLE (1 << 10)
> >
> > +#define GEN12_HDC_CHICKEN0
> _MMIO(0xE5F0)
> > +#define LSC_L1_FLUSH_CTL_3D_DATAPORT_FLUSH_EVENTS_MASK
> REG_GENMASK(13, 11)
> > +
> > +#define SARB_CHICKEN1
> _MMIO(0xe90c)
> > +#define COMP_CKN_IN
> REG_GENMASK(30, 29)
> > +
> > #define GEN8_HDC_CHICKEN1
> _MMIO(0x7304)
> >
> > /* GEN9 chicken */
> > @@ -8523,6 +8552,10 @@ enum {
> > #define PIXEL_ROUNDING_TRUNC_FB_PASSTHRU (1 << 15)
> > #define PER_PIXEL_ALPHA_BYPASS_EN (1 <<
> 7)
> >
> > +#define VFLSKPD
> _MMIO(0x62a8)
> > +#define DIS_OVER_FETCH_CACHE REG_BIT(1)
> > +#define DIS_MULT_MISS_RD_SQUASH REG_BIT(0)
> > +
> > #define FF_MODE2 _MMIO(0x6604)
> > #define FF_MODE2_GS_TIMER_MASK REG_GENMASK(31, 24)
> > #define FF_MODE2_GS_TIMER_224
> REG_FIELD_PREP(FF_MODE2_GS_TIMER_MASK, 224)
> > @@ -9346,6 +9379,9 @@ enum {
> > #define GEN8_SDEUNIT_CLOCK_GATE_DISABLE (1 << 14)
> > #define GEN8_HDCUNIT_CLOCK_GATE_DISABLE_HDCREQ (1 << 28)
> >
> > +#define UNSLCGCTL9430
> _MMIO(0x9430)
> > +#define MSQDUNIT_CLKGATE_DIS
> REG_BIT(3)
> > +
> > #define GEN6_GFXPAUSE
> _MMIO(0xA000)
> > #define GEN6_RPNSWREQ
> _MMIO(0xA008)
> > #define GEN6_TURBO_DISABLE (1 <<
> 31)
> > @@ -9661,24 +9697,39 @@ enum {
> > #define GEN9_CCS_TLB_PREFETCH_ENABLE (1 << 3)
> >
> > #define GEN8_ROW_CHICKEN _MMIO(0xe4f0)
> > -#define FLOW_CONTROL_ENABLE (1 << 15)
> > -#define PARTIAL_INSTRUCTION_SHOOTDOWN_DISABLE (1 <<
> 8)
> > -#define STALL_DOP_GATING_DISABLE (1 << 5)
> > -#define THROTTLE_12_5 (7 <<
> 2)
> > -#define DISABLE_EARLY_EOT (1 <<
> 1)
> > +#define FLOW_CONTROL_ENABLE
> REG_BIT(15)
> > +#define UGM_BACKUP_MODE
> REG_BIT(13)
> > +#define MDQ_ARBITRATION_MODE
> REG_BIT(12)
> > +#define PARTIAL_INSTRUCTION_SHOOTDOWN_DISABLE
> REG_BIT(8)
> > +#define STALL_DOP_GATING_DISABLE
> REG_BIT(5)
> > +#define THROTTLE_12_5
> REG_GENMASK(4, 2)
> > +#define DISABLE_EARLY_EOT
> REG_BIT(1)
> >
> > #define GEN7_ROW_CHICKEN2
> _MMIO(0xe4f4)
> > +#define GEN12_DISABLE_READ_SUPPRESSION REG_BIT(15)
> > #define GEN12_DISABLE_EARLY_READ REG_BIT(14)
> > +#define GEN12_ENABLE_LARGE_GRF_MODE
> REG_BIT(12)
> > #define GEN12_PUSH_CONST_DEREF_HOLD_DIS REG_BIT(8)
> >
> > +#define LSC_CHICKEN_BIT_0
> _MMIO(0xe7c8)
> > +#define FORCE_1_SUB_MESSAGE_PER_FRAGMENT REG_BIT(15)
> > +#define LSC_CHICKEN_BIT_0_UDW
> _MMIO(0xe7c8 + 4)
> > +#define DIS_CHAIN_2XSIMD8
> REG_BIT(55 - 32)
> > +#define FORCE_SLM_FENCE_SCOPE_TO_TILE
> REG_BIT(42 - 32)
> > +#define FORCE_UGM_FENCE_SCOPE_TO_TILE
> REG_BIT(41 - 32)
> > +#define MAXREQS_PER_BANK
> REG_GENMASK(39 - 32, 37 - 32)
> > +#define DISABLE_128B_EVICTION_COMMAND_UDW
> REG_BIT(36 - 32)
> > +
> > #define GEN7_ROW_CHICKEN2_GT2 _MMIO(0xf4f4)
> > #define DOP_CLOCK_GATING_DISABLE (1 << 0)
> > #define PUSH_CONSTANT_DEREF_DISABLE (1 << 8)
> > #define GEN11_TDL_CLOCK_GATING_FIX_DISABLE (1 << 1)
> >
> > -#define GEN9_ROW_CHICKEN4 _MMIO(0xe48c)
> > -#define GEN12_DISABLE_TDL_PUSH REG_BIT(9)
> > -#define GEN11_DIS_PICK_2ND_EU REG_BIT(7)
> > +#define GEN9_ROW_CHICKEN4
> _MMIO(0xe48c)
> > +#define GEN12_DISABLE_GRF_CLEAR
> REG_BIT(13)
> > +#define GEN12_DISABLE_TDL_PUSH
> REG_BIT(9)
> > +#define GEN11_DIS_PICK_2ND_EU
> REG_BIT(7)
> > +#define GEN12_DISABLE_HDR_PAST_PAYLOAD_HOLD_FIX
> REG_BIT(4)
> >
> > #define HSW_ROW_CHICKEN3 _MMIO(0xe49c)
> > #define HSW_ROW_CHICKEN3_L3_GLOBAL_ATOMICS_DISABLE (1 << 6)
> > @@ -9693,9 +9744,10 @@ enum {
> > #define GEN8_SAMPLER_POWER_BYPASS_DIS (1 << 1)
> >
> > #define GEN9_HALF_SLICE_CHICKEN7 _MMIO(0xe194)
> > -#define GEN9_SAMPLER_HASH_COMPRESSED_READ_ADDR (1 <<
> 8)
> > -#define GEN9_ENABLE_YV12_BUGFIX (1 << 4)
> > -#define GEN9_ENABLE_GPGPU_PREEMPTION (1 << 2)
> > +#define DG2_DISABLE_ROUND_ENABLE_ALLOW_FOR_SSLA
> REG_BIT(15)
> > +#define GEN9_SAMPLER_HASH_COMPRESSED_READ_ADDR
> REG_BIT(8)
> > +#define GEN9_ENABLE_YV12_BUGFIX
> REG_BIT(4)
> > +#define GEN9_ENABLE_GPGPU_PREEMPTION
> REG_BIT(2)
> >
> > /* Audio */
> > #define G4X_AUD_VID_DID
> _MMIO(DISPLAY_MMIO_BASE(dev_priv) + 0x62020)
> > @@ -12519,12 +12571,17 @@ enum skl_power_gate {
> > #define PMFLUSH_GAPL3UNBLOCK (1 << 21)
> > #define PMFLUSHDONE_LNEBLK (1 << 22)
> >
> > +#define XEHP_L3NODEARBCFG _MMIO(0xb0b4)
> > +#define XEHP_LNESPARE
> REG_BIT(19)
> > +
> > #define GEN12_GLOBAL_MOCS(i) _MMIO(0x4000 + (i) * 4) /* Global
> MOCS regs */
> >
> > #define GEN12_GSMBASE
> _MMIO(0x108100)
> > #define GEN12_DSMBASE
> _MMIO(0x1080C0)
> >
> > #define XEHP_CLOCK_GATE_DIS _MMIO(0x101014)
> > +#define SGSI_SIDECLK_DIS REG_BIT(17)
> > +#define SGGI_DIS REG_BIT(15)
> > #define SGR_DIS REG_BIT(13)
> >
> > /* gamt regs */
> > @@ -12903,4 +12960,7 @@ enum skl_power_gate {
> > #define CLKGATE_DIS_MISC
> _MMIO(0x46534)
> > #define CLKGATE_DIS_MISC_DMASC_GATING_DIS REG_BIT(21)
> >
> > +#define SLICE_COMMON_ECO_CHICKEN1
> _MMIO(0x731C)
> > +#define MSC_MSAA_REODER_BUF_BYPASS_DISABLE
> REG_BIT(14)
> > +
> > #endif /* _I915_REG_H_ */
> > diff --git a/drivers/gpu/drm/i915/intel_pm.c
> b/drivers/gpu/drm/i915/intel_pm.c
> > index 16fa3306d83d..a1d9a6ac3e49 100644
> > --- a/drivers/gpu/drm/i915/intel_pm.c
> > +++ b/drivers/gpu/drm/i915/intel_pm.c
> > @@ -7486,6 +7486,22 @@ static void xehpsdv_init_clock_gating(struct
> drm_i915_private *dev_priv)
> > intel_uncore_rmw(&dev_priv->uncore,
> XEHP_CLOCK_GATE_DIS, 0, SGR_DIS);
> > }
> >
> > +static void dg2_init_clock_gating(struct drm_i915_private *i915)
> > +{
> > + /* Wa_22010954014:dg2_g10 */
> > + if (IS_DG2_G10(i915))
> > + intel_uncore_rmw(&i915->uncore,
> XEHP_CLOCK_GATE_DIS, 0,
> > +
> SGSI_SIDECLK_DIS);
> > +
> > + /*
> > + * Wa_14010733611:dg2_g10
> > + * Wa_22010146351:dg2_g10
> > + */
> > + if (IS_DG2_GRAPHICS_STEP(i915, G10, STEP_A0, STEP_B0))
> > + intel_uncore_rmw(&i915->uncore,
> XEHP_CLOCK_GATE_DIS, 0,
> > + SGR_DIS |
> SGGI_DIS);
> > +}
> > +
> > static void cnp_init_clock_gating(struct drm_i915_private *dev_priv)
> > {
> > if (!HAS_PCH_CNP(dev_priv))
> > @@ -7896,6 +7912,7 @@ static const struct drm_i915_clock_gating_funcs
> platform##_clock_gating_funcs =
> > .init_clock_gating = platform##_init_clock_gating,
> \
> > }
> >
> > +CG_FUNCS(dg2);
> > CG_FUNCS(xehpsdv);
> > CG_FUNCS(adlp);
> > CG_FUNCS(dg1);
> > @@ -7933,7 +7950,9 @@ CG_FUNCS(nop);
> > */
> > void intel_init_clock_gating_hooks(struct drm_i915_private *dev_priv)
> > {
> > - if (IS_XEHPSDV(dev_priv))
> > + if (IS_DG2(dev_priv))
> > + dev_priv->clock_gating_funcs =
> &dg2_clock_gating_funcs;
> > + else if (IS_XEHPSDV(dev_priv))
> > dev_priv->clock_gating_funcs =
> &xehpsdv_clock_gating_funcs;
> > else if (IS_ALDERLAKE_P(dev_priv))
> > dev_priv->clock_gating_funcs =
> &adlp_clock_gating_funcs;
> > --
> > 2.33.0
> >
