<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +             return MAX(DEV_SIZE_MB_BTRFS, DEV_SIZE_MB_DEFAULT);<br>
<br>
I do not think that we should harcode this here. I would be for a more<br>
dynamic approach, i.e. add a function into the tst_supported_fs_types.c<br>
that loops over supported filesystems and chooses max over the minimal<br>
values for all supported filesystems. That way if we run on embedded<br>
targets the device size will be 16MB as long as btrfs-progs is not<br>
installed. Also that way we can easily define minimal size for xfs 300MB<br>
and things will work for embedded as long as xfs-progs are not<br>
installed.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">+1 this is a quite good suggestion that solved what we were concerned about different XFS code paths.</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>