Re: [Intel-xe] [PATCH 2/2] drm/xe: Improve vram info debug printing

[Matthew Brost <matthew.brost@intel.com>]

On Fri, Aug 11, 2023 at 03:39:23PM -0400, Oak Zeng wrote:
> Print both device physical address range and CPU io range
> of vram. Also print vram's actual size, usable size excluding
> stolen memory, and CPU io accessible size.
> V1:
>   - Add back small BAR device info (Matt)
> 
> Signed-off-by: Oak Zeng <oak.zeng@intel.com>
> Reviewed-by: Michael J. Ruhl <michael.j.ruhl@intel.com>

Reviewed-by: Matthew Brost <matthew.brost@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_mmio.c | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_mmio.c b/drivers/gpu/drm/xe/xe_mmio.c
> index 41ee89247ddb..bb6823db14d4 100644
> --- a/drivers/gpu/drm/xe/xe_mmio.c
> +++ b/drivers/gpu/drm/xe/xe_mmio.c
> @@ -286,12 +286,13 @@ int xe_mmio_probe_vram(struct xe_device *xe)
>  		tile->mem.vram.usable_size = vram_size;
>  		tile->mem.vram.mapping = xe->mem.vram.mapping + tile_offset;
>  
> -		drm_info(&xe->drm, "VRAM[%u, %u]: %pa, %pa\n", id, tile->id,
> -			 &tile->mem.vram.io_start, &tile->mem.vram.usable_size);
> -
>  		if (tile->mem.vram.io_size < tile->mem.vram.usable_size)
> -			drm_info(&xe->drm, "VRAM[%u, %u]: CPU access limited to %pa\n", id,
> -				 tile->id, &tile->mem.vram.io_size);
> +			drm_info(&xe->drm, "Small BAR device\n");
> +		drm_info(&xe->drm, "VRAM[%u, %u]: Actual physical size %pa, usable size exclude stolen %pa, CPU accessible size %pa\n", id,
> +			 tile->id, &tile->mem.vram.actual_physical_size, &tile->mem.vram.usable_size, &tile->mem.vram.io_size);
> +		drm_info(&xe->drm, "VRAM[%u, %u]: DPA range: [%pa-%llx], io range: [%pa-%llx]\n", id, tile->id,
> +			 &tile->mem.vram.dpa_base, tile->mem.vram.dpa_base + tile->mem.vram.actual_physical_size,
> +			 &tile->mem.vram.io_start, tile->mem.vram.io_start + tile->mem.vram.io_size);
>  
>  		/* calculate total size using tile size to get the correct HW sizing */
>  		total_size += tile_size;
> -- 
> 2.26.3
>