Re: [PATCH v2 5/6] drm/xe/xe_ttm_vram: Report max_pages reported by firmware in debugfs

["Ghimiray, Himal Prasad" <himal.prasad.ghimiray@intel.com>]

On 07-09-2026 15:17, Riana Tauro wrote:
> Report the maximum number of pages returned by firmware at probe,
> resolving the existing TODO.
> 
> Cc: Tejas Upadhyay <tejas.upadhyay@intel.com>
> Cc: Himal Prasad Ghimiray <himal.prasad.ghimiray@intel.com>
> Signed-off-by: Riana Tauro <riana.tauro@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_ttm_vram_mgr.c       | 6 +-----
>   drivers/gpu/drm/xe/xe_ttm_vram_mgr_types.h | 2 --
>   2 files changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_ttm_vram_mgr.c b/drivers/gpu/drm/xe/xe_ttm_vram_mgr.c
> index 2c4722a956a0..cc33ba7e23c2 100644
> --- a/drivers/gpu/drm/xe/xe_ttm_vram_mgr.c
> +++ b/drivers/gpu/drm/xe/xe_ttm_vram_mgr.c
> @@ -1012,11 +1012,7 @@ static int vram_bad_pages_show(struct seq_file *m, void *unused)
>   	struct xe_tile *tile;
>   	u8 id;
>   
> -	man = ttm_manager_type(&xe->ttm, XE_PL_VRAM0);
> -	if (man)
> -		/* TODO Hook with RAS to show max_pages fetched from FW */
> -		seq_printf(m, "max_pages: %d\n",
> -			   to_xe_ttm_vram_mgr(man)->max_pages);
> +	seq_printf(m, "max_pages: %u\n", xe_ras_get_max_pages(xe));

LGTM
Reviewed-by: Himal Prasad Ghimiray <himal.prasad.ghimiray@intel.com>

>   
>   	for_each_tile(tile, xe, id) {
>   		struct xe_vram_region *vr = tile->mem.vram;
> diff --git a/drivers/gpu/drm/xe/xe_ttm_vram_mgr_types.h b/drivers/gpu/drm/xe/xe_ttm_vram_mgr_types.h
> index efcf3e1d4e80..dc97b0ad0e51 100644
> --- a/drivers/gpu/drm/xe/xe_ttm_vram_mgr_types.h
> +++ b/drivers/gpu/drm/xe/xe_ttm_vram_mgr_types.h
> @@ -37,8 +37,6 @@ struct xe_ttm_vram_mgr {
>   	struct mutex lock;
>   	/** @mem_type: The TTM memory type */
>   	u32 mem_type;
> -	/** @max_pages: max pages that can be in offline queue retrieved from FW */
> -	u16 max_pages;
>   };
>   
>   /**