On Mon, Dec 3, 2018 at 4:41 PM Cyril Hrubis <chrubis@xxxxxxx> wrote:
>
> Hi!
> > +static struct tst_test test = {
> > + .needs_root = 1,
> > + .needs_tmpdir = 1,
> > + .mount_device = 1,
> > + .mntpoint = mntpoint,
> > + .setup = setup,
> > + .options = options,
> > + .test_all = test_readahead,
> > +};
>
> I do wonder if it makes sense to set the all_filesystems flag now.
>
That would be fine by me.
> Also does it make any sense to run the overlay test against all
> filesystems?
It does.
Overlayfs may behave differently on different underlying filesystems
depending on their characteristics.
> I would expect that it does not, so we will have to figure
> out how to run the tests. Maybe it would make most sense to treat the
> overlayfs as another filesystem in all_filesystem tests...
>
It does not make sense to treat overlayfs as another filesystem in
all_filesystem tests IMO because:
- "overlayfs" is not a well defined setup, you will need to define the
underlying filesystem as well, and which one will it be?
- For good test coverage it is best to test, for each fs
test over fs directly, test over overlayfs + with fs as underlying fs
Thanks,
Amir.
