<div dir="ltr">I have a concern about this change when it is used on a filesystem that doesn't support file pre-allocation. The function should work on a filesystem that doesn't support an underlying fallocate(-ish) system call, but in that case I believe it tries to actually write data to force allocation. In that case it'd be slow.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 19, 2016 at 10:16 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This also makes lld marginally faster.<br>
<br>
Cheers,<br>
Rafael<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On 19 July 2016 at 10:06, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> This makes sure that space is actually available. With this change<br>
> running lld on a full file system causes it to exit with<br>
><br>
>  failed to open foo: No space left on device<br>
><br>
> instead of crashing with a sigbus.<br>
><br>
> Cheers,<br>
> Rafael<br>
</div></div></blockquote></div><br></div>