[PATCH] tests/intel/*_query: Add new hwconfig table entry

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu Sep 12 08:57:41 UTC 2024


Hi Julia,
On 2024-09-09 at 18:12:38 -0700, Julia Filipchuk wrote:
> Two new entries added to the hardware config table. Updated enum and
> tests accordingly.
> 
> Signed-off-by: Julia Filipchuk <julia.filipchuk at intel.com>

Adding John and Ashutosh to Cc.
Cc: John Harrison <John.C.Harrison at Intel.com>
Cc: Ashutosh Dixit <ashutosh.dixit at intel.com>

Regards,
Kamil

> ---
>  lib/intel_hwconfig_types.h | 2 ++
>  tests/intel/i915_query.c   | 2 ++
>  tests/intel/xe_query.c     | 4 +++-
>  3 files changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/intel_hwconfig_types.h b/lib/intel_hwconfig_types.h
> index 0e6ce5b17..b5d9f1c9e 100644
> --- a/lib/intel_hwconfig_types.h
> +++ b/lib/intel_hwconfig_types.h
> @@ -97,6 +97,8 @@ enum intel_hwconfig {
>  	INTEL_HWCONFIG_MIN_MESH_URB_ENTRIES,			/* 79 */
>  	INTEL_HWCONFIG_MAX_MESH_URB_ENTRIES,			/* 80 */
>  	INTEL_HWCONFIG_MAX_GSC,					/* 81 */
> +	INTEL_HWCONFIG_SYNC_NUM_RT_STACKS_PER_DSS,		/* 82 */
> +	INTEL_HWCONFIG_NUM_XECU,				/* 83 */
>  	__INTEL_HWCONFIG_KEY_LIMIT
>  };
>  
> diff --git a/tests/intel/i915_query.c b/tests/intel/i915_query.c
> index 54dbb193a..df7e46a63 100644
> --- a/tests/intel/i915_query.c
> +++ b/tests/intel/i915_query.c
> @@ -1340,6 +1340,8 @@ static const char * const hwconfig_keys[] = {
>  	[INTEL_HWCONFIG_MIN_MESH_URB_ENTRIES] = "Min Mesh URB Entries",
>  	[INTEL_HWCONFIG_MAX_MESH_URB_ENTRIES] = "Max Mesh URB Entries",
>  	[INTEL_HWCONFIG_MAX_GSC] = "MaxGSC",
> +	[INTEL_HWCONFIG_SYNC_NUM_RT_STACKS_PER_DSS] = "Sync Num RT Stacks Per DSS",
> +	[INTEL_HWCONFIG_NUM_XECU] = "Num of XeCU",
>  };
>  
>  static const char * const hwconfig_memtypes[] = {
> diff --git a/tests/intel/xe_query.c b/tests/intel/xe_query.c
> index ddb2e76e5..3a0a83135 100644
> --- a/tests/intel/xe_query.c
> +++ b/tests/intel/xe_query.c
> @@ -50,7 +50,7 @@ void dump_hex_debug(void *buffer, int len)
>  /* Please reflect intel_hwconfig_types.h changes below
>   * static_asserti_value + get_hwconfig_name
>   *   Thanks :-) */
> -static_assert(INTEL_HWCONFIG_MAX_GSC + 1 == __INTEL_HWCONFIG_KEY_LIMIT, "");
> +static_assert(INTEL_HWCONFIG_NUM_XECU + 1 == __INTEL_HWCONFIG_KEY_LIMIT, "");
>  
>  #define CASE_STRINGIFY(A) case INTEL_HWCONFIG_##A: return #A;
>  const char* get_hwconfig_name(int param)
> @@ -137,6 +137,8 @@ const char* get_hwconfig_name(int param)
>  	CASE_STRINGIFY(MIN_MESH_URB_ENTRIES);
>  	CASE_STRINGIFY(MAX_MESH_URB_ENTRIES);
>  	CASE_STRINGIFY(MAX_GSC);
> +	CASE_STRINGIFY(SYNC_NUM_RT_STACKS_PER_DSS);
> +	CASE_STRINGIFY(NUM_XECU);
>  	}
>  	igt_assert_lt(param, __INTEL_HWCONFIG_KEY_LIMIT);
>  	igt_assert(!"Missing config table enum");
> -- 
> 2.45.2
> 


More information about the igt-dev mailing list