[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