<div dir="ltr">All filesystems I'm using now support the fallocate feature, so I have not experienced slow down caused by emulation. So I do not have a strong opinion -- as I said I just had a concern. If you think posix_fallocate is the right approach, let's try it.<div><br></div><div>Please write a comment saying that we are calling posix_fallocate to get a disk full error early.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 19, 2016 at 3:08 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">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>> 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></span><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"><span class="">
<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></span>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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>
</blockquote></div><br></div>