Revert "kbuild: fix modpost segfault for 64bit mipsel kernel"
authorLinus Torvalds <torvalds@g5.osdl.org>
Mon, 8 May 2006 20:38:42 +0000 (13:38 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Mon, 8 May 2006 20:38:42 +0000 (13:38 -0700)
This reverts commit c8d8b837ebe4b4f11e1b0c4a2bdc358c697692ed, which
caused problems for the x86 build. Quoth Sam:

  "It was discussed on mips list but apparently the fix was bogus.  I
   will not have time to look into it so mips can carry this local fix
   until we get a proper fix in mainline."

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
scripts/mod/modpost.c
scripts/mod/modpost.h

index b36e884f5f967dae578955bc0df37e8b4cb1c7e0..6d04504b2fc111a0bbd7ac8745fa9f96e008e72a 100644 (file)
@@ -709,17 +709,10 @@ static void check_sec_ref(struct module *mod, const char *modname,
                for (rela = start; rela < stop; rela++) {
                        Elf_Rela r;
                        const char *secname;
-                       unsigned int r_sym;
                        r.r_offset = TO_NATIVE(rela->r_offset);
-                       if (hdr->e_ident[EI_CLASS] == ELFCLASS64 &&
-                           hdr->e_machine == EM_MIPS) {
-                               r_sym = ELF64_MIPS_R_SYM(rela->r_info);
-                               r_sym = TO_NATIVE(r_sym);
-                       } else {
-                               r_sym = ELF_R_SYM(TO_NATIVE(rela->r_info));
-                       }
+                       r.r_info   = TO_NATIVE(rela->r_info);
                        r.r_addend = TO_NATIVE(rela->r_addend);
-                       sym = elf->symtab_start + r_sym;
+                       sym = elf->symtab_start + ELF_R_SYM(r.r_info);
                        /* Skip special sections */
                        if (sym->st_shndx >= SHN_LORESERVE)
                                continue;
index 89b96c6d8ef587f3995d9975ca67e9ee2971ea18..b14255c72a375edcfe6da9f61589ba2e433deb94 100644 (file)
 #define ELF_R_TYPE  ELF64_R_TYPE
 #endif
 
-/* The 64-bit MIPS ELF ABI uses an unusual reloc format. */
-typedef struct
-{
-  Elf32_Word    r_sym;         /* Symbol index */
-  unsigned char r_ssym;                /* Special symbol for 2nd relocation */
-  unsigned char r_type3;       /* 3rd relocation type */
-  unsigned char r_type2;       /* 2nd relocation type */
-  unsigned char r_type1;       /* 1st relocation type */
-} _Elf64_Mips_R_Info;
-
-typedef union
-{
-  Elf64_Xword  r_info_number;
-  _Elf64_Mips_R_Info r_info_fields;
-} _Elf64_Mips_R_Info_union;
-
-#define ELF64_MIPS_R_SYM(i) \
-  ((__extension__ (_Elf64_Mips_R_Info_union)(i)).r_info_fields.r_sym)
-
 #if KERNEL_ELFDATA != HOST_ELFDATA
 
 static inline void __endian(const void *src, void *dest, unsigned int size)