This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: PATCH: Replace Elf64_XXX with ElfW(XXX) in dl-irel.h
On Wed, Mar 21, 2012 at 2:54 PM, Roland McGrath <roland@hack.frob.com> wrote:
>> This patch replaces Elf64_XXX with ElfW(XXX) in dl-irel.h. ?It also
>> uses ELF32_R_TYPE:
>>
>> #define ELF32_R_TYPE(val) ? ? ? ? ? ? ((val) & 0xff)
>>
>> instead ELF64_R_TYPE:
>>
>> #define ELF64_R_TYPE(i) ? ? ? ? ? ? ? ? ? ? ? ((i) & 0xffffffff)
>>
>> gets the currect relocation type for both ELF32 and ELF64 for x86-64 since
>> x86-64 relocation type is less than 256. ?OK to install?
>
> That is a kludgey and dangerous assumption. ?It would be better to have an
> ELFW macro like ElfW, so you can use "ELFW(R_TYPE) (reloc->r_info)". ?It's
> probably not a good idea to add a name like ELFW to the public header, even
> though ElfW is already there. ?So the new macro would go in include/link.h.
>
>
Here is the patch to implement it. Tested on Linux/x86-64 and Linux/x32.
OK to install?
Thanks.
--
H.J.
2012-03-22 H.J. Lu <hongjiu.lu@intel.com>
* include/link.h (ELFW): New macro.
* sysdeps/x86_64/dl-irel.h: Replace Elf64_XXX with ElfW(XXX).
Replace ELF64_R_TYPE with ELFW(R_TYPE).
diff --git a/include/link.h b/include/link.h
index c5265ee..363533c 100644
--- a/include/link.h
+++ b/include/link.h
@@ -330,4 +330,6 @@ extern int __dl_iterate_phdr (int (*callback) (struct dl_phdr_info *info,
size_t size, void *data),
void *data);
+#define ELFW(type) _ElfW (ELF, __ELF_NATIVE_CLASS, type)
+
#endif /* include/link.h */
diff --git a/sysdeps/x86_64/dl-irel.h b/sysdeps/x86_64/dl-irel.h
index bc7e54e..830f5db 100644
--- a/sysdeps/x86_64/dl-irel.h
+++ b/sysdeps/x86_64/dl-irel.h
@@ -25,23 +25,23 @@
#define ELF_MACHINE_IRELA 1
-static inline Elf64_Addr
+static inline ElfW(Addr)
__attribute ((always_inline))
-elf_ifunc_invoke (Elf64_Addr addr)
+elf_ifunc_invoke (ElfW(Addr) addr)
{
- return ((Elf64_Addr (*) (void)) (addr)) ();
+ return ((ElfW(Addr) (*) (void)) (addr)) ();
}
static inline void
__attribute ((always_inline))
-elf_irela (const Elf64_Rela *reloc)
+elf_irela (const ElfW(Rela) *reloc)
{
- Elf64_Addr *const reloc_addr = (void *) reloc->r_offset;
- const unsigned long int r_type = ELF64_R_TYPE (reloc->r_info);
+ ElfW(Addr) *const reloc_addr = (void *) reloc->r_offset;
+ const unsigned long int r_type = ELFW(R_TYPE) (reloc->r_info);
if (__builtin_expect (r_type == R_X86_64_IRELATIVE, 1))
{
- Elf64_Addr value = elf_ifunc_invoke(reloc->r_addend);
+ ElfW(Addr) value = elf_ifunc_invoke(reloc->r_addend);
*reloc_addr = value;
}
else