drm/xe/xe2lpm: Fixup Wa_14020756599

This WA needs to be applied to graphics GT when the media version
is 2000. Currently, media version 2000 is always paired with
graphics version 2004 which will result in writing same register
with same bits twice. We can't add optional rule in rtp
framework and also writing same register with same bits gives
warning.

Currently, media version 2000 is always paired with graphics version
2004, so just checking the latter is sufficient.

V2(Lucas):
  - Add more detail in commit message
  - Improve code comment to follow guideline

Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/2016
Fixes: 131328aa56 ("drm/xe/xe2lpm: Add permanent Wa_14020756599")
Signed-off-by: Tejas Upadhyay <tejas.upadhyay@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240607122528.1048610-1-tejas.upadhyay@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
This commit is contained in:
Tejas Upadhyay 2024-06-07 17:55:28 +05:30 committed by Lucas De Marchi
parent 4ca1a12a1b
commit 24d0d98af1
3 changed files with 7 additions and 26 deletions

View File

@ -324,8 +324,3 @@ bool xe_rtp_match_first_gslice_fused_off(const struct xe_gt *gt,
return dss >= dss_per_gslice;
}
bool xe_rtp_match_when_media2000(const struct xe_gt *gt,
const struct xe_hw_engine *hwe)
{
return (gt_to_xe(gt))->info.media_verx100 == 2000;
}

View File

@ -427,18 +427,4 @@ bool xe_rtp_match_first_render_or_compute(const struct xe_gt *gt,
bool xe_rtp_match_first_gslice_fused_off(const struct xe_gt *gt,
const struct xe_hw_engine *hwe);
/*
* xe_rtp_match_when_media2000 - Match when media GT version 2000
*
* @gt: GT structure
* @hwe: Engine instance
*
* Its one of the case where we need to apply workaround on primary GT
* based on if media GT version 2000 is present. Thus this API will help
* us to match media version 2000.
*
* Returns: true if media GT version 2000, false otherwise.
*/
bool xe_rtp_match_when_media2000(const struct xe_gt *gt,
const struct xe_hw_engine *hwe);
#endif

View File

@ -677,6 +677,13 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
ENGINE_CLASS(RENDER)),
XE_RTP_ACTIONS(SET(CHICKEN_RASTER_2, TBIMR_FAST_CLIP))
},
/*
* This WA is also needed on primary GT when the media version is 2000.
* Currently, media version 2000 is always paired with graphics version
* 2004, so just checking the latter is sufficient. In the future, media
* version 2000 can be used with some other graphics version where WA
* still needs to be implemented
*/
{ XE_RTP_NAME("14020756599"),
XE_RTP_RULES(GRAPHICS_VERSION(2004), ENGINE_CLASS(RENDER)),
XE_RTP_ACTIONS(SET(WM_CHICKEN3, HIZ_PLANE_COMPRESSION_DIS))
@ -705,13 +712,6 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
DIS_AUTOSTRIP))
},
/* Xe2_LPM */
{ XE_RTP_NAME("14020756599"),
XE_RTP_RULES(ENGINE_CLASS(RENDER), FUNC(xe_rtp_match_when_media2000)),
XE_RTP_ACTIONS(SET(WM_CHICKEN3, HIZ_PLANE_COMPRESSION_DIS))
},
{}
};