[Intel-gfx] [PATCH v2 09/50] drm/i915/xehp: Xe_HP forcewake support

Matt Atwood matthew.s.atwood at intel.com
Tue Jul 20 20:57:29 UTC 2021


On Tue, Jul 13, 2021 at 08:14:59PM -0700, Matt Roper wrote:
> Implement Xe_HP forcewake handling.  While we're at it, let's reorder to
> the forcewake assignment if/else ladder to match our usual driver
> conventions.
> 
> Co-authored-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Signed-off-by: Stuart Summers <stuart.summers at intel.com>
> Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
Reviewed-by: Matt Atwood <matthew.s.atwood at intel.com>
> ---
>  .../drm/i915/gt/intel_execlists_submission.c  |   4 +
>  drivers/gpu/drm/i915/intel_uncore.c           | 336 +++++++++++++++---
>  drivers/gpu/drm/i915/intel_uncore.h           |  14 +-
>  drivers/gpu/drm/i915/selftests/intel_uncore.c |   2 +
>  4 files changed, 302 insertions(+), 54 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_execlists_submission.c b/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
> index 56e25090da67..cf1ac0010056 100644
> --- a/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
> +++ b/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
> @@ -3306,6 +3306,10 @@ int intel_execlists_submission_setup(struct intel_engine_cs *engine)
>  			i915_mmio_reg_offset(RING_EXECLIST_SQ_CONTENTS(base));
>  		execlists->ctrl_reg = uncore->regs +
>  			i915_mmio_reg_offset(RING_EXECLIST_CONTROL(base));
> +
> +		engine->fw_domain = intel_uncore_forcewake_for_reg(engine->uncore,
> +				    RING_EXECLIST_CONTROL(engine->mmio_base),
> +				    FW_REG_WRITE);
>  	} else {
>  		execlists->submit_reg = uncore->regs +
>  			i915_mmio_reg_offset(RING_ELSP(base));
> diff --git a/drivers/gpu/drm/i915/intel_uncore.c b/drivers/gpu/drm/i915/intel_uncore.c
> index ee1c6fbc3d97..8cf53f54559d 100644
> --- a/drivers/gpu/drm/i915/intel_uncore.c
> +++ b/drivers/gpu/drm/i915/intel_uncore.c
> @@ -24,6 +24,8 @@
>  #include <linux/pm_runtime.h>
>  #include <asm/iosf_mbi.h>
>  
> +#include "gt/intel_lrc_reg.h" /* for shadow reg list */
> +
>  #include "i915_drv.h"
>  #include "i915_trace.h"
>  #include "i915_vgpu.h"
> @@ -68,8 +70,14 @@ static const char * const forcewake_domain_names[] = {
>  	"vdbox1",
>  	"vdbox2",
>  	"vdbox3",
> +	"vdbox4",
> +	"vdbox5",
> +	"vdbox6",
> +	"vdbox7",
>  	"vebox0",
>  	"vebox1",
> +	"vebox2",
> +	"vebox3",
>  };
>  
>  const char *
> @@ -952,30 +960,80 @@ static const i915_reg_t gen8_shadowed_regs[] = {
>  };
>  
>  static const i915_reg_t gen11_shadowed_regs[] = {
> -	RING_TAIL(RENDER_RING_BASE),		/* 0x2000 (base) */
> -	GEN6_RPNSWREQ,				/* 0xA008 */
> -	GEN6_RC_VIDEO_FREQ,			/* 0xA00C */
> -	RING_TAIL(BLT_RING_BASE),		/* 0x22000 (base) */
> -	RING_TAIL(GEN11_BSD_RING_BASE),		/* 0x1C0000 (base) */
> -	RING_TAIL(GEN11_BSD2_RING_BASE),	/* 0x1C4000 (base) */
> -	RING_TAIL(GEN11_VEBOX_RING_BASE),	/* 0x1C8000 (base) */
> -	RING_TAIL(GEN11_BSD3_RING_BASE),	/* 0x1D0000 (base) */
> -	RING_TAIL(GEN11_BSD4_RING_BASE),	/* 0x1D4000 (base) */
> -	RING_TAIL(GEN11_VEBOX2_RING_BASE),	/* 0x1D8000 (base) */
> +	RING_TAIL(RENDER_RING_BASE),			/* 0x2000 (base) */
> +	RING_EXECLIST_CONTROL(RENDER_RING_BASE),        /* 0x2550 */
> +	GEN6_RPNSWREQ,					/* 0xA008 */
> +	GEN6_RC_VIDEO_FREQ,				/* 0xA00C */
> +	RING_TAIL(BLT_RING_BASE),			/* 0x22000 (base) */
> +	RING_EXECLIST_CONTROL(BLT_RING_BASE),		/* 0x22550 */
> +	RING_TAIL(GEN11_BSD_RING_BASE),			/* 0x1C0000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD_RING_BASE),	/* 0x1C0550 */
> +	RING_TAIL(GEN11_BSD2_RING_BASE),		/* 0x1C4000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD2_RING_BASE),	/* 0x1C4550 */
> +	RING_TAIL(GEN11_VEBOX_RING_BASE),		/* 0x1C8000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_VEBOX_RING_BASE),	/* 0x1C8550 */
> +	RING_TAIL(GEN11_BSD3_RING_BASE),		/* 0x1D0000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD3_RING_BASE),	/* 0x1D0550 */
> +	RING_TAIL(GEN11_BSD4_RING_BASE),		/* 0x1D4000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD4_RING_BASE),	/* 0x1D4550 */
> +	RING_TAIL(GEN11_VEBOX2_RING_BASE),		/* 0x1D8000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_VEBOX2_RING_BASE),	/* 0x1D8550 */
>  	/* TODO: Other registers are not yet used */
>  };
>  
>  static const i915_reg_t gen12_shadowed_regs[] = {
> -	RING_TAIL(RENDER_RING_BASE),		/* 0x2000 (base) */
> -	GEN6_RPNSWREQ,				/* 0xA008 */
> -	GEN6_RC_VIDEO_FREQ,			/* 0xA00C */
> -	RING_TAIL(BLT_RING_BASE),		/* 0x22000 (base) */
> -	RING_TAIL(GEN11_BSD_RING_BASE),		/* 0x1C0000 (base) */
> -	RING_TAIL(GEN11_BSD2_RING_BASE),	/* 0x1C4000 (base) */
> -	RING_TAIL(GEN11_VEBOX_RING_BASE),	/* 0x1C8000 (base) */
> -	RING_TAIL(GEN11_BSD3_RING_BASE),	/* 0x1D0000 (base) */
> -	RING_TAIL(GEN11_BSD4_RING_BASE),	/* 0x1D4000 (base) */
> -	RING_TAIL(GEN11_VEBOX2_RING_BASE),	/* 0x1D8000 (base) */
> +	RING_TAIL(RENDER_RING_BASE),			/* 0x2000 (base) */
> +	RING_EXECLIST_CONTROL(RENDER_RING_BASE),	/* 0x2550 */
> +	GEN6_RPNSWREQ,					/* 0xA008 */
> +	GEN6_RC_VIDEO_FREQ,				/* 0xA00C */
> +	RING_TAIL(BLT_RING_BASE),			/* 0x22000 (base) */
> +	RING_EXECLIST_CONTROL(BLT_RING_BASE),		/* 0x22550 */
> +	RING_TAIL(GEN11_BSD_RING_BASE),			/* 0x1C0000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD_RING_BASE),	/* 0x1C0550 */
> +	RING_TAIL(GEN11_BSD2_RING_BASE),		/* 0x1C4000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD2_RING_BASE),	/* 0x1C4550 */
> +	RING_TAIL(GEN11_VEBOX_RING_BASE),		/* 0x1C8000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_VEBOX_RING_BASE),	/* 0x1C8550 */
> +	RING_TAIL(GEN11_BSD3_RING_BASE),		/* 0x1D0000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD3_RING_BASE),	/* 0x1D0550 */
> +	RING_TAIL(GEN11_BSD4_RING_BASE),		/* 0x1D4000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD4_RING_BASE),	/* 0x1D4550 */
> +	RING_TAIL(GEN11_VEBOX2_RING_BASE),		/* 0x1D8000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_VEBOX2_RING_BASE),	/* 0x1D8550 */
> +	/* TODO: Other registers are not yet used */
> +};
> +
> +static const i915_reg_t xehp_shadowed_regs[] = {
> +	RING_TAIL(RENDER_RING_BASE),			/* 0x2000 (base) */
> +	RING_EXECLIST_CONTROL(RENDER_RING_BASE),        /* 0x2550 */
> +	GEN6_RPNSWREQ,					/* 0xA008 */
> +	GEN6_RC_VIDEO_FREQ,				/* 0xA00C */
> +	RING_TAIL(BLT_RING_BASE),			/* 0x22000 (base) */
> +	RING_EXECLIST_CONTROL(BLT_RING_BASE),		/* 0x22550 */
> +	RING_TAIL(GEN11_BSD_RING_BASE),			/* 0x1C0000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD_RING_BASE),	/* 0x1C0550 */
> +	RING_TAIL(GEN11_BSD2_RING_BASE),		/* 0x1C4000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD2_RING_BASE),	/* 0x1C4550 */
> +	RING_TAIL(GEN11_VEBOX_RING_BASE),		/* 0x1C8000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_VEBOX_RING_BASE),	/* 0x1C8550 */
> +	RING_TAIL(GEN11_BSD3_RING_BASE),		/* 0x1D0000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD3_RING_BASE),	/* 0x1D0550 */
> +	RING_TAIL(GEN11_BSD4_RING_BASE),		/* 0x1D4000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_BSD4_RING_BASE),	/* 0x1D4550 */
> +	RING_TAIL(GEN11_VEBOX2_RING_BASE),		/* 0x1D8000 (base) */
> +	RING_EXECLIST_CONTROL(GEN11_VEBOX2_RING_BASE),	/* 0x1D8550 */
> +	RING_TAIL(XEHP_BSD5_RING_BASE),			/* 0x1E0000 (base) */
> +	RING_EXECLIST_CONTROL(XEHP_BSD5_RING_BASE),	/* 0x1E0550 */
> +	RING_TAIL(XEHP_BSD6_RING_BASE),			/* 0x1E4000 (base) */
> +	RING_EXECLIST_CONTROL(XEHP_BSD6_RING_BASE),	/* 0x1E4550 */
> +	RING_TAIL(XEHP_VEBOX3_RING_BASE),		/* 0x1E8000 (base) */
> +	RING_EXECLIST_CONTROL(XEHP_VEBOX3_RING_BASE),	/* 0x1E8550 */
> +	RING_TAIL(XEHP_BSD7_RING_BASE),			/* 0x1F0000 (base) */
> +	RING_EXECLIST_CONTROL(XEHP_BSD7_RING_BASE),	/* 0x1F0550 */
> +	RING_TAIL(XEHP_BSD8_RING_BASE),			/* 0x1F4000 (base) */
> +	RING_EXECLIST_CONTROL(XEHP_BSD8_RING_BASE),	/* 0x1F4550 */
> +	RING_TAIL(XEHP_VEBOX4_RING_BASE),		/* 0x1F8000 (base) */
> +	RING_EXECLIST_CONTROL(XEHP_VEBOX4_RING_BASE),	/* 0x1F8550 */
>  	/* TODO: Other registers are not yet used */
>  };
>  
> @@ -991,17 +1049,18 @@ static int mmio_reg_cmp(u32 key, const i915_reg_t *reg)
>  		return 0;
>  }
>  
> -#define __is_genX_shadowed(x) \
> -static bool is_gen##x##_shadowed(u32 offset) \
> +#define __is_X_shadowed(x) \
> +static bool is_##x##_shadowed(u32 offset) \
>  { \
> -	const i915_reg_t *regs = gen##x##_shadowed_regs; \
> -	return BSEARCH(offset, regs, ARRAY_SIZE(gen##x##_shadowed_regs), \
> +	const i915_reg_t *regs = x##_shadowed_regs; \
> +	return BSEARCH(offset, regs, ARRAY_SIZE(x##_shadowed_regs), \
>  		       mmio_reg_cmp); \
>  }
>  
> -__is_genX_shadowed(8)
> -__is_genX_shadowed(11)
> -__is_genX_shadowed(12)
> +__is_X_shadowed(gen8)
> +__is_X_shadowed(gen11)
> +__is_X_shadowed(gen12)
> +__is_X_shadowed(xehp)
>  
>  static enum forcewake_domains
>  gen6_reg_write_fw_domains(struct intel_uncore *uncore, i915_reg_t reg)
> @@ -1065,6 +1124,15 @@ static const struct intel_forcewake_range __chv_fw_ranges[] = {
>  	__fwd; \
>  })
>  
> +#define __xehp_fwtable_reg_write_fw_domains(uncore, offset) \
> +({ \
> +	enum forcewake_domains __fwd = 0; \
> +	const u32 __offset = (offset); \
> +	if (!is_xehp_shadowed(__offset)) \
> +		__fwd = find_fw_domain(uncore, __offset); \
> +	__fwd; \
> +})
> +
>  /* *Must* be sorted by offset ranges! See intel_fw_table_check(). */
>  static const struct intel_forcewake_range __gen9_fw_ranges[] = {
>  	GEN_FW_RANGE(0x0, 0xaff, FORCEWAKE_GT),
> @@ -1249,6 +1317,145 @@ static const struct intel_forcewake_range __gen12_fw_ranges[] = {
>  		0x1d3f00 - 0x1d3fff: VD2 */
>  };
>  
> +/* *Must* be sorted by offset ranges! See intel_fw_table_check(). */
> +static const struct intel_forcewake_range __xehp_fw_ranges[] = {
> +	GEN_FW_RANGE(0x0, 0x1fff, 0), /*
> +		  0x0 -  0xaff: reserved
> +		0xb00 - 0x1fff: always on */
> +	GEN_FW_RANGE(0x2000, 0x26ff, FORCEWAKE_RENDER),
> +	GEN_FW_RANGE(0x2700, 0x4aff, FORCEWAKE_GT),
> +	GEN_FW_RANGE(0x4b00, 0x51ff, 0), /*
> +		0x4b00 - 0x4fff: reserved
> +		0x5000 - 0x51ff: always on */
> +	GEN_FW_RANGE(0x5200, 0x7fff, FORCEWAKE_RENDER),
> +	GEN_FW_RANGE(0x8000, 0x813f, FORCEWAKE_GT),
> +	GEN_FW_RANGE(0x8140, 0x815f, FORCEWAKE_RENDER),
> +	GEN_FW_RANGE(0x8160, 0x81ff, 0), /*
> +		0x8160 - 0x817f: reserved
> +		0x8180 - 0x81ff: always on */
> +	GEN_FW_RANGE(0x8200, 0x82ff, FORCEWAKE_GT),
> +	GEN_FW_RANGE(0x8300, 0x84ff, FORCEWAKE_RENDER),
> +	GEN_FW_RANGE(0x8500, 0x94cf, FORCEWAKE_GT), /*
> +		0x8500 - 0x87ff: gt
> +		0x8800 - 0x8fff: reserved
> +		0x9000 - 0x947f: gt
> +		0x9480 - 0x94cf: reserved */
> +	GEN_FW_RANGE(0x94d0, 0x955f, FORCEWAKE_RENDER),
> +	GEN_FW_RANGE(0x9560, 0x97ff, 0), /*
> +		0x9560 - 0x95ff: always on
> +		0x9600 - 0x97ff: reserved */
> +	GEN_FW_RANGE(0x9800, 0xcfff, FORCEWAKE_GT), /*
> +		0x9800 - 0xb4ff: gt
> +		0xb500 - 0xbfff: reserved
> +		0xc000 - 0xcfff: gt */
> +	GEN_FW_RANGE(0xd000, 0xd7ff, 0),
> +	GEN_FW_RANGE(0xd800, 0xdbff, FORCEWAKE_GT),
> +	GEN_FW_RANGE(0xdc00, 0xdcff, FORCEWAKE_RENDER),
> +	GEN_FW_RANGE(0xdd00, 0xde7f, FORCEWAKE_GT), /*
> +		0xdd00 - 0xddff: gt
> +		0xde00 - 0xde7f: reserved */
> +	GEN_FW_RANGE(0xde80, 0xe8ff, FORCEWAKE_RENDER), /*
> +		0xde80 - 0xdfff: render
> +		0xe000 - 0xe0ff: reserved
> +		0xe100 - 0xe8ff: render */
> +	GEN_FW_RANGE(0xe900, 0xffff, FORCEWAKE_GT), /*
> +		0xe900 - 0xe9ff: gt
> +		0xea00 - 0xefff: reserved
> +		0xf000 - 0xffff: gt */
> +	GEN_FW_RANGE(0x10000, 0x13fff, 0), /*
> +		0x10000 - 0x11fff: reserved
> +		0x12000 - 0x127ff: always on
> +		0x12800 - 0x13fff: reserved */
> +	GEN_FW_RANGE(0x14000, 0x141ff, FORCEWAKE_MEDIA_VDBOX0),
> +	GEN_FW_RANGE(0x14200, 0x143ff, FORCEWAKE_MEDIA_VDBOX2),
> +	GEN_FW_RANGE(0x14400, 0x145ff, FORCEWAKE_MEDIA_VDBOX4),
> +	GEN_FW_RANGE(0x14600, 0x147ff, FORCEWAKE_MEDIA_VDBOX6),
> +	GEN_FW_RANGE(0x14800, 0x1ffff, FORCEWAKE_RENDER), /*
> +		0x14800 - 0x14fff: render
> +		0x15000 - 0x16dff: reserved
> +		0x16e00 - 0x1ffff: render */
> +	GEN_FW_RANGE(0x20000, 0x21fff, FORCEWAKE_MEDIA_VDBOX0), /*
> +		0x20000 - 0x20fff: VD0
> +		0x21000 - 0x21fff: reserved */
> +	GEN_FW_RANGE(0x22000, 0x23fff, FORCEWAKE_GT),
> +	GEN_FW_RANGE(0x24000, 0x2417f, 0), /*
> +		0x24000 - 0x2407f: always on
> +		0x24080 - 0x2417f: reserved */
> +	GEN_FW_RANGE(0x24180, 0x249ff, FORCEWAKE_GT), /*
> +		0x24180 - 0x241ff: gt
> +		0x24200 - 0x249ff: reserved */
> +	GEN_FW_RANGE(0x24a00, 0x251ff, FORCEWAKE_RENDER), /*
> +		0x24a00 - 0x24a7f: render
> +		0x24a80 - 0x251ff: reserved */
> +	GEN_FW_RANGE(0x25200, 0x25fff, FORCEWAKE_GT), /*
> +		0x25200 - 0x252ff: gt
> +		0x25300 - 0x25fff: reserved */
> +	GEN_FW_RANGE(0x26000, 0x2ffff, FORCEWAKE_RENDER), /*
> +		0x26000 - 0x27fff: render
> +		0x28000 - 0x29fff: reserved
> +		0x2a000 - 0x2ffff: undocumented */
> +	GEN_FW_RANGE(0x30000, 0x3ffff, FORCEWAKE_GT),
> +	GEN_FW_RANGE(0x40000, 0x1bffff, 0),
> +	GEN_FW_RANGE(0x1c0000, 0x1c3fff, FORCEWAKE_MEDIA_VDBOX0), /*
> +		0x1c0000 - 0x1c2bff: VD0
> +		0x1c2c00 - 0x1c2cff: reserved
> +		0x1c2d00 - 0x1c2dff: VD0
> +		0x1c2e00 - 0x1c3eff: reserved
> +		0x1c3f00 - 0x1c3fff: VD0 */
> +	GEN_FW_RANGE(0x1c4000, 0x1c7fff, FORCEWAKE_MEDIA_VDBOX1), /*
> +		0x1c4000 - 0x1c6bff: VD1
> +		0x1c6c00 - 0x1c6cff: reserved
> +		0x1c6d00 - 0x1c6dff: VD1
> +		0x1c6e00 - 0x1c7fff: reserved */
> +	GEN_FW_RANGE(0x1c8000, 0x1cbfff, FORCEWAKE_MEDIA_VEBOX0), /*
> +		0x1c8000 - 0x1ca0ff: VE0
> +		0x1ca100 - 0x1cbfff: reserved */
> +	GEN_FW_RANGE(0x1cc000, 0x1ccfff, FORCEWAKE_MEDIA_VDBOX0),
> +	GEN_FW_RANGE(0x1cd000, 0x1cdfff, FORCEWAKE_MEDIA_VDBOX2),
> +	GEN_FW_RANGE(0x1ce000, 0x1cefff, FORCEWAKE_MEDIA_VDBOX4),
> +	GEN_FW_RANGE(0x1cf000, 0x1cffff, FORCEWAKE_MEDIA_VDBOX6),
> +	GEN_FW_RANGE(0x1d0000, 0x1d3fff, FORCEWAKE_MEDIA_VDBOX2), /*
> +		0x1d0000 - 0x1d2bff: VD2
> +		0x1d2c00 - 0x1d2cff: reserved
> +		0x1d2d00 - 0x1d2dff: VD2
> +		0x1d2e00 - 0x1d3eff: reserved
> +		0x1d3f00 - 0x1d3fff: VD2 */
> +	GEN_FW_RANGE(0x1d4000, 0x1d7fff, FORCEWAKE_MEDIA_VDBOX3), /*
> +		0x1d4000 - 0x1d6bff: VD3
> +		0x1d6c00 - 0x1d6cff: reserved
> +		0x1d6d00 - 0x1d6dff: VD3
> +		0x1d6e00 - 0x1d7fff: reserved */
> +	GEN_FW_RANGE(0x1d8000, 0x1dffff, FORCEWAKE_MEDIA_VEBOX1), /*
> +		0x1d8000 - 0x1da0ff: VE1
> +		0x1da100 - 0x1dffff: reserved */
> +	GEN_FW_RANGE(0x1e0000, 0x1e3fff, FORCEWAKE_MEDIA_VDBOX4), /*
> +		0x1e0000 - 0x1e2bff: VD4
> +		0x1e2c00 - 0x1e2cff: reserved
> +		0x1e2d00 - 0x1e2dff: VD4
> +		0x1e2e00 - 0x1e3eff: reserved
> +		0x1e3f00 - 0x1e3fff: VD4 */
> +	GEN_FW_RANGE(0x1e4000, 0x1e7fff, FORCEWAKE_MEDIA_VDBOX5), /*
> +		0x1e4000 - 0x1e6bff: VD5
> +		0x1e6c00 - 0x1e6cff: reserved
> +		0x1e6d00 - 0x1e6dff: VD5
> +		0x1e6e00 - 0x1e7fff: reserved */
> +	GEN_FW_RANGE(0x1e8000, 0x1effff, FORCEWAKE_MEDIA_VEBOX2), /*
> +		0x1e8000 - 0x1ea0ff: VE2
> +		0x1ea100 - 0x1effff: reserved */
> +	GEN_FW_RANGE(0x1f0000, 0x1f3fff, FORCEWAKE_MEDIA_VDBOX6), /*
> +		0x1f0000 - 0x1f2bff: VD6
> +		0x1f2c00 - 0x1f2cff: reserved
> +		0x1f2d00 - 0x1f2dff: VD6
> +		0x1f2e00 - 0x1f3eff: reserved
> +		0x1f3f00 - 0x1f3fff: VD6 */
> +	GEN_FW_RANGE(0x1f4000, 0x1f7fff, FORCEWAKE_MEDIA_VDBOX7), /*
> +		0x1f4000 - 0x1f6bff: VD7
> +		0x1f6c00 - 0x1f6cff: reserved
> +		0x1f6d00 - 0x1f6dff: VD7
> +		0x1f6e00 - 0x1f7fff: reserved */
> +	GEN_FW_RANGE(0x1f8000, 0x1fa0ff, FORCEWAKE_MEDIA_VEBOX3),
> +};
> +
>  static void
>  ilk_dummy_write(struct intel_uncore *uncore)
>  {
> @@ -1502,6 +1709,7 @@ __gen_write(func, 8) \
>  __gen_write(func, 16) \
>  __gen_write(func, 32)
>  
> +__gen_reg_write_funcs(xehp_fwtable);
>  __gen_reg_write_funcs(gen12_fwtable);
>  __gen_reg_write_funcs(gen11_fwtable);
>  __gen_reg_write_funcs(fwtable);
> @@ -1582,8 +1790,14 @@ static int __fw_domain_init(struct intel_uncore *uncore,
>  	BUILD_BUG_ON(FORCEWAKE_MEDIA_VDBOX1 != (1 << FW_DOMAIN_ID_MEDIA_VDBOX1));
>  	BUILD_BUG_ON(FORCEWAKE_MEDIA_VDBOX2 != (1 << FW_DOMAIN_ID_MEDIA_VDBOX2));
>  	BUILD_BUG_ON(FORCEWAKE_MEDIA_VDBOX3 != (1 << FW_DOMAIN_ID_MEDIA_VDBOX3));
> +	BUILD_BUG_ON(FORCEWAKE_MEDIA_VDBOX4 != (1 << FW_DOMAIN_ID_MEDIA_VDBOX4));
> +	BUILD_BUG_ON(FORCEWAKE_MEDIA_VDBOX5 != (1 << FW_DOMAIN_ID_MEDIA_VDBOX5));
> +	BUILD_BUG_ON(FORCEWAKE_MEDIA_VDBOX6 != (1 << FW_DOMAIN_ID_MEDIA_VDBOX6));
> +	BUILD_BUG_ON(FORCEWAKE_MEDIA_VDBOX7 != (1 << FW_DOMAIN_ID_MEDIA_VDBOX7));
>  	BUILD_BUG_ON(FORCEWAKE_MEDIA_VEBOX0 != (1 << FW_DOMAIN_ID_MEDIA_VEBOX0));
>  	BUILD_BUG_ON(FORCEWAKE_MEDIA_VEBOX1 != (1 << FW_DOMAIN_ID_MEDIA_VEBOX1));
> +	BUILD_BUG_ON(FORCEWAKE_MEDIA_VEBOX2 != (1 << FW_DOMAIN_ID_MEDIA_VEBOX2));
> +	BUILD_BUG_ON(FORCEWAKE_MEDIA_VEBOX3 != (1 << FW_DOMAIN_ID_MEDIA_VEBOX3));
>  
>  	d->mask = BIT(domain_id);
>  
> @@ -1870,36 +2084,36 @@ static int uncore_forcewake_init(struct intel_uncore *uncore)
>  		return ret;
>  	forcewake_early_sanitize(uncore, 0);
>  
> -	if (IS_GRAPHICS_VER(i915, 6, 7)) {
> -		ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen6);
> -
> -		if (IS_VALLEYVIEW(i915)) {
> -			ASSIGN_FW_DOMAINS_TABLE(uncore, __vlv_fw_ranges);
> -			ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable);
> -		} else {
> -			ASSIGN_READ_MMIO_VFUNCS(uncore, gen6);
> -		}
> -	} else if (GRAPHICS_VER(i915) == 8) {
> -		if (IS_CHERRYVIEW(i915)) {
> -			ASSIGN_FW_DOMAINS_TABLE(uncore, __chv_fw_ranges);
> -			ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable);
> -			ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable);
> -		} else {
> -			ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen8);
> -			ASSIGN_READ_MMIO_VFUNCS(uncore, gen6);
> -		}
> -	} else if (IS_GRAPHICS_VER(i915, 9, 10)) {
> -		ASSIGN_FW_DOMAINS_TABLE(uncore, __gen9_fw_ranges);
> -		ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable);
> -		ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable);
> -	} else if (GRAPHICS_VER(i915) == 11) {
> -		ASSIGN_FW_DOMAINS_TABLE(uncore, __gen11_fw_ranges);
> -		ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen11_fwtable);
> +	if (GRAPHICS_VER_FULL(i915) >= IP_VER(12, 50)) {
> +		ASSIGN_FW_DOMAINS_TABLE(uncore, __xehp_fw_ranges);
> +		ASSIGN_WRITE_MMIO_VFUNCS(uncore, xehp_fwtable);
>  		ASSIGN_READ_MMIO_VFUNCS(uncore, gen11_fwtable);
> -	} else {
> +	} else if (GRAPHICS_VER(i915) >= 12) {
>  		ASSIGN_FW_DOMAINS_TABLE(uncore, __gen12_fw_ranges);
>  		ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen12_fwtable);
>  		ASSIGN_READ_MMIO_VFUNCS(uncore, gen12_fwtable);
> +	} else if (GRAPHICS_VER(i915) == 11) {
> +		ASSIGN_FW_DOMAINS_TABLE(uncore, __gen11_fw_ranges);
> +		ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen11_fwtable);
> +		ASSIGN_READ_MMIO_VFUNCS(uncore, gen11_fwtable);
> +	} else if (IS_GRAPHICS_VER(i915, 9, 10)) {
> +		ASSIGN_FW_DOMAINS_TABLE(uncore, __gen9_fw_ranges);
> +		ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable);
> +		ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable);
> +	} else if (IS_CHERRYVIEW(i915)) {
> +		ASSIGN_FW_DOMAINS_TABLE(uncore, __chv_fw_ranges);
> +		ASSIGN_WRITE_MMIO_VFUNCS(uncore, fwtable);
> +		ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable);
> +	} else if (GRAPHICS_VER(i915) == 8) {
> +		ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen8);
> +		ASSIGN_READ_MMIO_VFUNCS(uncore, gen6);
> +	} else if (IS_VALLEYVIEW(i915)) {
> +		ASSIGN_FW_DOMAINS_TABLE(uncore, __vlv_fw_ranges);
> +		ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen6);
> +		ASSIGN_READ_MMIO_VFUNCS(uncore, fwtable);
> +	} else if (IS_GRAPHICS_VER(i915, 6, 7)) {
> +		ASSIGN_WRITE_MMIO_VFUNCS(uncore, gen6);
> +		ASSIGN_READ_MMIO_VFUNCS(uncore, gen6);
>  	}
>  
>  	uncore->pmic_bus_access_nb.notifier_call = i915_pmic_bus_access_notifier;
> @@ -1988,6 +2202,22 @@ void intel_uncore_prune_engine_fw_domains(struct intel_uncore *uncore,
>  		if (HAS_ENGINE(gt, _VCS(i)))
>  			continue;
>  
> +		/*
> +		 * Starting with XeHP, the power well for an even-numbered
> +		 * VDBOX is also used for shared units within the
> +		 * media slice such as SFC.  So even if the engine
> +		 * itself is fused off, we still need to initialize
> +		 * the forcewake domain if any of the other engines
> +		 * in the same media slice are present.
> +		 */
> +		if (GRAPHICS_VER_FULL(uncore->i915) >= IP_VER(12, 50) && i % 2 == 0) {
> +			if ((i + 1 < I915_MAX_VCS) && HAS_ENGINE(gt, _VCS(i + 1)))
> +				continue;
> +
> +			if (HAS_ENGINE(gt, _VECS(i / 2)))
> +				continue;
> +		}
> +
>  		if (fw_domains & BIT(domain_id))
>  			fw_domain_fini(uncore, domain_id);
>  	}
> diff --git a/drivers/gpu/drm/i915/intel_uncore.h b/drivers/gpu/drm/i915/intel_uncore.h
> index a18bdb57af7b..3c0b0a8b5250 100644
> --- a/drivers/gpu/drm/i915/intel_uncore.h
> +++ b/drivers/gpu/drm/i915/intel_uncore.h
> @@ -52,8 +52,14 @@ enum forcewake_domain_id {
>  	FW_DOMAIN_ID_MEDIA_VDBOX1,
>  	FW_DOMAIN_ID_MEDIA_VDBOX2,
>  	FW_DOMAIN_ID_MEDIA_VDBOX3,
> +	FW_DOMAIN_ID_MEDIA_VDBOX4,
> +	FW_DOMAIN_ID_MEDIA_VDBOX5,
> +	FW_DOMAIN_ID_MEDIA_VDBOX6,
> +	FW_DOMAIN_ID_MEDIA_VDBOX7,
>  	FW_DOMAIN_ID_MEDIA_VEBOX0,
>  	FW_DOMAIN_ID_MEDIA_VEBOX1,
> +	FW_DOMAIN_ID_MEDIA_VEBOX2,
> +	FW_DOMAIN_ID_MEDIA_VEBOX3,
>  
>  	FW_DOMAIN_ID_COUNT
>  };
> @@ -66,10 +72,16 @@ enum forcewake_domains {
>  	FORCEWAKE_MEDIA_VDBOX1	= BIT(FW_DOMAIN_ID_MEDIA_VDBOX1),
>  	FORCEWAKE_MEDIA_VDBOX2	= BIT(FW_DOMAIN_ID_MEDIA_VDBOX2),
>  	FORCEWAKE_MEDIA_VDBOX3	= BIT(FW_DOMAIN_ID_MEDIA_VDBOX3),
> +	FORCEWAKE_MEDIA_VDBOX4	= BIT(FW_DOMAIN_ID_MEDIA_VDBOX4),
> +	FORCEWAKE_MEDIA_VDBOX5	= BIT(FW_DOMAIN_ID_MEDIA_VDBOX5),
> +	FORCEWAKE_MEDIA_VDBOX6	= BIT(FW_DOMAIN_ID_MEDIA_VDBOX6),
> +	FORCEWAKE_MEDIA_VDBOX7	= BIT(FW_DOMAIN_ID_MEDIA_VDBOX7),
>  	FORCEWAKE_MEDIA_VEBOX0	= BIT(FW_DOMAIN_ID_MEDIA_VEBOX0),
>  	FORCEWAKE_MEDIA_VEBOX1	= BIT(FW_DOMAIN_ID_MEDIA_VEBOX1),
> +	FORCEWAKE_MEDIA_VEBOX2	= BIT(FW_DOMAIN_ID_MEDIA_VEBOX2),
> +	FORCEWAKE_MEDIA_VEBOX3	= BIT(FW_DOMAIN_ID_MEDIA_VEBOX3),
>  
> -	FORCEWAKE_ALL = BIT(FW_DOMAIN_ID_COUNT) - 1
> +	FORCEWAKE_ALL = BIT(FW_DOMAIN_ID_COUNT) - 1,
>  };
>  
>  struct intel_uncore_funcs {
> diff --git a/drivers/gpu/drm/i915/selftests/intel_uncore.c b/drivers/gpu/drm/i915/selftests/intel_uncore.c
> index 8ef9e6a4ad05..720b60853f8b 100644
> --- a/drivers/gpu/drm/i915/selftests/intel_uncore.c
> +++ b/drivers/gpu/drm/i915/selftests/intel_uncore.c
> @@ -68,6 +68,7 @@ static int intel_shadow_table_check(void)
>  		{ gen8_shadowed_regs, ARRAY_SIZE(gen8_shadowed_regs) },
>  		{ gen11_shadowed_regs, ARRAY_SIZE(gen11_shadowed_regs) },
>  		{ gen12_shadowed_regs, ARRAY_SIZE(gen12_shadowed_regs) },
> +		{ xehp_shadowed_regs, ARRAY_SIZE(xehp_shadowed_regs) },
>  	};
>  	const i915_reg_t *reg;
>  	unsigned int i, j;
> @@ -103,6 +104,7 @@ int intel_uncore_mock_selftests(void)
>  		{ __gen9_fw_ranges, ARRAY_SIZE(__gen9_fw_ranges), true },
>  		{ __gen11_fw_ranges, ARRAY_SIZE(__gen11_fw_ranges), true },
>  		{ __gen12_fw_ranges, ARRAY_SIZE(__gen12_fw_ranges), true },
> +		{ __xehp_fw_ranges, ARRAY_SIZE(__xehp_fw_ranges), true },
>  	};
>  	int err, i;
>  
> -- 
> 2.25.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx


More information about the Intel-gfx mailing list