<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,sans-serif">On Tue, Mar 29, 2016 at 6:17 PM, Eric Sandeen </span><span dir="ltr" style="font-family:arial,sans-serif"><<a href="mailto:sandeen@sandeen.net" target="_blank">sandeen@sandeen.net</a>></span><span style="font-family:arial,sans-serif"> wrote:</span><br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 3/29/16 11:11 AM, Jan Tulak wrote:<br>
> On Thu, Mar 24, 2016 at 5:33 PM, Eric Sandeen <<a href="mailto:sandeen@sandeen.net">sandeen@sandeen.net</a> <mailto:<a href="mailto:sandeen@sandeen.net">sandeen@sandeen.net</a>>>wrote:<br>
><br>
><br>
><br>
>     On 3/24/16 6:15 AM, <a href="mailto:jtulak@redhat.com">jtulak@redhat.com</a> <mailto:<a href="mailto:jtulak@redhat.com">jtulak@redhat.com</a>> wrote:<br>
>     > From: Dave Chinner <<a href="mailto:dchinner@redhat.com">dchinner@redhat.com</a> <mailto:<a href="mailto:dchinner@redhat.com">dchinner@redhat.com</a>>><br>
<span class="">>     ><br>
>     > Because passing "-n ftype=2" should fail.<br>
><br>
>     but passing crc=1 ftype=1 shouldn't fail, should it?<br>
>     Seems like it will here.<br>
><br>
><br>
> ​From man page:<br>
> When  CRCs are enabled via -m crc=1, the ftype functionality is always enabled. This feature can not be<br>
> ​<br>
> turned off for such filesystem configurations.​<br>
><br>
><br>
> ​So I think it should not be possible to enter both crc and ftype at<br>
> the same time - which is the current behaviour. It feels strange a<br>
> bit to allow ftype=1 (which does nothing with crc=1), but fail on<br>
> ftype=0​<br>
<br>
</span>My point is that -m crc=1 -d ftype=1 simply restates the defaults.<br>
Why should that combination fail?<br>
<br>
And -m crc=0 -d ftype=0 is also perfectly acceptable.<br>
<br>
In fact, -m crc=1 -d ftype=0 is the only one of the 4 combinations<br>
which is not ok, but AFAICT your patch fails the other 3 as well.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​I removed the entire crc check - there is another one later in the code, so this was 1) redundant, 2) wrong, because it depended on whether -m crc=0 was before or after -n ftype (if after, default crc value was used for the check).</div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline"><br></div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">However, it causes lots of conflicts on the following patches, so I will send it later, once there are more changes...</div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline"><br></div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">Cheers, and thanks for catching it.</div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline"><br></div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">Jan</div></div><div><br></div></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Jan Tulak<br></div><a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a> / <a href="mailto:jan@tulak.me" target="_blank">jan@tulak.me</a></div></div></div></div>
</div></div>