Re: [PATCH i-g-t v3 00/11] Implement Wa_14026539277

[Gustavo Sousa <gustavo.sousa@intel.com>]

Gustavo Sousa <gustavo.sousa@intel.com> writes:

> Wa_14026539277 requires handling from both KMD and userspace (which
> includes IGT).  Do the required changes in IGT for Wa_14026539277.
>
> Since we need to check for Wa_14026539277 as early as during IGT's
> xe_device initialization (i.e. xe_device_get()), some preparatory
> steps are required before the changes specific to the workaround.
>
> So this series does that preliminary work with the initial set of
> patches and then adds the workaround implementation as the final
> patch.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
> Changes in v3:
> - Add tweaks related to igt_warn_on() to avoid CI noise like
>   https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_15892/shard-bmg-4/igt@core_setmaster@master-drop-set-user.html .
> - Link to v2: https://patch.msgid.link/20260923-wa_14026539277-v2-0-528b575299d2@intel.com
>
> Changes in v2:
> - The approach remains basically the same, but there were some changes
>   on how the debugfs dumps are cached: now a new struct called
>   xe_wa_cache is used.
> - Besides that, feedback from v1 was incorportated. See the changelogs
>   in individual patches for details.
> - Link to v1: https://patch.msgid.link/20260721-wa_14026539277-v1-0-0c147eeefefa@intel.com
>
> ---
> Gustavo Sousa (11):
>       lib/xe: Move lib/intel_wa to lib/xe/xe_wa
>       lib/xe: Make debugfs_file_has_wa() aware of the debugfs file layout
>       lib/xe: Gather workarounds debugfs dumps in a cache struct
>       lib/xe: Cache workaround information in xe_device
>       lib/xe: Return boolean from xe_wa()
>       lib/xe: Assert that cache is available for xe_wa()
>       tests/intel/xe_pat: Adapt pat_entry_is_wb() to Xe3p
>       lib/xe: Add xe_wa_from_cache()
>       lib/intel_pat: Encapsulate management of xe_device's pat_cache
>       lib/intel_pat: Pass xe_device to xe_get_pat_config()
>       intel: Implement Wa_14026539277
>
>  lib/intel_pat.c       |  43 +++++++--
>  lib/intel_pat.h       |   5 ++
>  lib/intel_wa.c        |  66 --------------
>  lib/intel_wa.h        |  11 ---
>  lib/meson.build       |   2 +-
>  lib/tests/meson.build |   1 +
>  lib/tests/xe_wa.c     |  42 +++++++++
>  lib/tests/xe_wa.h     |  12 +++
>  lib/xe/xe_query.c     |  15 ++--
>  lib/xe/xe_query.h     |  10 ++-
>  lib/xe/xe_wa.c        | 240 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  lib/xe/xe_wa.h        |  19 ++++
>  tests/intel/xe_oa.c   |   4 +-
>  tests/intel/xe_pat.c  |  35 ++++++--
>  14 files changed, 403 insertions(+), 102 deletions(-)
> ---
> base-commit: e40d80d42f9b8f1d22475b7f609252a34cd7355b
> change-id: 20260714-wa_14026539277-c79e97449287

Pushed.  Thanks Matt Atwood and Matt Roper for the reviews!

--
Gustavo Sousa