Re: [bug] SLUB + mm/slab.c boot crash in -rc9

From: Ingo Molnar
Date: Tue Apr 15 2008 - 17:29:29 EST



* Christoph Lameter <clameter@xxxxxxx> wrote:

> On Tue, 15 Apr 2008, Ingo Molnar wrote:
>
> > > static DEFINE_PER_CPU(char, dummy)[10000];
> >
> > what's the guarantee that it's at the end of the section? I'd like
> > to pad the per cpu areas at their end. (doubling their size is a
> > good way to achieve that)
>
> No guarantee. Its up to the linker. Sorry. We could add a new
> percpu.last section but that requires a number of changes to linking.

ah. Then the patch below should do the trick, right?

Ingo

------------->
Subject: larger: percpu
From: Ingo Molnar <mingo@xxxxxxx>
Date: Tue Apr 15 23:13:18 CEST 2008

Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
---
arch/x86/kernel/vmlinux_32.lds.S | 1 +
1 file changed, 1 insertion(+)

Index: linux/arch/x86/kernel/vmlinux_32.lds.S
===================================================================
--- linux.orig/arch/x86/kernel/vmlinux_32.lds.S
+++ linux/arch/x86/kernel/vmlinux_32.lds.S
@@ -186,6 +186,7 @@ SECTIONS
__per_cpu_start = .;
*(.data.percpu)
*(.data.percpu.shared_aligned)
+ . = . + 65536;
__per_cpu_end = .;
}
. = ALIGN(PAGE_SIZE);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/