This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add ARM relocation constants to elf/elf.h


"Joseph S. Myers" <joseph@codesourcery.com> writes:

> On Wed, 23 Jan 2013, Petr Machata wrote:
>
>> Roland McGrath <roland@hack.frob.com> writes:
>> 
>> > We strongly prefer to have a short comment on every reloc type #define.
>> 
>> I don't think I have enough expertise to concisely explain what every
>> relocation does.  I can at least put in e.g. what instructions each
>> static relocation applies to, and add snippets from the standard where
>> available.  Joseph, what is your preference?  FWIW, few relocation
>> values in ARM block have any comments at all.
>
> I'm not clear there is a great deal of use in having all these relocations 
> (that are irrelevant to ld.so) in elf.h; what are the expected users, 
> given that binutils has its own relocation definitions?  (Cf. how AArch64 
> only lists a limited subset of relocations in elf.h, not all the static 
> relocations that ld.so doesn't use.)

My impression always was that elf.h is essentially a C language
rendition of various Elf standards (sans algorithms).

What prompted me to post this patch in particular was that GCL folks
seem to be in a need of those defines [1].  I'm not in contact with them
personally, this issue was brought to my attention via Fedora ARM list.
I offered composing a patch as it seemed an obvious omission.

> But if they are added, I think including comments is appropriate.

OK, I'll prepare a patch with comments.

Thanks,
PM

[1] https://lists.gnu.org/archive/html/gcl-devel/2013-01/msg00012.html


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]