Re: [Intel-xe] [PATCH 01/15] drm/xe/xe2: Update render/compute context image sizes

[Matt Atwood <matthew.s.atwood@intel.com>]

On Fri, Aug 11, 2023 at 09:06:04AM -0700, Lucas De Marchi wrote:
> From: Matt Roper <matthew.d.roper@intel.com>
> 
> The render and compute context are significantly smaller on Xe2 than on
> previous platforms.
> 
> Registers:
>  - Render:  3008 dwords -> 12032 bytes -> round to 3 pages
>  - Compute: 1424 dwords ->  5696 bytes -> round to 2 pages
> 
> We also allocate one additional page for the HWSP, so the total
> allocation sizes for render and compute are 4 and 3 pages respectively.
> 
Bspec: 65182, 56578, 55793
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Matt Atwood <matthew.s.atwood@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_lrc.c | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
> index 09db8da261a39..a6479deb62376 100644
> --- a/drivers/gpu/drm/xe/xe_lrc.c
> +++ b/drivers/gpu/drm/xe/xe_lrc.c
> @@ -37,9 +37,16 @@ size_t xe_lrc_size(struct xe_device *xe, enum xe_engine_class class)
>  {
>  	switch (class) {
>  	case XE_ENGINE_CLASS_RENDER:
> +		if (GRAPHICS_VER(xe) >= 20)
> +			return 4 * SZ_4K;
> +		else
> +			return 14 * SZ_4K;
>  	case XE_ENGINE_CLASS_COMPUTE:
>  		/* 14 pages since graphics_ver == 11 */
> -		return 14 * SZ_4K;
> +		if (GRAPHICS_VER(xe) >= 20)
> +			return 3 * SZ_4K;
> +		else
> +			return 14 * SZ_4K;
>  	default:
>  		WARN(1, "Unknown engine class: %d", class);
>  		fallthrough;
> -- 
> 2.40.1
>