All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tim Chen <tim.c.chen@linux.intel.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: "Chen, Yu C" <yu.c.chen@intel.com>,
	Kyle Meyer <kyle.meyer@hpe.com>,
	 bp@alien8.de, dave.hansen@linux.intel.com, mingo@redhat.com,
	tglx@kernel.org, 	vinicius.gomes@intel.com, brgerst@gmail.com,
	hpa@zytor.com, kprateek.nayak@amd.com,
		linux-kernel@vger.kernel.org, patryk.wlazlyn@linux.intel.com,
		rafael.j.wysocki@intel.com, russ.anderson@hpe.com,
	x86@kernel.org, 	zhao1.liu@intel.com
Subject: Re: [PATCH v2] sched/topology: Check average distances to remote packages
Date: Wed, 25 Feb 2026 14:55:58 -0800	[thread overview]
Message-ID: <caf4e9a532c9622d8b059394bb65050577aba38e.camel@linux.intel.com> (raw)
In-Reply-To: <20260225223024.GA606826@noisy.programming.kicks-ass.net>

On Wed, 2026-02-25 at 23:30 +0100, Peter Zijlstra wrote:
> On Wed, Feb 25, 2026 at 01:37:11PM -0800, Tim Chen wrote:
> > On Wed, 2026-02-25 at 17:32 +0100, Peter Zijlstra wrote:
> > > On Wed, Feb 25, 2026 at 04:44:09PM +0100, Peter Zijlstra wrote:
> > > 
> > > > Yes, so this assumes that all u sized clusters on the trace are similar
> > > > and 'sane' without verification.
> > > 
> > > That gave me an idea; how's this then?
> > 
> > Sorry I was sick for a few days.  Just catching up on this
> > thread here. I think your patch takes care of both GNR SNC-3 
> > with 3 compute dies (with non-symmetric remote
> > distances) and generic SNC-2 with 2 dies (symmetric
> > distances) very well.
> > 
> > Minor suggestion below for the patch.
> > 
> > Will ask the original GNR teams with the problem to try
> > it out.
> 
> Since HPE can obviously have a sane SLIT table; why can't we simply
> claim the SLIT table they had is broken and needs fixing?

From what I can see HPE seems to use SNC-2 variant of GNR so the SLIT
is symmetric.

Unfortunately in the topology for the 2 socket GNR that has 3 dies, there
are truly unsymmetric paths from between die A to die B between remote
sockets from what I'm told. 

> 
> Also, is there really no enumeration of the SNC mode available; must we
> really divinate?

Let me dig into that a bit. I was also thinking with that information,
it will make the code a lot simpler.

Tim

  parent reply	other threads:[~2026-02-25 22:56 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-05  0:24 [PATCH v2] sched/topology: Check average distances to remote packages Kyle Meyer
2026-02-23 16:42 ` Kyle Meyer
2026-02-23 17:03 ` Peter Zijlstra
2026-02-25  1:43   ` Kyle Meyer
2026-02-25  9:05     ` Chen, Yu C
2026-02-25 12:30     ` Peter Zijlstra
2026-02-25 13:36       ` Peter Zijlstra
2026-02-25 15:39       ` Chen, Yu C
2026-02-25 15:44         ` Peter Zijlstra
2026-02-25 16:32           ` Peter Zijlstra
2026-02-25 16:40             ` Peter Zijlstra
2026-02-25 21:37             ` Tim Chen
2026-02-25 22:30               ` Peter Zijlstra
2026-02-25 22:54                 ` Peter Zijlstra
2026-02-25 22:55                 ` Tim Chen [this message]
2026-02-25 23:29                   ` Kyle Meyer
2026-02-26 18:14                     ` Tim Chen
2026-02-25 16:41       ` Kyle Meyer
2026-02-25 16:49         ` Peter Zijlstra

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=caf4e9a532c9622d8b059394bb65050577aba38e.camel@linux.intel.com \
    --to=tim.c.chen@linux.intel.com \
    --cc=bp@alien8.de \
    --cc=brgerst@gmail.com \
    --cc=dave.hansen@linux.intel.com \
    --cc=hpa@zytor.com \
    --cc=kprateek.nayak@amd.com \
    --cc=kyle.meyer@hpe.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=patryk.wlazlyn@linux.intel.com \
    --cc=peterz@infradead.org \
    --cc=rafael.j.wysocki@intel.com \
    --cc=russ.anderson@hpe.com \
    --cc=tglx@kernel.org \
    --cc=vinicius.gomes@intel.com \
    --cc=x86@kernel.org \
    --cc=yu.c.chen@intel.com \
    --cc=zhao1.liu@intel.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.