[Intel-xe] [PATCH v1 5/7] drm/xe/uapi: Remove unused QUERY_CONFIG_MEM_REGION_COUNT

Souza, Jose jose.souza at intel.com
Fri Nov 10 15:55:19 UTC 2023


On Fri, 2023-11-10 at 15:41 +0000, Francois Dugast wrote:
> As part of uAPI cleanup, remove this constant which is not used. Memory
> regions can be queried with DRM_XE_DEVICE_QUERY_MEM_USAGE.

Reviewed-by: José Roberto de Souza <jose.souza at intel.com>

> 
> Signed-off-by: Francois Dugast <francois.dugast at intel.com>
> ---
>  drivers/gpu/drm/xe/xe_query.c | 2 --
>  include/uapi/drm/xe_drm.h     | 4 ++--
>  2 files changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_query.c b/drivers/gpu/drm/xe/xe_query.c
> index 43bc6094855c..27f66de4c1cd 100644
> --- a/drivers/gpu/drm/xe/xe_query.c
> +++ b/drivers/gpu/drm/xe/xe_query.c
> @@ -334,8 +334,6 @@ static int query_config(struct xe_device *xe, struct drm_xe_device_query *query)
>  		xe->info.vram_flags & XE_VRAM_FLAGS_NEED64K ? SZ_64K : SZ_4K;
>  	config->info[XE_QUERY_CONFIG_VA_BITS] = xe->info.va_bits;
>  	config->info[XE_QUERY_CONFIG_GT_COUNT] = xe->info.gt_count;
> -	config->info[XE_QUERY_CONFIG_MEM_REGION_COUNT] =
> -		hweight_long(xe->info.mem_region_mask);
>  	config->info[XE_QUERY_CONFIG_MAX_EXEC_QUEUE_PRIORITY] =
>  		xe_exec_queue_device_get_max_priority(xe);
>  
> diff --git a/include/uapi/drm/xe_drm.h b/include/uapi/drm/xe_drm.h
> index 0f8c5afd3584..1ac9ae0591de 100644
> --- a/include/uapi/drm/xe_drm.h
> +++ b/include/uapi/drm/xe_drm.h
> @@ -311,6 +311,7 @@ struct drm_xe_query_mem_usage {
>   * If a query is made with a struct drm_xe_device_query where .query
>   * is equal to DRM_XE_DEVICE_QUERY_CONFIG, then the reply uses
>   * struct drm_xe_query_config in .data.
> + *
>   */
>  struct drm_xe_query_config {
>  	/** @num_params: number of parameters returned in info */
> @@ -325,8 +326,7 @@ struct drm_xe_query_config {
>  #define XE_QUERY_CONFIG_MIN_ALIGNMENT		2
>  #define XE_QUERY_CONFIG_VA_BITS			3
>  #define XE_QUERY_CONFIG_GT_COUNT		4
> -#define XE_QUERY_CONFIG_MEM_REGION_COUNT	5
> -#define XE_QUERY_CONFIG_MAX_EXEC_QUEUE_PRIORITY	6
> +#define XE_QUERY_CONFIG_MAX_EXEC_QUEUE_PRIORITY	5
>  	/** @info: array of elements containing the config info */
>  	__u64 info[];
>  };



More information about the Intel-xe mailing list