Skip to contentSkip to conversation

xen x86_64 and CONFIG_FLATMEM

PATCH In review Mark McLoughlin 2 participants 3 messages Jul 19, 2006 – Jul 25, 2006

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 xen x86_64 and CONFIG_FLATMEM

Hi,
With CONFIG_SPARSEMEM on xen x86_64, you get an oops in mem_init where
it references mem_map; mem_map is only initialized with CONFIG_FLATMEM.

Suggest a Kconfig change like the one attached?

Cheers,
Mark.

linux-2.6-xen-sparse/arch/x86_64/Kconfig +1 -1
diff -r ecb8ff1fcf1f linux-2.6-xen-sparse/arch/x86_64/Kconfig
@@ -330,7 +330,7 @@ config ARCH_DISCONTIGMEM_DEFAULT
 
 config ARCH_SPARSEMEM_ENABLE
 	def_bool y
-	depends on (NUMA || EXPERIMENTAL)
+	depends on (NUMA || EXPERIMENTAL) && !X86_64_XEN
 
 config ARCH_MEMORY_PROBE
 	def_bool y

_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xensource.com/xen-devel

#1 lore raw
Quoted 4 lines · Mark McLoughlin

With CONFIG_SPARSEMEM on xen x86_64, you get an oops in mem_init where
it references mem_map; mem_map is only initialized with CONFIG_FLATMEM.

Suggest a Kconfig change like the one attached?

Perhaps better to forcibly disable CONFIG_NUMA if CONFIG_XEN is enabled? We can't (yet) pick up NUMA info from Xen anyway, and it probably enables a bunch of code that hasn't been much tested on Xen.

-- Keir

#2 lore raw
Quoted 10 lines · Keir Fraser
Quoted 4 lines · Mark McLoughlin

With CONFIG_SPARSEMEM on xen x86_64, you get an oops in mem_init where
it references mem_map; mem_map is only initialized with CONFIG_FLATMEM.

Suggest a Kconfig change like the one attached?

Perhaps better to forcibly disable CONFIG_NUMA if CONFIG_XEN is enabled? We can't (yet) pick up NUMA info from Xen anyway, and it probably enables a bunch of code that hasn't been much tested on Xen.

NUMA is already disabled if CONFIG_XEN is enabled. But SPARSEMEM can be
enabled even without NUMA:

config ARCH_SPARSEMEM_ENABLE
def_bool y

-       depends on (NUMA || EXPERIMENTAL)
+       depends on (NUMA || EXPERIMENTAL) && !X86_64_XEN

Cheers,
Mark.