[Bug ld/20113] Partial relro support for s390x ld

[krebbel at linux dot ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=20113

Andreas Krebbel <krebbel at linux dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |krebbel at linux dot ibm.com

--- Comment #12 from Andreas Krebbel <krebbel at linux dot ibm.com> ---
I've implemented the relro support for Binutils based on the work from Marcin
Kościelnicki. Marcin unfortunately didn't finish the bounty so I had to
complete it myself. I've decided to do this only for S/390 64 bit since 32 bit
binaries do not have a bright future anyway.

Unfortunately I forgot to close this bugzilla accordingly.

I think the patch posted here is wrong. SEPARATE_GOTPLT must remain as 0 on our
platform. Please revert the patch.


Here the patches which implement partial relro for IBM Z:

commit afca762f598d453c563f244cd3777715b1a0cb72
Author: Andreas Krebbel <address@hidden>
Date:   Thu Dec 21 13:12:03 2017 +0100

    S/390: Improve partial relro support for 64 bit

commit a38137289e91fd548fc27fb6566a439233b94d65
Author: Andreas Krebbel <address@hidden>
Date:   Mon Jun 11 13:23:00 2018 +0200

    ld: Enable using separate linker script for -z relro

    With this patch dedicated linker scripts can be generated for partial
    relro triggered by defining GENERATE_RELRO_SCRIPT in the target
    specific scripts.

    This is necessary for e.g. S/390 where usually the .got.plt comes
    first and prevents the relro segment from being extended across the
    non-plt GOT entries.

    The patch started with the work from Marcin taken from the mwk user
    branches.  However, the patch needed substantial changes due to the
    'separate code' feature which got committed in the meantime.

    ld/ChangeLog:

    2018-07-18  Andreas Krebbel  <address@hidden>
            Marcin Kościelnicki <address@hidden>

-- 
You are receiving this mail because:
You are on the CC list for the bug.