Re: [LTP] [PATCH 1/1] lib: Add support for debugging .all_filesystems

[Petr Vorel <pvorel@suse.cz>]

Hi Li,

> On Tue, Dec 14, 2021 at 3:34 AM Petr Vorel <pvorel@suse.cz> wrote:

> > LTP_ALL_FILESYSTEMS_ONLY_FS_TYPE environment variable tests only that
> > filesystem instead of all supported filesystems.

...
> > +++ b/doc/user-guide.txt
...
> > 'TST_NEEDS_DEVICE=1').
> > +| 'LTP_ALL_FILESYSTEMS_ONLY_FS_TYPE' | Testing only specified filesystem
> > instead all
> > +                          supported (for tests with '.all_filesystems').

> My only concern is that the variable name is too long and not easy
> to remember/understand usage. Can we find a better name?
>     LTP_ONE_FS_TYPE
>     LTP_ASGD_FD_TYPE
>     LTP_FS_TYPE_DEBUG
Yes, it's too long. LTP_FS_TYPE_DEBUG sounds best to me, but no strong opinion
about it.

If we agree we want this, we can consider to use some switch instead or print
this variable in help output (-h), with other variables from
https://github.com/linux-test-project/ltp/wiki/User-Guidelines.

> or some else.

> P.s.
> And btw I often use .skip_filesystems skip other FS for debugging :).
I do use it as well, but it's I wanted to have way to debug single filesystem
without having to recompile (easier on embedded or non-intel platforms run via
some frameworks).

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp