mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Joachim Deguara" <joachim.deguara@amd.com>
To: "Andi Kleen" <ak@suse.de>
Cc: "lkml List" <linux-kernel@vger.kernel.org>,
	gregkh@suse.de, lenb@kernel.org,
	"Christoph Lameter" <clameter@sgi.com>
Subject: Re: [PATCH 0/2] faking and fixing the NUMA SLIT
Date: Wed, 18 Jul 2007 11:57:28 +0200	[thread overview]
Message-ID: <200707181157.29052.joachim.deguara@amd.com> (raw)
In-Reply-To: <200707181142.20811.ak@suse.de>

On Wednesday 18 July 2007 11:42:20 Andi Kleen wrote:
> On Wednesday 18 July 2007 11:30:01 Joachim Deguara wrote:
> > The problem with NUMA distances in the SLIT is that they are often wrong,
> > oh wait they aren't there at all because the BIOS didn't create a SLIT
> > since Windows does not use it.  If Linux does not find a slit it just
> > says the distance to local=10 and remote=20 according to ACPI spec.  The
> > problem is when we have a 4P system (or larger), there is generally one
> > node where we have two hops and its distance should be >20.
> >
> > Following are patches to first fake the SLIT in the ACPI code and then
> > add ability to write the distances from sysfs.
>
> The main use for the SLIT information are the zone fallback lists in
> the VM. These are created at boot.  If you change the SLIT later these
> won't be regenerated.
>
> The scheduler also uses it for load balancing, but it is much less
> important there than in the VM.

I looked at how node_distance() was called from page_alloc.c and sched.c but I 
overlooked that those results are really only used at init time.  So the 
backing mechanism of patching the SLIT is right but sysfs is way to late for 
those uses and creating a boot option is as you say ugly.  It would be great 
if BIOS just did this for us and correctly but what do you really expect 
that ;)

I would be happy to add it as a boot option if there is any popular support 
for the idea.

-Joachim






  reply	other threads:[~2007-07-18 11:06 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-18  9:30 Joachim Deguara
2007-07-18  9:30 ` [PATCH 1/2] fake " Joachim Deguara
2007-07-18  9:31 ` [PATCH 2/2] make node distance writeable Joachim Deguara
2007-07-18  9:42 ` [PATCH 0/2] faking and fixing the NUMA SLIT Andi Kleen
2007-07-18  9:57   ` Joachim Deguara [this message]
2007-07-23 20:25   ` Christoph Lameter
2007-07-23 22:10     ` Len Brown

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=200707181157.29052.joachim.deguara@amd.com \
    --to=joachim.deguara@amd.com \
    --cc=ak@suse.de \
    --cc=clameter@sgi.com \
    --cc=gregkh@suse.de \
    --cc=lenb@kernel.org \
    --cc=linux-kernel@vger.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®