Prev: ACPI, APEI, ERST debug support
Next: usb gadget: don't save bind callback in struct usb_gadget_driver
From: Mikael Pettersson on 5 Aug 2010 04:50 Olof Johansson writes: > commit d0679c730395d0bde9a46939e7ba255b4ba7dd7c moved -fno-dwarf2-cfi-asm to be > powerpc-only, causing R_ARM_REL32 to be used with some toolchains on ARM. > > Implement support for it to make modules work with more toolchains. > > Signed-off-by: Olof Johansson <olof(a)lixom.net> Michal Marek's patch(*) to use -fno-dwarf2-cfi-asm already exists and is IMO the correct solution in that it addresses the cause of the problem rather than its symptoms. NAK from me for now. (*) http://lkml.org/lkml/2010/7/26/154 /Mikael > --- > arch/arm/include/asm/elf.h | 1 + > arch/arm/kernel/module.c | 4 ++++ > 2 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/include/asm/elf.h b/arch/arm/include/asm/elf.h > index 51662fe..b1f1991 100644 > --- a/arch/arm/include/asm/elf.h > +++ b/arch/arm/include/asm/elf.h > @@ -50,6 +50,7 @@ typedef struct user_fp elf_fpregset_t; > #define R_ARM_NONE 0 > #define R_ARM_PC24 1 > #define R_ARM_ABS32 2 > +#define R_ARM_REL32 3 > #define R_ARM_CALL 28 > #define R_ARM_JUMP24 29 > #define R_ARM_V4BX 40 > diff --git a/arch/arm/kernel/module.c b/arch/arm/kernel/module.c > index c628bdf..ec9526d 100644 > --- a/arch/arm/kernel/module.c > +++ b/arch/arm/kernel/module.c > @@ -132,6 +132,10 @@ apply_relocate(Elf32_Shdr *sechdrs, const char *strtab, unsigned int symindex, > *(u32 *)loc += sym->st_value; > break; > > + case R_ARM_REL32: > + *(u32 *)loc += sym->st_value - loc; > + break; > + > case R_ARM_PC24: > case R_ARM_CALL: > case R_ARM_JUMP24: > -- > 1.5.6.5 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel(a)lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Olof Johansson on 5 Aug 2010 11:10
On Thu, Aug 05, 2010 at 10:48:13AM +0200, Mikael Pettersson wrote: > Olof Johansson writes: > > commit d0679c730395d0bde9a46939e7ba255b4ba7dd7c moved -fno-dwarf2-cfi-asm to be > > powerpc-only, causing R_ARM_REL32 to be used with some toolchains on ARM. > > > > Implement support for it to make modules work with more toolchains. > > > > Signed-off-by: Olof Johansson <olof(a)lixom.net> > > Michal Marek's patch(*) to use -fno-dwarf2-cfi-asm already exists > and is IMO the correct solution in that it addresses the cause > of the problem rather than its symptoms. > > NAK from me for now. > > (*) http://lkml.org/lkml/2010/7/26/154 Sigh. I searched the list for related patches and didn't see any. Sorry for the duplicate. -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |