Re: [PATCH v2 21/23] drm/xe/xe2_hpg: Drop invalid workaround Wa_15010599737

["Vivekanandan, Balasubramani" <balasubramani.vivekanandan@intel.com>]

On 20.02.2026 09:27, Matt Roper wrote:
> Wa_15010599737 was a workaround originally proposed (and ultimately
> rejected) for DG2-G10.  There's no record of it ever being relevant or
> even considered for any other platforms.
> 
> The specific bit this workaround was setting is documented as "This bit
> should be set to 1 for the DX9 API and 0 for all other APIs" which means
> that it should almost always be left at the default value of 0 on Linux.
> The register itself is directly accessible from userspace, so in the
> special cases where it might be relevant (e.g., Wine/Proton running
> Windows DX9 apps), the userspace drivers already have the ability to
> change the setting without involvement of the kernel.
> 
> Fixes: 7f3ee7d88058 ("drm/xe/xe2hpg: Add initial GT workarounds")
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>

LGTM.

Reviewed-by: Balasubramani Vivekanandan <balasubramani.vivekanandan@intel.com>

Regards,
Bala 

> ---
>  drivers/gpu/drm/xe/xe_wa.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_wa.c b/drivers/gpu/drm/xe/xe_wa.c
> index fa5dce4ead48bae424a49c62142683dbf1971376..105ccd8115f09993b7ef82e70cb87d74008f267f 100644
> --- a/drivers/gpu/drm/xe/xe_wa.c
> +++ b/drivers/gpu/drm/xe/xe_wa.c
> @@ -734,10 +734,7 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
>  	},
>  
>  	/* Xe2_HPG */
> -	{ XE_RTP_NAME("15010599737"),
> -	  XE_RTP_RULES(GRAPHICS_VERSION(2001), ENGINE_CLASS(RENDER)),
> -	  XE_RTP_ACTIONS(SET(CHICKEN_RASTER_1, DIS_SF_ROUND_NEAREST_EVEN))
> -	},
> +
>  	{ XE_RTP_NAME("14020756599"),
>  	  XE_RTP_RULES(GRAPHICS_VERSION(2001), ENGINE_CLASS(RENDER)),
>  	  XE_RTP_ACTIONS(SET(WM_CHICKEN3, HIZ_PLANE_COMPRESSION_DIS))
> 
> -- 
> 2.53.0
>