[PATCH v3 04/13] drm/xe/psmi: Add Wa_14020001231

[Lucas De Marchi]

On Wed, Aug 13, 2025 at 11:14:57PM +0530, Riana Tauro wrote:
>
>
>On 8/8/2025 10:59 PM, Lucas De Marchi wrote:
>>From: Badal Nilawar <badal.nilawar at intel.com>
>>
>>Enable Wa 14020001231 to block psmi interrupts during C6 entry exit
>>flow. It's only enabled if PSMI is enabled in runtime.
>>
>>Signed-off-by: Badal Nilawar <badal.nilawar at intel.com>
>>Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
>>---
>>v2:
>>  - Enable only when PSMI is enabled
>>---
>>  drivers/gpu/drm/xe/abi/guc_klvs_abi.h | 1 +
>>  drivers/gpu/drm/xe/xe_guc_ads.c       | 4 ++++
>>  drivers/gpu/drm/xe/xe_wa_oob.rules    | 4 ++++
>>  3 files changed, 9 insertions(+)
>>
>>diff --git a/drivers/gpu/drm/xe/abi/guc_klvs_abi.h b/drivers/gpu/drm/xe/abi/guc_klvs_abi.h
>>index 31dbfeee289e7..0e78351c6ef5a 100644
>>--- a/drivers/gpu/drm/xe/abi/guc_klvs_abi.h
>>+++ b/drivers/gpu/drm/xe/abi/guc_klvs_abi.h
>>@@ -390,6 +390,7 @@ enum  {
>>   */
>>  enum xe_guc_klv_ids {
>>  	GUC_WORKAROUND_KLV_BLOCK_INTERRUPTS_WHEN_MGSR_BLOCKED				= 0x9002,
>>+	GUC_WORKAROUND_KLV_DISABLE_PSMI_INTERRUPTS_AT_C6_ENTRY_RESTORE_AT_EXIT		= 0x9004,
>>  	GUC_WORKAROUND_KLV_ID_GAM_PFQ_SHADOW_TAIL_POLLING				= 0x9005,
>>  	GUC_WORKAROUND_KLV_ID_DISABLE_MTP_DURING_ASYNC_COMPUTE				= 0x9007,
>>  	GUC_WA_KLV_NP_RD_WRITE_TO_CLEAR_RCSM_AT_CGP_LATE_RESTORE			= 0x9008,
>>diff --git a/drivers/gpu/drm/xe/xe_guc_ads.c b/drivers/gpu/drm/xe/xe_guc_ads.c
>>index 2ceaa197cb2f0..c42fc78798ca0 100644
>>--- a/drivers/gpu/drm/xe/xe_guc_ads.c
>>+++ b/drivers/gpu/drm/xe/xe_guc_ads.c
>>@@ -359,6 +359,10 @@ static void guc_waklv_init(struct xe_guc_ads *ads)
>>  				 GUC_WA_KLV_RESTORE_UNSAVED_MEDIA_CONTROL_REG);
>>  	}
>>+	if (XE_WA(gt, 14020001231))
>
>
>This commit has changed XE_WA to XE_GT_WA. Both WA patches need this change
>4d5c98eb77fe ("drm/xe: rename XE_WA to XE_GT_WA")

yep, I was waiting for reviews and comments to settle/resolve before
spinning a new version.

Lucas De Marchi