[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.00.1111211416000.5318@chino.kir.corp.google.com>
Date: Mon, 21 Nov 2011 14:24:18 -0800 (PST)
From: David Rientjes <rientjes@...gle.com>
To: Petr Holasek <pholasek@...hat.com>
cc: Andrew Morton <akpm@...ux-foundation.org>,
Thomas Gleixner <tglx@...utronix.de>,
Ingo Molnar <mingo@...hat.com>,
"H. Peter Anvin" <hpa@...or.com>, linux-kernel@...r.kernel.org,
x86@...nel.org, Anton Arapov <anton@...hat.com>
Subject: Re: NUMA emulation x86_64: numa=fake parameter for custom nodes
distance
On Mon, 21 Nov 2011, Petr Holasek wrote:
> This patch was designed as nothing more than helper for debugging/testing
> purposes, e.g. when it is useful to have more values in exports than only
> LOCAL_DISTANCEs. So that's the reason why it disregards former distances
> between physical nodes.
>
I understand, but like I said: the only debugging and testing it would be
useful for is node ordering. The actual latency of memory accesses are
not going to be representative of the new distances and will lead to
confusion since they're wrong. It's also pretty limited in even that
regard because all nodes are now spaced by the same distance so they're
just spread out linearly instead of actually representing a real NUMA
architecture.
> Faking the SLIT table is a really good point, if this patch would be
> eventually rejected, I will rework the patch in that manner.
>
That has applicability even outside of debugging, you could override your
own machine's slit if you know it's bogus. The way it's defined is very
lengthy, however, and would require (4 * nr_nodes^2) characters at maximum
since the max distance is three characters, 255 (unreachable node), and
you'd need to separate them by one character, a comma. That's 256 chars
for eight nodes!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists