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

[Jordan Justen]

Where can I find the documentation for this?

-Jordan

On 2024-09-09 18:12:38, 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>
> ---
>  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
>