On Thu, 2008-08-28 at 15:23 +0100, David Woodhouse wrote: > On Thu, 2008-08-28 at 00:38 +1000, Stephen Rothwell wrote: > > Hi Arjan, > > > > On Thu, 28 Aug 2008 00:33:08 +1000 Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote: > > > > > > The original reported trace was during setup_system which is very early in > > > the boot. > > > > But, of course, that version didn't have the necessary extra dereference > > of the function address ... > > > > And the later debug patch did not check the address at register time, > > only at notify time. > > > > The later trace also looks to be early in the boot. > > It's isa_bridge_notify(), which is neither within _[se]text nor > _[se]inittext, so the core_kernel_text() function disavows it. > > Where are __devinit functions supposed to end up? The TEXT_TEXT macro defined in <asm-generic/vmlinux.lds.h> should get this right... but we don't use it. Is there any particular reason for that, or should we.... Signed-off-by: David Woodhouse <David.Woodhouse@xxxxxxxxx> --- linux-2.6.26.ppc64/arch/powerpc/kernel/vmlinux.lds.S~ 2008-07-13 22:51:29.000000000 +0100 +++ linux-2.6.26.ppc64/arch/powerpc/kernel/vmlinux.lds.S 2008-08-28 15:39:14.000000000 +0100 @@ -35,10 +35,11 @@ SECTIONS ALIGN_FUNCTION(); *(.text.head) _text = .; - *(.text .fixup .text.init.refok .exit.text.refok) + TEXT_TEXT SCHED_TEXT LOCK_TEXT KPROBES_TEXT + *(.fixup) #ifdef CONFIG_PPC32 *(.got1) -- David Woodhouse Open Source Technology Centre David.Woodhouse@xxxxxxxxx Intel Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
