[LTP] [PATCH 1/3] lib: Add proper filesystem skiplist

Cyril Hrubis chrubis@suse.cz
Wed Mar 10 15:28:47 CET 2021


Hi!
> > -static int has_kernel_support(const char *fs_type, int flags)
> > +int tst_fs_in_skiplist(const char *fs_type, const char *const *skiplist)
> > +{
> > +	unsigned int i;
> > +
> > +	if (!skiplist)
> > +		return 0;
> > +
> > +	for (i = 0; skiplist[i]; i++) {
> > +		if (!strcmp(fs_type, skiplist[i])) {
> > +			tst_res(TINFO,
> It'd be nice if skipping message could be TCONF to avoid:
> tst_supported_fs_types.c:57: TINFO: Skipping tmpfs as requested by the test
> tst_test.c:1056: TCONF: Skipping due to unsupported filesystem
> 
> but understand why - it'd make code more complicated (has_kernel_support() which
> is used in tst_fs_is_supported() should not emit TCONF).

I can make a internal wrapper for the tst_fs_is_supported() that prints
the TINFO message and use that in the tst_supported_fs_types, which
would make it silent in the case it's called from the test library...

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list