Skip to content

Fix NUMA sched domain build errors for GNR and CWF

Tim Chen lkml stable 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 3 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2sched: Create architecture specific sched domain distanceskernel/sched/topology.c+86 -220 replies
2/2sched/topology: Fix sched domain build error for GNR, CWF in SNC-3 modearch/x86/kernel/smpboot.c+70 -03 replies

Reviews trailers per patch

All 2 patches have a Reviewed-by or Acked-by.

PatchChen YuZhao Liu
1/2R-b
2/2R-b T-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 6.18 0/2] Fix NUMA sched domain build errors for GNR and CWF

While testing Granite Rapids (GNR) and Clearwater Forest (CWF) systems in SNC-3 mode, we encountered sched domain build errors in dmesg. The scheduler domain code did not expect asymmetric node distances from a local node to multiple nodes in a remote package. As a result, remote nodes ended up being grouped partially with local nodes with asymemtric groupings, and creating too many levels in the NUMA sched domain hierarchy.

To address this, we simplify remote node distances for the purpose of sched domain construction on GNR and CWF. Specifically, we replace the individual distances to nodes within the same remote package with their average distance. This resolves the domain build errors and reduces the number of NUMA sched domain levels.

The actual SLIT NUMA node distances are still preserved separately, in case they are needed when building sched domains. NUMA balancing continues to use the true distances when selecting a closer remote node for a task’s numa_group.

These patches have been merged upstream.

Thanks,
Tim

Tim Chen (2):
sched: Create architecture specific sched domain distances sched/topology: Fix sched domain build error for GNR, CWF in SNC-3
mode

 arch/x86/kernel/smpboot.c |  70 ++++++++++++++++++++++++
 kernel/sched/topology.c   | 108 ++++++++++++++++++++++++++++++--------
 2 files changed, 156 insertions(+), 22 deletions(-)
Signature · 1 line
2.32.0