[LTP] [PATCH] tst_supported_fs: Implement checking paths against skiplist

Petr Vorel pvorel@suse.cz
Thu Sep 22 10:43:33 CEST 2022


> On 22. 09. 22 6:16, Li Wang wrote:
> >                      if (skiplist) {
> >     -                       if (tst_fs_in_skiplist(argv[optind], (const char * const*)skiplist))
> >     -                               tst_brk(TCONF, "%s is skipped", argv[optind]);
> >     +                       if (tst_fs_in_skiplist(fsname, (const char * const*)skiplist))
> >     +                               tst_brk(TCONF, "%s is skipped", fsname);


> > TCONF does not means return 1, we might need explicitly 'return 1' here.

> Yes, but that's a problem which already existed before my patch and affects
> the other single-FS checks as well. The fix should go into a separate patch
> after release.

Yes, that's not related. I changed it in eb47b4497 ("tst_supported_fs: Support
skip list when query single fs") as with 32 one can detect it's TCONF.
As long as 0 is success, non-zero for failure.
But I forget to update doc.

Thus I'll wait for feedback from others whether we should get back to 1 as error
(thus use TINFO) or update doc to mention to exit 32 (or more generally non-zero
exit).
very nit: I'd use exit instead of return in docs.

Kind regards,
Petr


More information about the ltp mailing list