<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 19, 2016 at 1:06 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> Well, is detecting a disk full event so important to risk that it could slow<br>
> down the linker on NFS or such? It'd depend how often you hit it, but to me<br>
> it doesn't feel like a good trade-off.<br>
<br>
</span>I would probably say it is. If you are linking on nfs you probably<br>
cannot expect any reasonable link time right now.<br>
<br>
If we want to support that we should resurrect my patch to link to a<br>
posix_align allocated anonymous memory and then just use write to pass<br>
it to the os.<br></blockquote><div><br></div><div>Yeah, this sounds like the right approach. AFAIK nobody has tested LLD on NFS and so any behaviors/perf we have there are going to be purely accidental. Also IIRC mmap on NFS has issues? And we rely on mmap heavily on the input side too.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
In any case, lets benchmark it before we start adding more #ifdefs and options.<br>
<br>
The attached patch uses a cmake check to avoid making things worse<br>
(but not any better) on systems that don't have posix_fallocate.<br>
<br>
What do you want me to benchmark? Linking the current benchmarks on<br>
NFS? Is linux as the server OK?<br>
<br>
Cheers,<br>
Rafael<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>