On Wed, Dec 15, 2021 at 12:46 AM Petr Vorel <pvorel@suse.cz> wrote:
Hi Cyril, Li,

> Hi!
> > +| 'LTP_SINGLE_FS_TYPE'  | Testing only specified filesystem instead all
>                                          ^   ^
>                                        |   "specifies"
>                                        there should be comma or dash here

I meant "specific", i.e. Testing only specific filesystem instead all supported,

+1

 
but "Testing only - specifies filesystem instead all supported" makes more
sense.

Hmm, I think it also makes sense to people who care about the
only one filesystem on their product. So this should _NOT_ be limit
for test/debug, because it can help to reduce testing time for specific 
requirements. 

Let's just provide a variable and leave the usage to users:).

 

> Other than that the rest looks fine, for the patchset:
> Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

Thanks! Waiting for Li and others for input before merging it.

Good work.
Reviewed-by: Li Wang <liwang@redhat.com>


--
Regards,
Li Wang