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

Martin Doucha
2022-09-22T08:14:55Z

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.

--
Martin Doucha mdoucha@suse.cz
QA Engineer for Software Maintenance
SUSE LINUX, s.r.o.
CORSO IIa
Krizikova 148/34
186 00 Prague 8
Czech Republic