[PATCH] D33251: [lld][ELF]Add option to make .dynamic read only

Petr Hosek via llvm-commits llvm-commits at lists.llvm.org
Tue May 23 16:17:39 PDT 2017


This option is not specific to vDSO, it's what we want to use as a default
in Fuchsia and I plan to change our Clang driver to always set this flag.
In case of vDSO, we want zero writability, irrespective of which loader is
being used to load that ELF file.

On Tue, May 23, 2017 at 3:55 PM Rui Ueyama via Phabricator <
reviews at reviews.llvm.org> wrote:

> ruiu added a comment.
>
> I'm not convinced that we need this, so please don't submit.
>
> If this is for your OS's vDSO which needs to be read by an in-kernel ELF
> loader which doesn't support the entire ELF spec, this patch is to work
> around your loader's limitation. If you don't want to implement a complete
> ELF loader, why do you want to "fix" the problem in the right way? It is
> odd that you do hack at one place and do the right thing at the other
> place. Your loader can ignore RW bit and map .dynamic sections to read-only
> segments.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D33251
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170523/98be30ae/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4843 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170523/98be30ae/attachment.bin>


More information about the llvm-commits mailing list