* [PATCH] Fix Scrt1.o on s390{,x}
@ 2003-07-04 19:22 Jakub Jelinek
2003-07-08 3:45 ` Ulrich Drepper
0 siblings, 1 reply; 2+ messages in thread
From: Jakub Jelinek @ 2003-07-04 19:22 UTC (permalink / raw)
To: Ulrich Drepper, Roland McGrath; +Cc: Glibc hackers
Hi!
On s390 without this patch PIEs were be just DT_TEXTREL, on s390x
crashed because __libc_start_main was usually not close enough to
_start.
2003-07-04 Jakub Jelinek <jakub@redhat.com>
* sysdeps/s390/s390-32/elf/start.S: Emit position independent code
if PIC.
* sysdeps/s390/s390-64/elf/start.S: Likewise.
--- libc/sysdeps/s390/s390-32/elf/start.S.jj 2002-12-09 15:37:23.000000000 -0500
+++ libc/sysdeps/s390/s390-32/elf/start.S 2003-07-04 15:07:32.000000000 -0400
@@ -1,5 +1,5 @@
/* Startup code compliant to the ELF s390 ABI.
- Copyright (C) 2000, 2001, 2002 Free Software Foundation, Inc.
+ Copyright (C) 2000, 2001, 2002, 2003 Free Software Foundation, Inc.
Contributed by Martin Schwidefsky (schwidefsky@de.ibm.com).
This file is part of the GNU C Library.
@@ -66,19 +66,35 @@ _start:
l %r6,.L2-.Llit(%r13) # load pointer to __libc_csu_fini
l %r5,.L1-.Llit(%r13) # load pointer to __libc_csu_init
l %r2,.L3-.Llit(%r13) # load pointer to main
+ l %r1,.L4-.Llit(%r13) # load pointer to __libc_start_main
+#ifdef PIC
+ l %r12,.L5-.Llit(%r13) # load .got pointer
+ la %r6,0(%r13,%r6)
+ la %r5,0(%r13,%r5)
+ la %r2,0(%r13,%r2)
+ la %r1,0(%r13,%r1)
+ la %r12,0(%r13,%r12)
+#endif
/* ok, now branch to the libc main routine */
- l %r1,.L4-.Llit(%r13)
basr %r14,%r1
/* crash if __libc_start_main returns */
.word 0
.Llit:
+#ifndef PIC
.L1: .long __libc_csu_init
.L2: .long __libc_csu_fini
.L3: .long main
.L4: .long __libc_start_main
+#else
+.L1: .long __libc_csu_init-.Llit
+.L2: .long __libc_csu_fini-.Llit
+.L3: .long main-.Llit
+.L4: .long __libc_start_main@plt-.Llit
+.L5: .long _GLOBAL_OFFSET_TABLE_-.Llit
+#endif
/* FIXME: FPU flags or what ?!? */
--- libc/sysdeps/s390/s390-64/elf/start.S.jj 2002-12-09 15:37:20.000000000 -0500
+++ libc/sysdeps/s390/s390-64/elf/start.S 2003-07-04 15:07:17.000000000 -0400
@@ -1,5 +1,5 @@
/* Startup code compliant to the 64 bit S/390 ELF ABI.
- Copyright (C) 2001, 2002 Free Software Foundation, Inc.
+ Copyright (C) 2001, 2002, 2003 Free Software Foundation, Inc.
Contributed by Martin Schwidefsky (schwidefsky@de.ibm.com).
This file is part of the GNU C Library.
@@ -64,7 +64,11 @@ _start:
larl %r2,main # load pointer to main
/* Ok, now branch to the libc main routine. */
+#ifdef PIC
+ brasl %r14,__libc_start_main@plt
+#else
brasl %r14,__libc_start_main
+#endif
/* Crash if __libc_start_main returns. */
.word 0
Jakub
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix Scrt1.o on s390{,x}
2003-07-04 19:22 [PATCH] Fix Scrt1.o on s390{,x} Jakub Jelinek
@ 2003-07-08 3:45 ` Ulrich Drepper
0 siblings, 0 replies; 2+ messages in thread
From: Ulrich Drepper @ 2003-07-08 3:45 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: Roland McGrath, Glibc hackers
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Jakub Jelinek wrote:
> 2003-07-04 Jakub Jelinek <jakub@redhat.com>
>
> * sysdeps/s390/s390-32/elf/start.S: Emit position independent code
> if PIC.
> * sysdeps/s390/s390-64/elf/start.S: Likewise.
Applied.
- --
- --------------. ,-. 444 Castro Street
Ulrich Drepper \ ,-----------------' \ Mountain View, CA 94041 USA
Red Hat `--' drepper at redhat.com `---------------------------
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)
iD8DBQE/Cj5W2ijCOnn/RHQRAji8AJ0VNa4Gd/tAgVV4tEHQ65nuoMrK5gCgkSEO
oULg16PsP1geTLikuwSdd40=
=Eo4j
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2003-07-08 3:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-07-04 19:22 [PATCH] Fix Scrt1.o on s390{,x} Jakub Jelinek
2003-07-08 3:45 ` Ulrich Drepper
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).