Re: [PATCH 2/3] lib/xe_query: add xe_hwconfig_lookup_value() helper

["Grzegorzek, Dominik" <dominik.grzegorzek@intel.com>]

On Thu, 2024-11-28 at 13:05 +0000, Jan Maslak wrote:
> Add a new helper function, xe_hw_config_lookup_value(), that
> returns a pointer to the value of a given hwconfig attribute.
> 
> Signed-off-by: Jan Maslak <jan.maslak@intel.com>

LGTM,
Reviewed-by: Dominik Grzegorzek <dominik.grzegorzek@intel.com>
> ---
>  lib/xe/xe_query.c | 39 +++++++++++++++++++++++++++++++++++++++
>  lib/xe/xe_query.h |  2 ++
>  2 files changed, 41 insertions(+)
> 
> diff --git a/lib/xe/xe_query.c b/lib/xe/xe_query.c
> index 49bed033b..f3731d9d3 100644
> --- a/lib/xe/xe_query.c
> +++ b/lib/xe/xe_query.c
> @@ -839,6 +839,45 @@ uint16_t xe_gt_get_tile_id(int fd, int gt)
>  	return xe_dev->gt_list->gt_list[gt].tile_id;
>  }
>  
> +/**
> + * xe_hwconfig_lookup_value:
> + * @fd: xe device fd
> + * @attribute: hwconfig attribute id
> + * @len: pointer to store length of the value (in uint32_t sized elements)
> + *
> + * Returns a pointer to the value of the hwconfig attribute @attribute and
> + * writes the number of uint32_t elements indicating the length of the value to @len.
> + */
> +uint32_t *xe_hwconfig_lookup_value(int fd, enum intel_hwconfig attribute, uint32_t *len)
> +{
> +	struct xe_device *xe_dev;
> +	uint32_t *hwconfig;
> +	uint32_t pos, hwconfig_len;
> +
> +	xe_dev = find_in_cache(fd);
> +	igt_assert(xe_dev);
> +
> +	hwconfig = xe_dev->hwconfig;
> +	igt_assert(hwconfig);
> +
> +	/* Extract the value from the hwconfig */
> +	pos = 0;
> +	hwconfig_len = xe_dev->hwconfig_size / sizeof(uint32_t);
> +	while (pos + 2 < hwconfig_len) {
> +		uint32_t attribute_id = hwconfig[pos];
> +		uint32_t attribute_len = hwconfig[pos + 1];
> +		uint32_t *attribute_data = &hwconfig[pos + 2];
> +
> +		if (attribute_id == attribute) {
> +			*len = attribute_len;
> +			return attribute_data;
> +		}
> +		pos += 2 + attribute_len;
> +	}
> +
> +	return NULL;
> +}
> +
>  igt_constructor
>  {
>  	xe_device_cache_init();
> diff --git a/lib/xe/xe_query.h b/lib/xe/xe_query.h
> index f8836578e..30ea5ad41 100644
> --- a/lib/xe/xe_query.h
> +++ b/lib/xe/xe_query.h
> @@ -15,6 +15,7 @@
>  #include "igt_aux.h"
>  #include "igt_list.h"
>  #include "igt_sizes.h"
> +#include "intel_hwconfig_types.h"
>  
>  #define XE_DEFAULT_ALIGNMENT           SZ_4K
>  #define XE_DEFAULT_ALIGNMENT_64K       SZ_64K
> @@ -118,6 +119,7 @@ struct drm_xe_engine *xe_find_engine_by_class(int fd, uint16_t engine_class);
>  bool xe_has_media_gt(int fd);
>  bool xe_is_media_gt(int fd, int gt);
>  uint16_t xe_gt_get_tile_id(int fd, int gt);
> +uint32_t *xe_hwconfig_lookup_value(int fd, enum intel_hwconfig attribute, uint32_t *len);
>  
>  struct xe_device *xe_device_get(int fd);
>  void xe_device_put(int fd);