<div dir="ltr">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.<div><br><div class="gmail_quote"><div dir="ltr">On Tue, May 23, 2017 at 3:55 PM Rui Ueyama via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu added a comment.<br>
<br>
I'm not convinced that we need this, so please don't submit.<br>
<br>
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.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D33251" rel="noreferrer" target="_blank">https://reviews.llvm.org/D33251</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>