<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 14, 2021 at 4:29 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Li,<br>
<br>
> On Tue, Dec 14, 2021 at 3:34 AM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br>
<br>
> > LTP_ALL_FILESYSTEMS_ONLY_FS_TYPE environment variable tests only that<br>
> > filesystem instead of all supported filesystems.<br>
<br>
...<br>
> > +++ b/doc/user-guide.txt<br>
...<br>
> > 'TST_NEEDS_DEVICE=1').<br>
> > +| 'LTP_ALL_FILESYSTEMS_ONLY_FS_TYPE' | Testing only specified filesystem<br>
> > instead all<br>
> > +                          supported (for tests with '.all_filesystems').<br>
<br>
> My only concern is that the variable name is too long and not easy<br>
> to remember/understand usage. Can we find a better name?<br>
>     LTP_ONE_FS_TYPE<br>
>     LTP_ASGD_FD_TYPE<br>
>     LTP_FS_TYPE_DEBUG<br>
Yes, it's too long. LTP_FS_TYPE_DEBUG sounds best to me, but no strong opinion<br>
about it.<br>
<br>
If we agree we want this, we can consider to use some switch instead or print<br>
this variable in help output (-h), with other variables from<br>
<a href="https://github.com/linux-test-project/ltp/wiki/User-Guidelines" rel="noreferrer" target="_blank">https://github.com/linux-test-project/ltp/wiki/User-Guidelines</a>.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Yes for sure, I agree to add this for debugging, since skip_filesystem is</div><div class="gmail_default" style="font-size:small">not very convenient to some degree.</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> or some else.<br>
<br>
> P.s.<br>
> And btw I often use .skip_filesystems skip other FS for debugging :).<br>
I do use it as well, but it's I wanted to have way to debug single filesystem<br>
without having to recompile (easier on embedded or non-intel platforms run via<br>
some frameworks).<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Indeed.</div><div class="gmail_default" style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Kind regards,<br>
Petr<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>