<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Aug 2, 2018 at 10:28 PM Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Aug 02, 2018 at 10:05:28PM +0300, Nir Soffer wrote:<br>
> File systems not supporting FALLOC_FL_ZERO_RANGE yet fall back to manual<br>
> zeroing.<br>
> <br>
> We can avoid this by combining two fallocate calls:<br>
> <br>
>     fallocate(FALLOC_FL_PUNHCH_HOLE)<br>
<br>
"PUNCH"<br>
<br>
>     fallocate(0)<br>
<br>
> +#ifdef FALLOC_FL_PUNCH_HOLE<br>
> +  /* If we can punch hole but may not trim, we can combine punching hole and<br>
> +      fallocate to zero a range. This is expected to be more efficient than<br>
> +      writing zeros manually. */<br>
<br>
Although we can fix this before pushing so there's no need to submit a<br>
new version, it's better if the comment style sticks to the same as<br>
used elsewhere, ie:<br>
<br>
  /* Commmnt<br>
   * More comment<br>
   */<br></blockquote><div><br></div><div>I did extra work to keep the style used in this file, would be nice to use vim default :-)</div></div></div>