From: Ehab Ababneh <ehab.ababneh@intel.com>
To: ilpo.jarvinen@linux.intel.com, hansg@kernel.org, tony.luck@intel.com
Cc: ashok.raj.linux@gmail.com, platform-driver-x86@vger.kernel.org,
linux-kernel@vger.kernel.org, tglx@kernel.org, mingo@redhat.com,
bp@alien8.de, dave.hansen@linux.intel.com, x86@kernel.org,
hpa@zytor.com, rostedt@goodmis.org, mhiramat@kernel.org,
mathieu.desnoyers@efficios.com,
linux-trace-kernel@vger.kernel.org, peterz@infradead.org,
sathyanarayanan.kuppuswamy@linux.intel.com, pengfei.xu@intel.com,
ravi.v.shankar@intel.com
Subject: [PATCH 00/17] platform/x86/intel/ifs: Support group-based In-Field Scan and Diamond Rapids
Date: Mon, 26 Oct 2026 11:38:03 -0700 [thread overview]
Message-ID: <20261026183820.129459-1-ehab.ababneh@intel.com> (raw)
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
next reply other threads:[~2026-10-26 18:35 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-26 18:38 Ehab Ababneh [this message]
2026-10-26 18:38 ` [PATCH 01/17] platform/x86/intel/ifs: Revert SBAF feature Ehab Ababneh
2026-10-26 18:38 ` [PATCH 02/17] platform/x86/intel/ifs: Fix IFS sysfs interfaces in the comments Ehab Ababneh
2026-10-26 18:38 ` [PATCH 03/17] platform/x86/intel/ifs: Clarify In-Field Scan MSR bit macro definitions Ehab Ababneh
2026-10-26 18:38 ` [PATCH 04/17] platform/x86/intel/ifs: Use u64 to define the bits of IFS MSR registers Ehab Ababneh
2026-10-26 18:38 ` [PATCH 05/17] platform/x86/intel/ifs: Early return on the IFS image hash/chunk error Ehab Ababneh
2026-10-26 18:38 ` [PATCH 06/17] platform/x86/intel/ifs: Refactor Array BIST function interface Ehab Ababneh
2026-10-26 18:38 ` [PATCH 07/17] stop_machine: Add stop_cluster_cpuslocked() for per-cluster operations Ehab Ababneh
2026-10-26 18:38 ` [PATCH 08/17] platform/x86/intel/ifs: Enumerate the bit to determine if CPUs need to join for IFS tests Ehab Ababneh
2026-10-26 18:38 ` [PATCH 09/17] platform/x86/intel/ifs: Select CPUs from topology_cluster_cpumask() for simultaneous " Ehab Ababneh
2026-10-26 18:38 ` [PATCH 10/17] platform/x86/intel/ifs: Validate TID_BIT_SHIFT matches topology_cluster_cpumask() Ehab Ababneh
2026-10-26 18:38 ` [PATCH 11/17] platform/x86/intel/ifs: Track and report scan results on a per-CPU basis Ehab Ababneh
2026-10-26 18:38 ` [PATCH 12/17] platform/x86/intel/ifs: Add new sysfs interfaces {status, details, cpus_ran}_list Ehab Ababneh
2026-10-26 18:38 ` [PATCH 13/17] platform/x86/intel/ifs: Add additional details sysfs interface Ehab Ababneh
2026-10-26 18:38 ` [PATCH 14/17] platform/x86/intel/ifs: Add Intel Diamond Rapids server support Ehab Ababneh
2026-10-26 18:38 ` [PATCH 15/17] platform/x86/intel/ifs: Remove limit on number of batch files Ehab Ababneh
2026-10-26 18:38 ` [PATCH 16/17] platform/x86/intel/ifs: Update IFS documentation Ehab Ababneh
2026-10-26 18:38 ` [PATCH 17/17] Documentation/ABI: Update IFS ABI doc Ehab Ababneh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261026183820.129459-1-ehab.ababneh@intel.com \
--to=ehab.ababneh@intel.com \
--cc=ashok.raj.linux@gmail.com \
--cc=bp@alien8.de \
--cc=dave.hansen@linux.intel.com \
--cc=hansg@kernel.org \
--cc=hpa@zytor.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-trace-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@efficios.com \
--cc=mhiramat@kernel.org \
--cc=mingo@redhat.com \
--cc=pengfei.xu@intel.com \
--cc=peterz@infradead.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=ravi.v.shankar@intel.com \
--cc=rostedt@goodmis.org \
--cc=sathyanarayanan.kuppuswamy@linux.intel.com \
--cc=tglx@kernel.org \
--cc=tony.luck@intel.com \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®