On Mon, Feb 13, 2012 at 11:26:52PM +0100, Uwe Kleine-König wrote: > On Mon, Feb 13, 2012 at 01:24:01PM -0800, Tony Lindgren wrote: > > It turns out that many compilers don't show section warnings on ARM > > currently because handling for ARM_CALL relocs are missing from > > modpost.c. > > > > Based on commit c2e26114 ([ARM] 3205/1: Handle new EABI relocations when > > loading kernel modules) it seems that R_ARM_PC24, R_ARM_CALL and > > R_ARM_JUMP24 can be handled the same way. > > > > As modpost.c includes elf.h, we need to also consider that at least > > Debian libc6-dev is missing defines for both R_ARM_CALL and R_ARM_JUMP24 > > in /usr/include/elf.h. > Huh, even unstable's /usr/include/elf.h doesn't have these symbols. > > > So for now let's just use the numbers in modpost.c. > > > > Cc: Rusty Russell <rusty@xxxxxxxxxxxxxxx> > > Cc: Ben Hutchings <ben@xxxxxxxxxxxxxxx> > > Cc: Anders Kaseorg <andersk@xxxxxxxxxxx> > > Cc: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> > > Cc: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx> > > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx> > > > > --- > > > > Anybody got better ideas for dealing with the missing elf.h > > defines? > Maybe: > > #ifndef R_ARM_CALL > #warning "you're elf.h include is outdated" "You are elf.h include is outdated" does not make sense. Why are you calling Tony an elf.h include? -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
