Some Intel Xeon processors, starting with Diamond Rapids (DMR), share
one SCAN test engine between multiple CPU cores in a module (Dual-Core
Module, DCM), instead of only between the threads of a single core. All
logical processors that share an engine must start the In-Field Scan
(IFS) test at the same time. The current driver assumes the sharing
domain is cpu_smt_mask(), which does not hold on these systems. Also,
although the cores in a module are tested together, each core gets its
own result: on a DCM, one core can pass while the other fails.
This series:
- Patch 1 removes the SBAF test, which is not planned for any CPU and
whose test images were never published.
- Patches 2-6 are cleanups and preparatory refactoring (comment fixes,
clearer MSR bit names, u64 MSR bitfields, early return on image
errors, a generic Array BIST parameter struct). Patches 2-4 have no
functional change.
- Patch 7 adds stop_cluster_cpuslocked(), the cluster-level counterpart
of stop_core_cpuslocked(), so that all threads of a cluster can
rendezvous and issue WRMSR at the same time.
- Patches 8-10 enumerate ALL_LP_JOIN_NEEDED, select the CPUs to test
together from topology_cluster_cpumask(), and check at driver init
that TID_BIT_SHIFT matches that mask.
- Patches 11-13 record per-CPU results for group tests and expose them
through new sysfs files (status_list, details_list, cpus_ran_list,
addnl_details_list). The existing status and details files remain,
but now report the result of the CPU written to run_test rather than
the first CPU in the group.
- Patch 14 adds Diamond Rapids to the CPU match table.
- Patch 15 removes the 256 limit on the batch file number.
- Patches 16-17 update the IFS documentation and ABI document.
Most of this series was developed by Jithu Joseph and Qiuxu Zhuo, who
have both since left Intel. I have taken over upstreaming it, so please
send questions and review comments to me. The patches keep their
original authorship and Signed-off-by tags. Where Qiuxu changed a patch
written by someone else, the change is noted in brackets in the commit
message, and I have added my Signed-off-by to every patch.
Jithu Joseph (7):
platform/x86/intel/ifs: Refactor Array BIST function interface
platform/x86/intel/ifs: Enumerate the bit to determine if CPUs need to
join for IFS tests
platform/x86/intel/ifs: Select CPUs from topology_cluster_cpumask()
for simultaneous IFS tests
platform/x86/intel/ifs: Track and report scan results on a per-CPU
basis
platform/x86/intel/ifs: Add new sysfs interfaces {status, details,
cpus_ran}_list
platform/x86/intel/ifs: Add additional details sysfs interface
platform/x86/intel/ifs: Add Intel Diamond Rapids server support
Qiuxu Zhuo (9):
platform/x86/intel/ifs: Revert SBAF feature
platform/x86/intel/ifs: Fix IFS sysfs interfaces in the comments
platform/x86/intel/ifs: Clarify In-Field Scan MSR bit macro
definitions
platform/x86/intel/ifs: Use u64 to define the bits of IFS MSR
registers
platform/x86/intel/ifs: Early return on the IFS image hash/chunk error
stop_machine: Add stop_cluster_cpuslocked() for per-cluster operations
platform/x86/intel/ifs: Validate TID_BIT_SHIFT matches
topology_cluster_cpumask()
platform/x86/intel/ifs: Update IFS documentation
Documentation/ABI: Update IFS ABI doc
Tony Luck (1):
platform/x86/intel/ifs: Remove limit on number of batch files
.../ABI/testing/sysfs-platform-intel-ifs | 28 +-
arch/x86/include/asm/msr-index.h | 17 +-
drivers/platform/x86/intel/ifs/core.c | 97 +++--
drivers/platform/x86/intel/ifs/ifs.h | 223 +++++-----
drivers/platform/x86/intel/ifs/load.c | 52 +--
drivers/platform/x86/intel/ifs/runtest.c | 394 ++++++------------
drivers/platform/x86/intel/ifs/sysfs.c | 104 ++++-
include/linux/stop_machine.h | 15 +
include/trace/events/intel_ifs.h | 27 --
kernel/stop_machine.c | 35 +-
tools/arch/x86/include/asm/msr-index.h | 14 +-
11 files changed, 487 insertions(+), 519 deletions(-)
base-commit: 98f039265e580291d5e76a11b2072541fbb1c393
--
2.43.0
