public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Eric Botcazou <ebotcazou@libertysurf.fr>
To: Jakub Jelinek <jakub@redhat.com>
Cc: binutils@sources.redhat.com,
	Christian Joensson <christian.joensson@gmail.com>
Subject: Re: Q: Estimated date for 2.16 release ?
Date: Mon, 14 Feb 2005 10:46:00 -0000	[thread overview]
Message-ID: <200502140118.48129.ebotcazou@libertysurf.fr> (raw)
In-Reply-To: <20050127143409.GU4777@sunsite.mff.cuni.cz>

[-- Attachment #1: Type: text/plain, Size: 736 bytes --]

[Sorry for the delay]

> My understanding is that Eric fixed the bug and all that needs to be done
> is simply adjust the testcase expected results (after verifying linker
> output is correct) for the added .got STT_SECTION symbol etc.

Right, tlssunpic32.rd only needs to account for .got in .dynsym again, which 
essentially boiled down to shifting the dynamic indexes.

> The test that was failing before Eric's patch was not even adjusted for
> my .dynsym changes.

Right, tlssunnopic32.rd was out of sync for the section order, the contents 
of .dynsym and the type of TLS symbols (it was NOTYPE).

Fixed thusly.


	* ld-sparc/tlssunnopic32.rd: Adjust for .dynsym changes.
	* ld-sparc/tlssunpic32.rd: Likewise.


-- 
Eric Botcazou

[-- Attachment #2: tls_sparc_testsuite.diff --]
[-- Type: text/x-diff, Size: 7343 bytes --]

Index: tlssunnopic32.rd
===================================================================
RCS file: /cvs/src/src/ld/testsuite/ld-sparc/tlssunnopic32.rd,v
retrieving revision 1.1
diff -u -r1.1 tlssunnopic32.rd
--- tlssunnopic32.rd	24 Jan 2003 23:44:45 -0000	1.1
+++ tlssunnopic32.rd	14 Feb 2005 00:01:16 -0000
@@ -15,11 +15,11 @@
  +\[ 3\] .dynstr +.*
  +\[ 4\] .rela.dyn +.*
  +\[ 5\] .text +PROGBITS +0+1000 0+1000 0+1000 0+ +AX +0 +0 4096
- +\[ 6\] .data +PROGBITS +0+12000 0+2000 0+ 0+ +WA +0 +0 4096
- +\[ 7\] .tbss +NOBITS +0+12000 0+2000 0+24 0+ WAT +0 +0 +4
- +\[ 8\] .dynamic +DYNAMIC +0+12000 0+2000 0+80 08 +WA +3 +0 +4
+ +\[ 6\] .tbss +NOBITS +0+12000 0+2000 0+24 0+ WAT +0 +0 +4
+ +\[ 7\] .dynamic +DYNAMIC +0+12000 0+2000 0+80 08 +WA +3 +0 +4
+ +\[ 8\] .got +PROGBITS +0+12080 0+2080 0+1c 04 +WA +0 +0 +4
  +\[ 9\] .plt +.*
- +\[10\] .got +PROGBITS +0+12080 0+2080 0+1c 04 +WA +0 +0 +4
+ +\[10\] .data +PROGBITS +0+13000 0+3000 0+ 0+ +WA +0 +0 4096
  +\[11\] .bss +.*
  +\[12\] .shstrtab +.*
  +\[13\] .symtab +.*
@@ -32,15 +32,15 @@
 Program Headers:
  +Type +Offset +VirtAddr +PhysAddr +FileSiz MemSiz +Flg Align
  +LOAD +0x0+ 0x0+ 0x0+ 0x0+2000 0x0+2000 R E 0x10000
- +LOAD +0x0+2000 0x0+12000 0x0+12000 0x0+9c 0x0+a0 RWE 0x10000
+ +LOAD +0x0+2000 0x0+12000 0x0+12000 0x0+1000 0x0+1000 RWE 0x10000
  +DYNAMIC +0x0+2000 0x0+12000 0x0+12000 0x0+80 0x0+80 RW +0x4
  +TLS +0x0+2000 0x0+12000 0x0+12000 0x0+ 0x0+24 R +0x4
 #...
 
 Relocation section '.rela.dyn' at offset 0x[0-9a-f]+ contains 12 entries:
  Offset +Info +Type +Sym. Value +Symbol's Name \+ Addend
-0+1004 +0+a09 R_SPARC_HI22 +0+12080 +\.got \+ 12080
-0+1008 +0+a0c R_SPARC_LO10 +0+12080 +\.got \+ 12080
+0+1004 +0+309 R_SPARC_HI22 +0+12080 +\.got \+ 12080
+0+1008 +0+30c R_SPARC_LO10 +0+12080 +\.got \+ 12080
 0+10dc +0+48 R_SPARC_TLS_LE_HIX22 +0+9
 0+10e0 +0+49 R_SPARC_TLS_LE_LOX10 +0+9
 0+10f8 +0+48 R_SPARC_TLS_LE_HIX22 +0+1c
@@ -49,31 +49,25 @@
 0+12088 +0+4e R_SPARC_TLS_TPOFF32 +0+4
 0+12094 +0+4e R_SPARC_TLS_TPOFF32 +0+14
 0+12098 +0+4e R_SPARC_TLS_TPOFF32 +0+18
-0+1208c +0+f4e R_SPARC_TLS_TPOFF32 +0+ +sg1 \+ 0
-0+12090 +0+114e R_SPARC_TLS_TPOFF32 +0+ +sg2 \+ 0
+0+1208c +0+94e R_SPARC_TLS_TPOFF32 +0+ +sg1 \+ 0
+0+12090 +0+b4e R_SPARC_TLS_TPOFF32 +0+ +sg2 \+ 0
 
-Symbol table '.dynsym' contains 20 entries:
+Symbol table '.dynsym' contains 14 entries:
  +Num: +Value +Size Type +Bind +Vis +Ndx Name
  +0: 0+ +0 NOTYPE +LOCAL +DEFAULT +UND *
- +1: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +1 *
- +2: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +2 *
- +3: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +3 *
- +4: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +4 *
- +5: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +5 *
- +6: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +6 *
- +7: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +7 *
- +8: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +8 *
- +9: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +9 *
- +10: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +10 *
- +11: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +11 *
- +12: 0+12000 +0 OBJECT +GLOBAL DEFAULT +ABS _DYNAMIC
- +13: 0+1000 +0 FUNC +GLOBAL DEFAULT +5 fn3
- +14: 0+12080 +0 OBJECT +GLOBAL DEFAULT +ABS _PROCEDURE_LINKAGE_TABLE_
- +15: 0+ +0 NOTYPE +GLOBAL DEFAULT +UND sg1
- +16: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS __bss_start
- +17: 0+ +0 NOTYPE +GLOBAL DEFAULT +UND sg2
- +18: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS _edata
- +19: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS _end
+ +1: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +5 *
+ +2: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +6 *
+ +3: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +8 *
+ +4: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +10 *
+ +5: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +11 *
+ +6: 0+12000 +0 OBJECT +GLOBAL DEFAULT +ABS _DYNAMIC
+ +7: 0+1000 +0 FUNC +GLOBAL DEFAULT +5 fn3
+ +8: [0-9a-f]+ +0 OBJECT +GLOBAL DEFAULT +ABS _PROCEDURE_LINKAGE_TABLE_
+ +9: 0+ +0 TLS +GLOBAL DEFAULT +UND sg1
+ +10: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS __bss_start
+ +11: 0+ +0 TLS +GLOBAL DEFAULT +UND sg2
+ +12: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS _edata
+ +13: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS _end
 
 Symbol table '.symtab' contains 33 entries:
  +Num: +Value +Size Type +Bind +Vis +Ndx Name
@@ -92,21 +86,21 @@
  +12: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +12 *
  +13: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +13 *
  +14: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +14 *
- +15: 0+ +0 TLS +LOCAL +DEFAULT +7 bl1
- +16: 0+4 +0 TLS +LOCAL +DEFAULT +7 bl2
- +17: 0+8 +0 TLS +LOCAL +DEFAULT +7 bl3
- +18: 0+c +0 TLS +LOCAL +DEFAULT +7 bl4
- +19: 0+10 +0 TLS +LOCAL +DEFAULT +7 bl5
- +20: 0+1c +0 TLS +LOCAL +HIDDEN +7 sh3
- +21: 0+20 +0 TLS +LOCAL +HIDDEN +7 sh4
- +22: 0+14 +0 TLS +LOCAL +HIDDEN +7 sh1
+ +15: 0+ +0 TLS +LOCAL +DEFAULT +6 bl1
+ +16: 0+4 +0 TLS +LOCAL +DEFAULT +6 bl2
+ +17: 0+8 +0 TLS +LOCAL +DEFAULT +6 bl3
+ +18: 0+c +0 TLS +LOCAL +DEFAULT +6 bl4
+ +19: 0+10 +0 TLS +LOCAL +DEFAULT +6 bl5
+ +20: 0+1c +0 TLS +LOCAL +HIDDEN +6 sh3
+ +21: 0+20 +0 TLS +LOCAL +HIDDEN +6 sh4
+ +22: 0+14 +0 TLS +LOCAL +HIDDEN +6 sh1
  +23: 0+12080 +0 OBJECT +LOCAL +HIDDEN +ABS _GLOBAL_OFFSET_TABLE_
- +24: 0+18 +0 TLS +LOCAL +HIDDEN +7 sh2
+ +24: 0+18 +0 TLS +LOCAL +HIDDEN +6 sh2
  +25: 0+12000 +0 OBJECT +GLOBAL DEFAULT +ABS _DYNAMIC
  +26: 0+1000 +0 FUNC +GLOBAL DEFAULT +5 fn3
- +27: 0+12080 +0 OBJECT +GLOBAL DEFAULT +ABS _PROCEDURE_LINKAGE_TABLE_
- +28: 0+ +0 NOTYPE +GLOBAL DEFAULT +UND sg1
+ +27: [0-9a-f]+ +0 OBJECT +GLOBAL DEFAULT +ABS _PROCEDURE_LINKAGE_TABLE_
+ +28: 0+ +0 TLS +GLOBAL DEFAULT +UND sg1
  +29: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS __bss_start
- +30: 0+ +0 NOTYPE +GLOBAL DEFAULT +UND sg2
+ +30: 0+ +0 TLS +GLOBAL DEFAULT +UND sg2
  +31: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS _edata
  +32: [0-9a-f]+ +0 NOTYPE +GLOBAL DEFAULT +ABS _end
Index: tlssunpic32.rd
===================================================================
RCS file: /cvs/src/src/ld/testsuite/ld-sparc/tlssunpic32.rd,v
retrieving revision 1.3
diff -u -r1.3 tlssunpic32.rd
--- tlssunpic32.rd	2 Nov 2004 05:40:05 -0000	1.3
+++ tlssunpic32.rd	14 Feb 2005 00:01:16 -0000
@@ -52,21 +52,22 @@
 0+12130 +0+4e R_SPARC_TLS_TPOFF32 +0+70
 0+12138 +0+4a R_SPARC_TLS_DTPMOD32 +0+
 0+12140 +0+4e R_SPARC_TLS_TPOFF32 +0+44
-0+12120 +0+a4e R_SPARC_TLS_TPOFF32 +0+10 +sg5 \+ 0
-0+12124 +0+d4a R_SPARC_TLS_DTPMOD32 +0+ +sg1 \+ 0
-0+12128 +0+d4c R_SPARC_TLS_DTPOFF32 +0+ +sg1 \+ 0
-0+12134 +0+104e R_SPARC_TLS_TPOFF32 +0+4 +sg2 \+ 0
+0+12120 +0+b4e R_SPARC_TLS_TPOFF32 +0+10 +sg5 \+ 0
+0+12124 +0+e4a R_SPARC_TLS_DTPMOD32 +0+ +sg1 \+ 0
+0+12128 +0+e4c R_SPARC_TLS_DTPOFF32 +0+ +sg1 \+ 0
+0+12134 +0+114e R_SPARC_TLS_TPOFF32 +0+4 +sg2 \+ 0
 
 Relocation section '.rela.plt' at offset 0x[0-9a-f]+ contains 1 entries:
  Offset +Info +Type +Sym. Value +Symbol's Name \+ Addend
-0+12174 +0+c15 R_SPARC_JMP_SLOT +0+ +__tls_get_addr \+ 0
+0+12174 +0+d15 R_SPARC_JMP_SLOT +0+ +__tls_get_addr \+ 0
 
-Symbol table '.dynsym' contains 21 entries:
+Symbol table '.dynsym' contains 22 entries:
  +Num: +Value +Size Type +Bind +Vis +Ndx Name
  +0: 0+ +0 NOTYPE +LOCAL +DEFAULT +UND *
  +[0-9]+: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +6 *
  +[0-9]+: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +7 *
  +[0-9]+: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +8 *
+ +[0-9]+: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +10 *
  +[0-9]+: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +12 *
  +[0-9]+: [0-9a-f]+ +0 SECTION LOCAL +DEFAULT +13 *
  +[0-9]+: 0+1c +0 TLS +GLOBAL DEFAULT +7 sg8

  reply	other threads:[~2005-02-14  0:17 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-27 13:45 Tomer Levi
2005-01-27 14:00 ` Daniel Jacobowitz
2005-01-27 14:07   ` Christian Joensson
2005-01-27 14:12     ` Eric Botcazou
2005-01-27 14:34     ` Jakub Jelinek
2005-02-14 10:46       ` Eric Botcazou [this message]
2005-02-14 13:01         ` Eric Botcazou
2005-01-27 14:26   ` Joel Sherrill <joel@OARcorp.com>
2005-01-27 14:36     ` Hans-Peter Nilsson
2005-02-24 18:53       ` Daniel Jacobowitz
2005-02-24 19:03         ` Hans-Peter Nilsson
2005-02-24 19:09         ` Joel Sherrill <joel@OARcorp.com>
2005-02-24 20:36           ` Eric Norum
2005-01-27 14:27   ` Hans-Peter Nilsson
2005-01-27 15:12     ` Ian Lance Taylor
2005-01-27 15:56       ` Hans-Peter Nilsson
2005-01-27 16:06         ` Daniel Jacobowitz
2005-01-31  2:11           ` Hans-Peter Nilsson
2005-02-24 17:54             ` Daniel Jacobowitz
2005-01-27 18:45   ` Thorsten Glaser
2005-02-24 17:49     ` Daniel Jacobowitz
2005-02-24 18:17       ` Thorsten Glaser
2005-04-25  8:06         ` Thorsten Glaser
2005-01-27 19:09   ` The Doctor
2005-01-31 11:40     ` Nick Clifton
2005-01-31  0:25   ` Ben Elliston
2005-01-31  0:42     ` Daniel Jacobowitz
2005-01-31  1:01       ` Ben Elliston
2005-02-24 17:50         ` Daniel Jacobowitz
2005-01-31  0:30   ` Ben Elliston
2005-01-31  0:42     ` Daniel Jacobowitz
2005-01-31 17:15       ` Nick Clifton
2005-02-24 18:40         ` Daniel Jacobowitz
2005-03-08 20:53           ` Daniel Jacobowitz
2005-03-10  9:32             ` Nick Clifton
2005-03-21 11:00           ` Alan Modra
2005-03-21 11:52             ` Daniel Jacobowitz
2005-01-31 16:25   ` Nick Clifton
2005-02-14 18:07   ` Aaron W. LaFramboise
2005-02-15 17:10     ` Nick Clifton
2005-02-24 18:59   ` Daniel Jacobowitz
2005-02-14 10:47 Anil Paranjape
2005-02-14 10:50 ` Miles Bader
2005-02-14 15:08   ` Nick Clifton
2005-02-14 16:17 ` Nick Clifton
2005-02-14 19:19 Anil Paranjape
2005-02-14 23:20 ` Nick Clifton
2005-02-16 12:11 Anil Paranjape
2005-02-16 15:49 ` Nick Clifton
2005-03-01 13:42 Anil Paranjape

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200502140118.48129.ebotcazou@libertysurf.fr \
    --to=ebotcazou@libertysurf.fr \
    --cc=binutils@sources.redhat.com \
    --cc=christian.joensson@gmail.com \
    --cc=jakub@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).