[Bug gas/20803] Sparc R_SPARC_32 reloc with miss-align offset.

2017-03-30 Thread chrisj at rtems dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=20803

--- Comment #6 from Chris Johns  ---
(In reply to Chris Johns from comment #5)
> I have just tested binutils 2.27 with the patch and 2.28 that contains this
> patch and the issue is back. The reloc details from readelf are:

This was a bug in the RTEMS ELF loader. I am sorry about the noise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/20803] Sparc R_SPARC_32 reloc with miss-align offset.

2017-03-29 Thread chrisj at rtems dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=20803

--- Comment #5 from Chris Johns  ---
I have just tested binutils 2.27 with the patch and 2.28 that contains this
patch and the issue is back. The reloc details from readelf are:

Relocation section '.rela.gcc_except_table.exception_dl' at offset 0x51cc
contains 2 entries:
 Offset InfoTypeSym. Value  Symbol's Name + Addend
0040  2a03 R_SPARC_32    _ZTISt9exception + 0
0044  2403 R_SPARC_32    _ZTI16dl_test_throw_me + 0

Relocation section '.rela.rodata._ZTI16dl_test_throw_me' at offset 0x51e4
contains 2 entries:
 Offset InfoTypeSym. Value  Symbol's Name + Addend
  2c03 R_SPARC_32   
_ZTVN10__cxxabiv117__class_type_infoE + 8
0004  2b03 R_SPARC_32    _ZTS16dl_test_throw_me + 0

The R_SPARC_32 should be R_SPARC_UA32.

I do not know how we reopen this bug.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/20803] Sparc R_SPARC_32 reloc with miss-align offset.

2016-11-15 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=20803

Nick Clifton  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Nick Clifton  ---
Patch applied.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/20803] Sparc R_SPARC_32 reloc with miss-align offset.

2016-11-15 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=20803

--- Comment #3 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=93ca393659ef6714724749950301757f47e5e9fd

commit 93ca393659ef6714724749950301757f47e5e9fd
Author: Nick Clifton 
Date:   Tue Nov 15 15:41:27 2016 +

Fix SPARC relocations generated for the .eh_frame section.

PR gas/20803
* config/tc-sparc.c (cons_fix_new_sparc): Use unaligned relocs in
the .eh_frame section.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/20803] Sparc R_SPARC_32 reloc with miss-align offset.

2016-11-14 Thread chrisj at rtems dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=20803

--- Comment #2 from Chris Johns  ---
(In reply to Nick Clifton from comment #1)
> Created attachment 9623 [details]
> Proposed patch

Thank you for the quick turn around.

> 
>   Please could you try out this patch and let me know if it is enough to
> solve the problem.
> 

Yes, this solves the problem we are seeing.

>   I am not sure if this approach is the correct way to fix the issue, but it
> does seem to be the simplest.  As far as I can tell, the relocs in the
> .eh_frame section can eb unaligned, so using R_SPARC_UA32 seems to be the
> correct thing to do.  I did look to see if I could enable
> sparc_no_align_cons when fixing the output for the .eh_frame section, but I
> could not find an easy way to do this. Hence I went for a hack based on the
> section name.  Not very subtle, but if it works then that is enough for now.

I also had a look and felt any change was a potential issue for some other
reason I would not be aware of. We will use this patch in RTEMS until the next
binutils release.

Thank you.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug gas/20803] Sparc R_SPARC_32 reloc with miss-align offset.

2016-11-11 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=20803

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

--- Comment #1 from Nick Clifton  ---
Created attachment 9623
  --> https://sourceware.org/bugzilla/attachment.cgi?id=9623&action=edit
Proposed patch

Hi Chris,

  Please could you try out this patch and let me know if it is enough to solve
the problem.

  I am not sure if this approach is the correct way to fix the issue, but it
does seem to be the simplest.  As far as I can tell, the relocs in the
.eh_frame section can eb unaligned, so using R_SPARC_UA32 seems to be the
correct thing to do.  I did look to see if I could enable sparc_no_align_cons
when fixing the output for the .eh_frame section, but I could not find an easy
way to do this. Hence I went for a hack based on the section name.  Not very
subtle, but if it works then that is enough for now.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils