<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 22, 2022 at 5:03 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
> > Although the tst_fs_type_name() functions could use some improvements,<br>
> > e.g. ext4 must be specified in skiplist as "ext2/ext3/ext4" to get properly<br>
<br>
> Yes, that's true, we need to make it keep consistent with two<br>
> skipping ways. Otherwise below test output "ext2/ext3/ext4"<br>
> looks like a bit mess to remember.<br>
<br>
> Better going with a single FS for matching (i.e. "ext4" for both fs_type<br>
> and '-d path').<br>
> But we can solve this in a separate patch later.<br>
<br>
> $ df -T . | tail -1 | awk '{print $2}'<br>
> ext4<br>
<br>
> $ ./tst_supported_fs -s "ext4" ext4<br>
> tst_supported_fs.c:135: TCONF: ext4 is skipped<br>
> $ echo $?<br>
> 32<br>
<br>
> $ ./tst_supported_fs -s "ext4" -d .<br>
> tst_supported_fs.c:137: TINFO: ext2/ext3/ext4 is not skipped<br>
> $ echo $?<br>
> 0<br>
<br>
> $ ./tst_supported_fs -s "ext2/ext3/ext4" -d .<br>
> tst_supported_fs.c:135: TCONF: ext2/ext3/ext4 is skipped<br>
> $ echo $?<br>
> 32<br>
<br>
The same problem is for .skip_filesystems on tests which does not use<br>
.all_filesystems. We only haven't noticed, because there was no reason to skip<br>
ext[234] so far. I'm looking into this.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">This seems a bit tricky to distinguish EXT2,3,4, from what I know,</div><div class="gmail_default" style="font-size:small">they use the same magic number. It will be difficult to get the</div><div class="gmail_default" style="font-size:small">FS block and extract smaller granularity of feature without</div><div class="gmail_default" style="font-size:small">using fs helper tools.</div><div class="gmail_default" style="font-size:small"><br></div></div><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>
BTW .skip_filesystems without .all_filesystems have other problems, e.g. using<br>
filesystems which aren't in fs_type_whitelist[] array (e.g. ramfs, nfs).<br>
That'd be nice to fix after the release.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">+1</div><div class="gmail_default" style="font-size:small"><br></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>