<div dir="ltr">Roland,<div><br></div><div>What I was trying to do is to understand it, and I'm not pushing it back blindly. I was posing questions to understand the proposal.</div><div><br></div><div>The original proposal didn't say much about the reason why we want a new flag. The first reason I was told was memory saving, which is dubious as we discussed here. And then vDSO's story came in. And then it is now clear that you guys just want to make all .dynamic read-only in the ABI of the new OS.</div><div><br></div><div>So the true reason why you want to add `-z rodynamic` is because in the Fuchsia ABI, .dynamic is read-only, and we want to have some way to make it actually so in ELF files. Probably virtually no memory savings, no security improvement, but this should be done because that's the right thing to do. If that's the reason, I think I'm OK with the new flag.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 25, 2017 at 2:12 PM, Roland McGrath <span dir="ltr"><<a href="mailto:mcgrathr@google.com" target="_blank">mcgrathr@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Look, we have a new system, and we're defining what the ABI will be<br>
for our system.<br>
We've decided we want .dynamic to be read-only and we are confident in<br>
that choice.<br>
It's OK if you aren't convinced that's the decision you would make in<br>
your ABI, but it's the decision we've made for our ABI.<br>
<br>
I don't see why you feel the need to push back on a new platform's ABI<br>
decisions when we've done the implementation<br>
work for LLD to support the ABI and the actual impact on the LLD code<br>
base is tiny.<br>
<br>
Since LLD doesn't support selecting its behavior based on target<br>
triple, we've added a switch.<br>
If you'd prefer this switch were spelled --target=fuchsia rather than<br>
-z rodynamic, that's fine with us.<br>
<br>
<br>
Thanks,<br>
Roland<br>
</blockquote></div><br></div>