]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
[PATCH] x86-64: x86_64-make-the-numa-hash-function-nodemap-allocation fix fix
authorAmul Shah <amul.shah@unisys.com>
Tue, 13 Feb 2007 12:26:20 +0000 (13:26 +0100)
committerAndi Kleen <andi@basil.nowhere.org>
Tue, 13 Feb 2007 12:26:20 +0000 (13:26 +0100)
- Removed an extraneous debug message from allocate_cachealigned_map

- Changed extract_lsb_from_nodes to return 63 for the case where there was
  only one memory node.  The prevents the creation of the dynamic hashmap.

- Changed extract_lsb_from_nodes to use only the starting memory address of
  a node.  On an ES7000, our nodes overlap the starting and ending address,
  meaning, that we see nodes like

00000 - 10000
10000 - 20000

  But other systems have nodes whose start and end addresses do not overlap.
   For example:

00000 - 0FFFF
10000 - 1FFFF

  In this case, using the ending address will result in an LSB much lower
  than what is possible.  In this case an LSB of 1 when in reality it should
  be 16.

Cc: Andi Kleen <ak@suse.de>
Cc: Rohit Seth <rohitseth@google.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Andi Kleen <ak@suse.de>
arch/x86_64/mm/numa.c

index 7d9c428f409438ed4d2d80c15e4393bcb7027586..1ec16ea9751993b08028c2556f14fc65f7edd039 100644 (file)
@@ -78,11 +78,8 @@ static int __init allocate_cachealigned_memnodemap(void)
        unsigned long pad, pad_addr;
 
        memnodemap = memnode.embedded_map;
-       if (memnodemapsize <= 48) {
-               printk(KERN_DEBUG "NUMA: Allocated memnodemap from %lx - %lx\n",
-                      nodemap_addr, nodemap_addr + nodemap_size);
+       if (memnodemapsize <= 48)
                return 0;
-       }
 
        pad = L1_CACHE_BYTES - 1;
        pad_addr = 0x8000;
@@ -110,7 +107,7 @@ static int __init allocate_cachealigned_memnodemap(void)
 static int __init
 extract_lsb_from_nodes (const struct bootnode *nodes, int numnodes)
 {
-       int i;
+       int i, nodes_used = 0;
        unsigned long start, end;
        unsigned long bitfield = 0, memtop = 0;
 
@@ -119,11 +116,15 @@ extract_lsb_from_nodes (const struct bootnode *nodes, int numnodes)
                end = nodes[i].end;
                if (start >= end)
                        continue;
-               bitfield |= start | end;
+               bitfield |= start;
+               nodes_used++;
                if (end > memtop)
                        memtop = end;
        }
-       i = find_first_bit(&bitfield, sizeof(unsigned long)*8);
+       if (nodes_used <= 1)
+               i = 63;
+       else
+               i = find_first_bit(&bitfield, sizeof(unsigned long)*8);
        memnodemapsize = (memtop >> i)+1;
        return i;
 }