<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 Wed, Sep 21, 2022 at 6:27 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">Filesystem detection of locally used filesystem was broken on tests<br>
which did not use TST_ALL_FILESYSTEMS as it 1) expected used filesystem<br>
is $TST_FS_TYPE 2) this variable was not yet set.<br>
<br>
Also this check makes sense only if test defines TST_SKIP_FILESYSTEMS<br>
(to align with the condition in do_test_setup() in C API).<br>
<br>
Move filesystem check after (optional) cd "$TST_TMPDIR" (TMPDIR can have<br>
different filesystem).<br>
<br>
Not printing extra TINFO "$_tst_fs is supported by the test" (which is<br>
printed in C API) when test is supported, because there is already<br>
similar TINFO from testcases/lib/tst_supported_fs.c:<br>
tst_supported_fs.c:104: TINFO: btrfs is not skipped<br>
<br>
Fixes: 1f6bd6e66 ("tst_test.sh: Add $TST_ALL_FILESYSTEMS")<br>
<br>
Reported-by: Martin Doucha <<a href="mailto:mdoucha@suse.cz" target="_blank">mdoucha@suse.cz</a>><br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
First, sorry for introducing a regression.<br>
<br>
I don't like df and tail dependency. Also df output is stable (even on<br>
busybox), but I'd prefer to use code from do_test_setup() in lib/tst_test.c:<br>
<br>
        long fs_type = tst_fs_type(".");<br>
        const char *fs_name = tst_fs_type_name(fs_type);<br>
<br>
Instead adding yet another binary, I wonder if we could add extra getopt<br>
parameter to ask for current filesystem.<br>
<br>
i.e. to replace:<br>
-tst_supported_fs -s skip_list fs_type<br>
+tst_supported_fs -s skip_list -d DIR<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">+1, and Martin already sent out the achievement patch,</div><div class="gmail_default" style="font-size:small">you can rebase this patch on that.</div><br></div><div><div class="gmail_default" style="font-size:small">And I'd like to vote for merging your both patches for</div><div class="gmail_default" style="font-size:small">adding to the new release.</div></div><div><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>