[users/roland/osabi] Recognize GNU .note.ABI-tag values 5 (syllable) and 6 (nacl)

  These two values are assigned in glibc trunk (5 has been assigned to
syllable for several years, and 6 is quite recent).  I don't actually know
anything about Syllable, but IMHO it makes sense for the other GNU packages
that think about these values at all (binutils/gdb) to grok all the values
that exist in the GNU package that owns this number space (libc).

OK for trunk and gdb's 7.9 branch?  (I don't really care about the readelf
change, but I'm happy to put it on binutils' 2.25 branch if anyone thinks
it's worthwhile.)


Thanks,
Roland


binutils/
	* readelf.c (print_gnu_note: NT_GNU_ABI_TAG): Recognize
	GNU_ABI_TAG_SYLLABLE and GNU_ABI_TAG_NACL.

gdb/
	* defs.h (enum gdb_osabi): Add GDB_OSABI_SYLLABLE and GDB_OSABI_NACL.
	* osabi.c (generic_elf_osabi_sniff_abi_tag_sections): Recognize
	GNU_ABI_TAG_SYLLABLE and GNU_ABI_TAG_NACL.
	* configure.tgt (*-*-nacl*): Set gdb_osabi=GDB_OSABI_NACL.

include/elf/
	* common.h (GNU_ABI_TAG_SYLLABLE): New macro.
	(GNU_ABI_TAG_NACL): New macro.
  

Hi Roland,

> binutils/
> 	* readelf.c (print_gnu_note: NT_GNU_ABI_TAG): Recognize
> 	GNU_ABI_TAG_SYLLABLE and GNU_ABI_TAG_NACL.

> include/elf/
> 	* common.h (GNU_ABI_TAG_SYLLABLE): New macro.
> 	(GNU_ABI_TAG_NACL): New macro.

These parts are approved.

Cheers
   Nick
  

On 05/29/2015 02:10 AM, Roland McGrath wrote:
> These two values are assigned in glibc trunk (5 has been assigned to
> syllable for several years, and 6 is quite recent).  I don't actually know
> anything about Syllable, but IMHO it makes sense for the other GNU packages
> that think about these values at all (binutils/gdb) to grok all the values
> that exist in the GNU package that owns this number space (libc).
> 
> OK for trunk and gdb's 7.9 branch?  (I don't really care about the readelf
> change, but I'm happy to put it on binutils' 2.25 branch if anyone thinks
> it's worthwhile.)

I think adding the constants to include/elf/common.h, and making
generic_elf_osabi_sniff_abi_tag_sections recognize syllable and nacl,
avoiding the internal_error (or warning after your other patch) is OK
and could go to the 7.9 branch.  Please split that out to a separate patch.

The rest of the gdb bits can be considered for trunk.  I don't think we want
GDB_OSABI_SYLLABLE exposed to the rest of the gdb (I don't know anything
about Syllable either), and to users (both "set osabi" and xml target
descriptions).  We may need to expose GDB_OSABI_NACL, but that's probably
best added along with the nacl port, assuming there's one.

Thanks,
Pedro Alves
  

On Fri, May 29, 2015 at 2:58 AM, Nicholas Clifton <nickc@redhat.com> wrote:
>> binutils/
>>         * readelf.c (print_gnu_note: NT_GNU_ABI_TAG): Recognize
>>         GNU_ABI_TAG_SYLLABLE and GNU_ABI_TAG_NACL.
>> include/elf/
>>         * common.h (GNU_ABI_TAG_SYLLABLE): New macro.
>>         (GNU_ABI_TAG_NACL): New macro.
>
> These parts are approved.

Committed.

Thanks,
Roland
  

On Fri, May 29, 2015 at 3:29 AM, Pedro Alves <palves@redhat.com> wrote:
> I think adding the constants to include/elf/common.h,

I've already committed common.h based on the binutils approval.

> and making
> generic_elf_osabi_sniff_abi_tag_sections recognize syllable and nacl,
> avoiding the internal_error (or warning after your other patch) is OK
> and could go to the 7.9 branch.  Please split that out to a separate patch.

Do you mean just silently ignoring these values?

> The rest of the gdb bits can be considered for trunk.  I don't think we want
> GDB_OSABI_SYLLABLE exposed to the rest of the gdb (I don't know anything
> about Syllable either), and to users (both "set osabi" and xml target
> descriptions).  We may need to expose GDB_OSABI_NACL, but that's probably
> best added along with the nacl port, assuming there's one.

OK.  I'll ignore Syllable entirely, since I was only doing it for
completeness.

For NaCl, I'll investigate further before proposing something.  We have a
patch that is mostly about x86 and needs some cleanup (or probably complete
rewriting).  NaCl's x86-64 is particularly odd and I don't want to worry
about the weird cruft we have for that right now.  At the moment, I'm
focused on on arm-nacl.  NaCl is an all-remote target (the only kind of
"native" debugging interface available is a gdb stub).  So there really is
almost nothing to do.  But when I get some time I'll look into building
--target=arm-nacl on trunk and see what it needs.


Thanks,
Roland
  

On 05/29/2015 05:47 PM, Roland McGrath wrote:
> On Fri, May 29, 2015 at 3:29 AM, Pedro Alves <palves@redhat.com> wrote:
>> I think adding the constants to include/elf/common.h,
> 
> I've already committed common.h based on the binutils approval.
> 
>> and making
>> generic_elf_osabi_sniff_abi_tag_sections recognize syllable and nacl,
>> avoiding the internal_error (or warning after your other patch) is OK
>> and could go to the 7.9 branch.  Please split that out to a separate patch.
> 
> Do you mean just silently ignoring these values?

I do.

> 
>> The rest of the gdb bits can be considered for trunk.  I don't think we want
>> GDB_OSABI_SYLLABLE exposed to the rest of the gdb (I don't know anything
>> about Syllable either), and to users (both "set osabi" and xml target
>> descriptions).  We may need to expose GDB_OSABI_NACL, but that's probably
>> best added along with the nacl port, assuming there's one.
> 
> OK.  I'll ignore Syllable entirely, since I was only doing it for
> completeness.
> 
> For NaCl, I'll investigate further before proposing something.  We have a
> patch that is mostly about x86 and needs some cleanup (or probably complete
> rewriting).  NaCl's x86-64 is particularly odd and I don't want to worry
> about the weird cruft we have for that right now.  At the moment, I'm
> focused on on arm-nacl.  NaCl is an all-remote target (the only kind of
> "native" debugging interface available is a gdb stub).  So there really is
> almost nothing to do.  But when I get some time I'll look into building
> --target=arm-nacl on trunk and see what it needs.

OK.  Looking forward to hear about this.

Thanks,
Pedro Alves