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

[Li Wang]

On Tue, Dec 14, 2021 at 4:29 PM Petr Vorel <pvorel@suse.cz> wrote:

> 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.
>

Yes for sure, I agree to add this for debugging, since skip_filesystem is
not very convenient to some degree.



>
> > 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).
>

Indeed.

>
> Kind regards,
> Petr
>
>

-- 
Regards,
Li Wang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20211214/c034f96f/attachment-0001.htm>