[Intel-xe] [PATCH v3 1/9] drm/xe: Fix Wa_22011802037 annotation

[Matt Roper]

On Tue, May 16, 2023 at 03:19:42PM -0700, Lucas De Marchi wrote:
> It was missing one digit, so not showing up as a proper WA number. Add
> the missing number and annotate it with a FIXME as there are more to be
> implemented to consider this WA done: ensure CS is stop before doing a
> reset, wait for pending.

Should we open a gitlab issue about this workaround being incomplete so
it doesn't slip through the cracks?  I'm not sure anyone will remember
to come back and address the FIXME otherwise.

Reviewed-by: Matt Roper <matthew.d.roper at intel.com>

> 
> Also, this WA applies to platforms up to graphics version 1270 (with the
> exception of MTL A*, that are not supported in xe). Fix platform check.
> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
> ---
>  drivers/gpu/drm/xe/xe_guc.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_guc.c b/drivers/gpu/drm/xe/xe_guc.c
> index eb4af4c71124..1b5d0b6ad180 100644
> --- a/drivers/gpu/drm/xe/xe_guc.c
> +++ b/drivers/gpu/drm/xe/xe_guc.c
> @@ -162,9 +162,11 @@ static u32 guc_ctl_wa_flags(struct xe_guc *guc)
>  		flags |= GUC_WA_DUAL_QUEUE;
>  
>  	/*
> -	 * Wa_2201180203
> +	 * Wa_22011802037: FIXME - there's more to be done than simply setting
> +	 * this flag: make sure each CS is stopped when preparing for GT reset
> +	 * and wait for pending MI_FW.
>  	 */
> -	if (GRAPHICS_VER(xe) <= 12)
> +	if (GRAPHICS_VERx100(xe) < 1270)
>  		flags |= GUC_WA_PRE_PARSER;
>  
>  	/* Wa_16011777198 */
> -- 
> 2.40.1
> 

-- 
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation