diff options
author | Minoru Usui <usui@mxm.nes.nec.co.jp> | 2008-01-30 07:33:35 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-01-30 07:33:35 -0500 |
commit | 9a1b62fe858ba6780a9aeb4ab5f7751038a6c15d (patch) | |
tree | 127603cfdbb2a200d8dd2b86ddff16ec3a0484b9 /arch/x86/mm/srat_64.c | |
parent | 53391fa20cab6df6b476a5a0ad6be653c9de0c46 (diff) |
x86: fix NUMA emulation on 64-bit
I found a small bug of NUMA emulation code for x86_64. (CONFIG_NUMA_EMU)
If machine is non-NUMA, find_node_by_addr() should return
NUMA_NO_NODE, but current implementation code returns existent maximum
NUMA node number + 1.
This is not existent NUMA node number.
However, this behaviour does not affect NUMA emulation fortunately, because
acpi_fake_nodes() that is caller of find_node_by_addr() gets pxm
(proximity domain) by node_to_pxm() from non-existent NUMA node number
that was returned by find_node_by_addr().
node_to_pxm() returns PXM_INVAL that means illegal or non-existent
NUMA node number.
Signed-off-by: Minoru Usui <usui@mxm.nes.nec.co.jp>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/mm/srat_64.c')
-rw-r--r-- | arch/x86/mm/srat_64.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/mm/srat_64.c b/arch/x86/mm/srat_64.c index 4aed38fa4a65..37308d6bb5d9 100644 --- a/arch/x86/mm/srat_64.c +++ b/arch/x86/mm/srat_64.c | |||
@@ -416,7 +416,7 @@ static int __init find_node_by_addr(unsigned long addr) | |||
416 | break; | 416 | break; |
417 | } | 417 | } |
418 | } | 418 | } |
419 | return i; | 419 | return ret; |
420 | } | 420 | } |
421 | 421 | ||
422 | /* | 422 | /* |