mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [Patch:000/004]Unify pxm_to_node id ver.4.
@ 2006-03-30 11:49 Yasunori Goto
  2006-03-30 11:55 ` [Patch:001/004]Unify pxm_to_node id ver.4. (generic part) Yasunori Goto
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Yasunori Goto @ 2006-03-30 11:49 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Linux Kernel ML, Yasunori Goto

Hi.

This patch set is to unify mapping from pxm to node id ver.4.
I posted patches for config of NODES_SHIFT.
So, MAX_PXM_DOMAIN is just defined by MAX_NUMNODES.
(But, old pxm's spec was 8 bit. So, if MAX_NUMNODES is 256 or smaller,
 then I use 256 for it.)

This patches are for 2.6.16-git16.
I tested them on ia64(Tiger4) with node emulation.

Please apply.

------------------------
Change log from ver.3
  - The patch for configuration of NODES_SHIFT is divided to other post.
    MAX_PXM_DOMAIN is defined by MAX_NUMNODES.

Change log from ver.2
  - update for 2.6.16-git14.
  - definition of pxm was changed from u8 to int. Pxm can be over 256.
  - CONFIG_NR_NODES is defined to configure MAX_PXM_DOMAINS.
  - redundant call of pxm_bit_set() is removed at acpi_numa_arch_fixup()
    of ia64 like followings.
	if (pxm_bit_test(i)) {
		:
		pxm_bit_set(i);  <---------------------- !!!
		:
	}

Change log from ver.1 
  - Fix old map from HP and SGI's code by Bob Picco-san.
  - Remove MAX_PXM_DOMAINS from asm-ia64/acpi.h. It is already defined at
    include/acpi/acpi_numa.h.
  - Fix return code of setup_node() at arch/x86_64/mm/srat.c
  - Fix ACPI_NUMA config for i386 by Andy Witcroft-san.
  - Define dummy functions for i386's compile error.
  - Remove garbage nid_to_pxm_map from acpi20_parse_srat() 
    at arch/i386/kernel/srat.c

----------------------------------
Description.

This patch is to unify mapping from pxm to node id.
In current code, i386, x86-64, and ia64 have its mapping by each own code.
But PXM is defined by ACPI and node id is used generically. So,
I think there is no reason to define it on each arch's code.
This mapping should be written at drivers/acpi/numa.c as a common code.



-- 
Yasunori Goto 



^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Patch:001/004]Unify pxm_to_node id ver.4. (generic part)
  2006-03-30 11:49 [Patch:000/004]Unify pxm_to_node id ver.4 Yasunori Goto
@ 2006-03-30 11:55 ` Yasunori Goto
  2006-03-30 11:55 ` [Patch:002/004]Unify pxm_to_node id ver.4. (for ia64) Yasunori Goto
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Yasunori Goto @ 2006-03-30 11:55 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Linux Kernel ML

This is new generic code for pxm_to_node_map.


Signed-off-by: Yasunori Goto <y-goto@jp.fujitsu.com>

 drivers/acpi/numa.c      |   48 +++++++++++++++++++++++++++++++++++++++++++++++
 include/acpi/acpi_numa.h |   23 ++++++++++++++++++++++
 include/linux/acpi.h     |    1 
 3 files changed, 72 insertions(+)

Index: pxm_ver4/drivers/acpi/numa.c
===================================================================
--- pxm_ver4.orig/drivers/acpi/numa.c	2006-03-29 22:11:18.000000000 +0900
+++ pxm_ver4/drivers/acpi/numa.c	2006-03-29 22:12:41.000000000 +0900
@@ -36,12 +36,60 @@
 #define _COMPONENT	ACPI_NUMA
 ACPI_MODULE_NAME("numa")
 
+static nodemask_t nodes_found_map = NODE_MASK_NONE;
+#define PXM_INVAL	-1
+#define NID_INVAL	-1
+
+/* maps to convert between proximity domain and logical node ID */
+int __cpuinitdata pxm_to_node_map[MAX_PXM_DOMAINS]
+				= { [0 ... MAX_PXM_DOMAINS - 1] = NID_INVAL };
+int __cpuinitdata node_to_pxm_map[MAX_NUMNODES]
+				= { [0 ... MAX_NUMNODES - 1] = PXM_INVAL };
+
 extern int __init acpi_table_parse_madt_family(enum acpi_table_id id,
 					       unsigned long madt_size,
 					       int entry_id,
 					       acpi_madt_entry_handler handler,
 					       unsigned int max_entries);
 
+int __cpuinit pxm_to_node(int pxm)
+{
+	if (pxm < 0)
+		return NID_INVAL;
+	return pxm_to_node_map[pxm];
+}
+
+int __cpuinit node_to_pxm(int node)
+{
+	if (node < 0)
+		return PXM_INVAL;
+	return node_to_pxm_map[node];
+}
+
+int __cpuinit acpi_map_pxm_to_node(int pxm)
+{
+	int node = pxm_to_node_map[pxm];
+
+	if (node < 0){
+		if (nodes_weight(nodes_found_map) >= MAX_NUMNODES)
+			return NID_INVAL;
+		node = first_unset_node(nodes_found_map);
+		pxm_to_node_map[pxm] = node;
+		node_to_pxm_map[node] = pxm;
+		node_set(node, nodes_found_map);
+	}
+
+	return node;
+}
+
+void __cpuinit acpi_unmap_pxm_to_node(int node)
+{
+	int pxm = node_to_pxm_map[node];
+	pxm_to_node_map[pxm] = NID_INVAL;
+	node_to_pxm_map[node] = PXM_INVAL;
+	node_clear(node, nodes_found_map);
+}
+
 void __init acpi_table_print_srat_entry(acpi_table_entry_header * header)
 {
 
Index: pxm_ver4/include/acpi/acpi_numa.h
===================================================================
--- /dev/null	1970-01-01 00:00:00.000000000 +0000
+++ pxm_ver4/include/acpi/acpi_numa.h	2006-03-29 22:19:30.000000000 +0900
@@ -0,0 +1,23 @@
+#ifndef __ACPI_NUMA_H
+#define __ACPI_NUMA_H
+
+#ifdef CONFIG_ACPI_NUMA
+#include <linux/kernel.h>
+
+/* Proximity bitmap length */
+#if MAX_NUMNODES > 256
+#define MAX_PXM_DOMAINS MAX_NUMNODES
+#else
+#define MAX_PXM_DOMAINS (256) /* Old pxm spec is defined 8 bit */
+#endif
+
+extern int __cpuinitdata pxm_to_node_map[MAX_PXM_DOMAINS];
+extern int __cpuinitdata node_to_pxm_map[MAX_NUMNODES];
+
+extern int __cpuinit pxm_to_node(int);
+extern int __cpuinit node_to_pxm(int);
+extern int __cpuinit acpi_map_pxm_to_node(int);
+extern void __cpuinit acpi_unmap_pxm_to_node(int);
+
+#endif				/* CONFIG_ACPI_NUMA */
+#endif				/* __ACP_NUMA_H */
Index: pxm_ver4/include/linux/acpi.h
===================================================================
--- pxm_ver4.orig/include/linux/acpi.h	2006-03-29 22:11:18.000000000 +0900
+++ pxm_ver4/include/linux/acpi.h	2006-03-29 22:12:41.000000000 +0900
@@ -38,6 +38,7 @@
 #include <acpi/acpi.h>
 #include <acpi/acpi_bus.h>
 #include <acpi/acpi_drivers.h>
+#include <acpi/acpi_numa.h>
 #include <asm/acpi.h>
 
 

-- 
Yasunori Goto 



^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Patch:002/004]Unify pxm_to_node id ver.4. (for ia64)
  2006-03-30 11:49 [Patch:000/004]Unify pxm_to_node id ver.4 Yasunori Goto
  2006-03-30 11:55 ` [Patch:001/004]Unify pxm_to_node id ver.4. (generic part) Yasunori Goto
@ 2006-03-30 11:55 ` Yasunori Goto
  2006-03-30 11:55 ` [Patch:003/004]Unify pxm_to_node id ver.4. (for x86-64) Yasunori Goto
  2006-03-30 11:55 ` [Patch:004/004]Unify pxm_to_node id ver.4. (for i386) Yasunori Goto
  3 siblings, 0 replies; 5+ messages in thread
From: Yasunori Goto @ 2006-03-30 11:55 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Linux Kernel ML

This is to use generic pxm_to_node() function instead of
old pxm_to_nid_map for ia64.

Signed-off-by: Yasunori Goto <y-goto@jp.fujitsu.com>

 arch/ia64/hp/common/sba_iommu.c |    2 +-
 arch/ia64/kernel/acpi.c         |   24 ++++++++----------------
 arch/ia64/pci/pci.c             |    2 +-
 arch/ia64/sn/kernel/setup.c     |    4 ++--
 include/asm-ia64/acpi.h         |   11 -----------
 5 files changed, 12 insertions(+), 31 deletions(-)

Index: pxm_ver4/arch/ia64/kernel/acpi.c
===================================================================
--- pxm_ver4.orig/arch/ia64/kernel/acpi.c	2006-03-29 22:06:10.000000000 +0900
+++ pxm_ver4/arch/ia64/kernel/acpi.c	2006-03-29 22:10:54.000000000 +0900
@@ -415,9 +415,6 @@ static int __initdata srat_num_cpus;	/* 
 static u32 __devinitdata pxm_flag[PXM_FLAG_LEN];
 #define pxm_bit_set(bit)	(set_bit(bit,(void *)pxm_flag))
 #define pxm_bit_test(bit)	(test_bit(bit,(void *)pxm_flag))
-/* maps to convert between proximity domain and logical node ID */
-int __devinitdata pxm_to_nid_map[MAX_PXM_DOMAINS];
-int __initdata nid_to_pxm_map[MAX_NUMNODES];
 static struct acpi_table_slit __initdata *slit_table;
 
 static int get_processor_proximity_domain(struct acpi_table_processor_affinity *pa)
@@ -533,22 +530,17 @@ void __init acpi_numa_arch_fixup(void)
 	 * MCD - This can probably be dropped now.  No need for pxm ID to node ID
 	 * mapping with sparse node numbering iff MAX_PXM_DOMAINS <= MAX_NUMNODES.
 	 */
-	/* calculate total number of nodes in system from PXM bitmap */
-	memset(pxm_to_nid_map, -1, sizeof(pxm_to_nid_map));
-	memset(nid_to_pxm_map, -1, sizeof(nid_to_pxm_map));
 	nodes_clear(node_online_map);
 	for (i = 0; i < MAX_PXM_DOMAINS; i++) {
 		if (pxm_bit_test(i)) {
-			int nid = num_online_nodes();
-			pxm_to_nid_map[i] = nid;
-			nid_to_pxm_map[nid] = i;
+			int nid = acpi_map_pxm_to_node(i);
 			node_set_online(nid);
 		}
 	}
 
 	/* set logical node id in memory chunk structure */
 	for (i = 0; i < num_node_memblks; i++)
-		node_memblk[i].nid = pxm_to_nid_map[node_memblk[i].nid];
+		node_memblk[i].nid = pxm_to_node(node_memblk[i].nid);
 
 	/* assign memory bank numbers for each chunk on each node */
 	for_each_online_node(i) {
@@ -562,7 +554,7 @@ void __init acpi_numa_arch_fixup(void)
 
 	/* set logical node id in cpu structure */
 	for (i = 0; i < srat_num_cpus; i++)
-		node_cpuid[i].nid = pxm_to_nid_map[node_cpuid[i].nid];
+		node_cpuid[i].nid = pxm_to_node(node_cpuid[i].nid);
 
 	printk(KERN_INFO "Number of logical nodes in system = %d\n",
 	       num_online_nodes());
@@ -575,11 +567,11 @@ void __init acpi_numa_arch_fixup(void)
 	for (i = 0; i < slit_table->localities; i++) {
 		if (!pxm_bit_test(i))
 			continue;
-		node_from = pxm_to_nid_map[i];
+		node_from = pxm_to_node(i);
 		for (j = 0; j < slit_table->localities; j++) {
 			if (!pxm_bit_test(j))
 				continue;
-			node_to = pxm_to_nid_map[j];
+			node_to = pxm_to_node(j);
 			node_distance(node_from, node_to) =
 			    slit_table->entry[i * slit_table->localities + j];
 		}
@@ -785,9 +777,9 @@ int acpi_map_cpu2node(acpi_handle handle
 
 	/*
 	 * Assuming that the container driver would have set the proximity
-	 * domain and would have initialized pxm_to_nid_map[pxm_id] && pxm_flag
+	 * domain and would have initialized pxm_to_node(pxm_id) && pxm_flag
 	 */
-	node_cpuid[cpu].nid = (pxm_id < 0) ? 0 : pxm_to_nid_map[pxm_id];
+	node_cpuid[cpu].nid = (pxm_id < 0) ? 0 : pxm_to_node(pxm_id);
 
 	node_cpuid[cpu].phys_id = physid;
 #endif
@@ -966,7 +958,7 @@ acpi_map_iosapic(acpi_handle handle, u32
 	if (pxm < 0)
 		return AE_OK;
 
-	node = pxm_to_nid_map[pxm];
+	node = pxm_to_node(pxm);
 
 	if (node >= MAX_NUMNODES || !node_online(node) ||
 	    cpus_empty(node_to_cpumask(node)))
Index: pxm_ver4/arch/ia64/pci/pci.c
===================================================================
--- pxm_ver4.orig/arch/ia64/pci/pci.c	2006-03-22 17:25:02.000000000 +0900
+++ pxm_ver4/arch/ia64/pci/pci.c	2006-03-29 22:10:54.000000000 +0900
@@ -353,7 +353,7 @@ pci_acpi_scan_root(struct acpi_device *d
 	pxm = acpi_get_pxm(controller->acpi_handle);
 #ifdef CONFIG_NUMA
 	if (pxm >= 0)
-		controller->node = pxm_to_nid_map[pxm];
+		controller->node = pxm_to_node(pxm);
 #endif
 
 	acpi_walk_resources(device->handle, METHOD_NAME__CRS, count_window,
Index: pxm_ver4/arch/ia64/hp/common/sba_iommu.c
===================================================================
--- pxm_ver4.orig/arch/ia64/hp/common/sba_iommu.c	2006-01-05 15:43:10.000000000 +0900
+++ pxm_ver4/arch/ia64/hp/common/sba_iommu.c	2006-03-29 22:10:54.000000000 +0900
@@ -1958,7 +1958,7 @@ sba_map_ioc_to_node(struct ioc *ioc, acp
 	if (pxm < 0)
 		return;
 
-	node = pxm_to_nid_map[pxm];
+	node = pxm_to_node(pxm);
 
 	if (node >= MAX_NUMNODES || !node_online(node))
 		return;
Index: pxm_ver4/arch/ia64/sn/kernel/setup.c
===================================================================
--- pxm_ver4.orig/arch/ia64/sn/kernel/setup.c	2006-03-29 22:06:10.000000000 +0900
+++ pxm_ver4/arch/ia64/sn/kernel/setup.c	2006-03-29 22:10:54.000000000 +0900
@@ -139,7 +139,7 @@ static int __init pxm_to_nasid(int pxm)
 	int i;
 	int nid;
 
-	nid = pxm_to_nid_map[pxm];
+	nid = pxm_to_node(pxm);
 	for (i = 0; i < num_node_memblks; i++) {
 		if (node_memblk[i].nid == nid) {
 			return NASID_GET(node_memblk[i].start_paddr);
@@ -704,7 +704,7 @@ void __init build_cnode_tables(void)
 	 * cnode == node for all C & M bricks.
 	 */
 	for_each_online_node(node) {
-		nasid = pxm_to_nasid(nid_to_pxm_map[node]);
+		nasid = pxm_to_nasid(node_to_pxm(node));
 		sn_cnodeid_to_nasid[node] = nasid;
 		physical_node_map[nasid] = node;
 	}
Index: pxm_ver4/include/asm-ia64/acpi.h
===================================================================
--- pxm_ver4.orig/include/asm-ia64/acpi.h	2006-03-29 22:06:16.000000000 +0900
+++ pxm_ver4/include/asm-ia64/acpi.h	2006-03-29 22:10:54.000000000 +0900
@@ -109,17 +109,6 @@ extern unsigned int get_cpei_target_cpu(
 extern void prefill_possible_map(void);
 extern int additional_cpus;
 
-#ifdef CONFIG_ACPI_NUMA
-/* Proximity bitmap length; _PXM is at most 255 (8 bit)*/
-#ifdef CONFIG_IA64_NR_NODES
-#define MAX_PXM_DOMAINS CONFIG_IA64_NR_NODES
-#else
-#define MAX_PXM_DOMAINS (256)
-#endif
-extern int __devinitdata pxm_to_nid_map[MAX_PXM_DOMAINS];
-extern int __initdata nid_to_pxm_map[MAX_NUMNODES];
-#endif
-
 extern u16 ia64_acpiid_to_sapicid[];
 
 /*

-- 
Yasunori Goto 



^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Patch:003/004]Unify pxm_to_node id ver.4. (for x86-64)
  2006-03-30 11:49 [Patch:000/004]Unify pxm_to_node id ver.4 Yasunori Goto
  2006-03-30 11:55 ` [Patch:001/004]Unify pxm_to_node id ver.4. (generic part) Yasunori Goto
  2006-03-30 11:55 ` [Patch:002/004]Unify pxm_to_node id ver.4. (for ia64) Yasunori Goto
@ 2006-03-30 11:55 ` Yasunori Goto
  2006-03-30 11:55 ` [Patch:004/004]Unify pxm_to_node id ver.4. (for i386) Yasunori Goto
  3 siblings, 0 replies; 5+ messages in thread
From: Yasunori Goto @ 2006-03-30 11:55 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Linux Kernel ML


This is to remove the code of pxm_to_node from x86-64 code.

Signed-off-by: Yasunori Goto <y-goto@jp.fujitsu.com>

 arch/x86_64/mm/srat.c     |   33 +--------------------------------
 include/asm-x86_64/numa.h |    1 -
 2 files changed, 1 insertion(+), 33 deletions(-)

Index: pxm_ver4/arch/x86_64/mm/srat.c
===================================================================
--- pxm_ver4.orig/arch/x86_64/mm/srat.c	2006-03-30 09:35:03.000000000 +0900
+++ pxm_ver4/arch/x86_64/mm/srat.c	2006-03-30 18:50:33.847831171 +0900
@@ -22,35 +22,15 @@
 static struct acpi_table_slit *acpi_slit;
 
 static nodemask_t nodes_parsed __initdata;
-static nodemask_t nodes_found __initdata;
 static struct bootnode nodes[MAX_NUMNODES] __initdata;
-static u8 pxm2node[256] = { [0 ... 255] = 0xff };
 
 /* Too small nodes confuse the VM badly. Usually they result
    from BIOS bugs. */
 #define NODE_MIN_SIZE (4*1024*1024)
 
-static int node_to_pxm(int n);
-
-int pxm_to_node(int pxm)
-{
-	if ((unsigned)pxm >= 256)
-		return -1;
-	/* Extend 0xff to (int)-1 */
-	return (signed char)pxm2node[pxm];
-}
-
 static __init int setup_node(int pxm)
 {
-	unsigned node = pxm2node[pxm];
-	if (node == 0xff) {
-		if (nodes_weight(nodes_found) >= MAX_NUMNODES)
-			return -1;
-		node = first_unset_node(nodes_found); 
-		node_set(node, nodes_found);
-		pxm2node[pxm] = node;
-	}
-	return pxm2node[pxm];
+	return acpi_map_pxm_to_node(pxm);
 }
 
 static __init int conflicting_nodes(unsigned long start, unsigned long end)
@@ -292,17 +272,6 @@ int __init acpi_scan_nodes(unsigned long
 	return 0;
 }
 
-static int node_to_pxm(int n)
-{
-       int i;
-       if (pxm2node[n] == n)
-               return n;
-       for (i = 0; i < 256; i++)
-               if (pxm2node[i] == n)
-                       return i;
-       return 0;
-}
-
 int __node_distance(int a, int b)
 {
 	int index;
Index: pxm_ver4/include/asm-x86_64/numa.h
===================================================================
--- pxm_ver4.orig/include/asm-x86_64/numa.h	2006-03-30 09:35:13.000000000 +0900
+++ pxm_ver4/include/asm-x86_64/numa.h	2006-03-30 18:50:33.885917108 +0900
@@ -8,7 +8,6 @@ struct bootnode {
 };
 
 extern int compute_hash_shift(struct bootnode *nodes, int numnodes);
-extern int pxm_to_node(int nid);
 
 #define ZONE_ALIGN (1UL << (MAX_ORDER+PAGE_SHIFT))
 

-- 
Yasunori Goto 



^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Patch:004/004]Unify pxm_to_node id ver.4. (for i386)
  2006-03-30 11:49 [Patch:000/004]Unify pxm_to_node id ver.4 Yasunori Goto
                   ` (2 preceding siblings ...)
  2006-03-30 11:55 ` [Patch:003/004]Unify pxm_to_node id ver.4. (for x86-64) Yasunori Goto
@ 2006-03-30 11:55 ` Yasunori Goto
  3 siblings, 0 replies; 5+ messages in thread
From: Yasunori Goto @ 2006-03-30 11:55 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Linux Kernel ML


This is to remove the code of pxm_to_nid_map from i386 code.
And, some of changing Kconfig and dummy function for compile.

Signed-off-by: Yasunori Goto <y-goto@jp.fujitsu.com>

 arch/i386/Kconfig       |    6 ++++++
 arch/i386/kernel/srat.c |   19 ++-----------------
 drivers/acpi/Kconfig    |    2 +-
 include/linux/acpi.h    |    8 ++++++++
 4 files changed, 17 insertions(+), 18 deletions(-)

Index: pxm_ver4/arch/i386/kernel/srat.c
===================================================================
--- pxm_ver4.orig/arch/i386/kernel/srat.c	2006-01-05 15:43:10.000000000 +0900
+++ pxm_ver4/arch/i386/kernel/srat.c	2006-03-30 18:50:52.164237197 +0900
@@ -39,7 +39,6 @@
 #define NODE_ARRAY_OFFSET(x)	((x) % 8)	/* 8 bits/char */
 #define BMAP_SET(bmap, bit)	((bmap)[NODE_ARRAY_INDEX(bit)] |= 1 << NODE_ARRAY_OFFSET(bit))
 #define BMAP_TEST(bmap, bit)	((bmap)[NODE_ARRAY_INDEX(bit)] & (1 << NODE_ARRAY_OFFSET(bit)))
-#define MAX_PXM_DOMAINS		256	/* 1 byte and no promises about values */
 /* bitmap length; _PXM is at most 255 */
 #define PXM_BITMAP_LEN (MAX_PXM_DOMAINS / 8) 
 static u8 pxm_bitmap[PXM_BITMAP_LEN];	/* bitmap of proximity domains */
@@ -213,19 +212,11 @@ static __init void node_read_chunk(int n
 		node_end_pfn[nid] = memory_chunk->end_pfn;
 }
 
-static u8 pxm_to_nid_map[MAX_PXM_DOMAINS];/* _PXM to logical node ID map */
-
-int pxm_to_node(int pxm)
-{
-	return pxm_to_nid_map[pxm];
-}
-
 /* Parse the ACPI Static Resource Affinity Table */
 static int __init acpi20_parse_srat(struct acpi_table_srat *sratp)
 {
 	u8 *start, *end, *p;
 	int i, j, nid;
-	u8 nid_to_pxm_map[MAX_NUMNODES];/* logical node ID to _PXM map */
 
 	start = (u8 *)(&(sratp->reserved) + 1);	/* skip header */
 	p = start;
@@ -235,10 +226,6 @@ static int __init acpi20_parse_srat(stru
 	memset(node_memory_chunk, 0, sizeof(node_memory_chunk));
 	memset(zholes_size, 0, sizeof(zholes_size));
 
-	/* -1 in these maps means not available */
-	memset(pxm_to_nid_map, -1, sizeof(pxm_to_nid_map));
-	memset(nid_to_pxm_map, -1, sizeof(nid_to_pxm_map));
-
 	num_memory_chunks = 0;
 	while (p < end) {
 		switch (*p) {
@@ -278,9 +265,7 @@ static int __init acpi20_parse_srat(stru
 	nodes_clear(node_online_map);
 	for (i = 0; i < MAX_PXM_DOMAINS; i++) {
 		if (BMAP_TEST(pxm_bitmap, i)) {
-			nid = num_online_nodes();
-			pxm_to_nid_map[i] = nid;
-			nid_to_pxm_map[nid] = i;
+			int nid = acpi_map_pxm_to_node(i);
 			node_set_online(nid);
 		}
 	}
@@ -288,7 +273,7 @@ static int __init acpi20_parse_srat(stru
 
 	/* set cnode id in memory chunk structure */
 	for (i = 0; i < num_memory_chunks; i++)
-		node_memory_chunk[i].nid = pxm_to_nid_map[node_memory_chunk[i].pxm];
+		node_memory_chunk[i].nid = pxm_to_node(node_memory_chunk[i].pxm);
 
 	printk("pxm bitmap: ");
 	for (i = 0; i < sizeof(pxm_bitmap); i++) {
Index: pxm_ver4/arch/i386/Kconfig
===================================================================
--- pxm_ver4.orig/arch/i386/Kconfig	2006-03-30 09:35:13.000000000 +0900
+++ pxm_ver4/arch/i386/Kconfig	2006-03-30 18:50:52.193534071 +0900
@@ -144,6 +144,12 @@ config ACPI_SRAT
 	bool
 	default y
 	depends on NUMA && (X86_SUMMIT || X86_GENERICARCH)
+	select ACPI_NUMA
+
+config HAVE_ARCH_PARSE_SRAT
+       bool
+       default y
+       depends on ACPI_SRAT
 
 config X86_SUMMIT_NUMA
 	bool
Index: pxm_ver4/drivers/acpi/Kconfig
===================================================================
--- pxm_ver4.orig/drivers/acpi/Kconfig	2006-03-30 09:35:03.000000000 +0900
+++ pxm_ver4/drivers/acpi/Kconfig	2006-03-30 18:50:52.226737196 +0900
@@ -162,7 +162,7 @@ config ACPI_THERMAL
 config ACPI_NUMA
 	bool "NUMA support"
 	depends on NUMA
-	depends on (IA64 || X86_64)
+	depends on (X86_32 || IA64 || X86_64)
 	default y if IA64_GENERIC || IA64_SGI_SN2
 
 config ACPI_ASUS
Index: pxm_ver4/include/linux/acpi.h
===================================================================
--- pxm_ver4.orig/include/linux/acpi.h	2006-03-30 09:35:13.000000000 +0900
+++ pxm_ver4/include/linux/acpi.h	2006-03-30 18:50:52.251151258 +0900
@@ -409,10 +409,18 @@ void acpi_table_print_madt_entry (acpi_t
 void acpi_table_print_srat_entry (acpi_table_entry_header *srat);
 
 /* the following four functions are architecture-dependent */
+#ifdef CONFIG_HAVE_ARCH_PARSE_SRAT
+#define NR_NODE_MEMBLKS MAX_NUMNODES
+#define acpi_numa_slit_init(slit) do {} while (0)
+#define acpi_numa_processor_affinity_init(pa) do {} while (0)
+#define acpi_numa_memory_affinity_init(ma) do {} while (0)
+#define acpi_numa_arch_fixup() do {} while (0)
+#else
 void acpi_numa_slit_init (struct acpi_table_slit *slit);
 void acpi_numa_processor_affinity_init (struct acpi_table_processor_affinity *pa);
 void acpi_numa_memory_affinity_init (struct acpi_table_memory_affinity *ma);
 void acpi_numa_arch_fixup(void);
+#endif
 
 #ifdef CONFIG_ACPI_HOTPLUG_CPU
 /* Arch dependent functions for cpu hotplug support */

-- 
Yasunori Goto 



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2006-03-30 11:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-03-30 11:49 [Patch:000/004]Unify pxm_to_node id ver.4 Yasunori Goto
2006-03-30 11:55 ` [Patch:001/004]Unify pxm_to_node id ver.4. (generic part) Yasunori Goto
2006-03-30 11:55 ` [Patch:002/004]Unify pxm_to_node id ver.4. (for ia64) Yasunori Goto
2006-03-30 11:55 ` [Patch:003/004]Unify pxm_to_node id ver.4. (for x86-64) Yasunori Goto
2006-03-30 11:55 ` [Patch:004/004]Unify pxm_to_node id ver.4. (for i386) Yasunori Goto

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®