<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, Apr 30, 2019 at 4:31 PM Murphy Zhou <<a href="mailto:xzhou@redhat.com">xzhou@redhat.com</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">On Tue, Apr 30, 2019 at 04:14:23PM +0800, Li Wang wrote:<br>
> On Tue, Apr 30, 2019 at 3:21 PM Murphy Zhou <<a href="mailto:xzhou@redhat.com" target="_blank">xzhou@redhat.com</a>> wrote:<br>
> <br>
> > Currently swapfiles on Overlayfs are not supported.<br>
> ><br>
> > So if we are on overlayfs and we get EINVAL from swapon() we return TCONF.<br>
> ><br>
> > Signed-off-by: Murphy Zhou <<a href="mailto:xzhou@redhat.com" target="_blank">xzhou@redhat.com</a>><br>
> > ---<br>
> >  testcases/kernel/syscalls/swapon/swapon02.c | 5 +++++<br>
> >  1 file changed, 5 insertions(+)<br>
> ><br>
> > diff --git a/testcases/kernel/syscalls/swapon/swapon02.c<br>
> > b/testcases/kernel/syscalls/swapon/swapon02.c<br>
> > index 4af5105c6..211cdfc4e 100644<br>
> > --- a/testcases/kernel/syscalls/swapon/swapon02.c<br>
> > +++ b/testcases/kernel/syscalls/swapon/swapon02.c<br>
> > @@ -86,6 +86,11 @@ static void verify_swapon(struct test_case_t *test)<br>
> >                         return;<br>
> >         }<br>
> ><br>
> > +       if (fs_type == TST_<span class="gmail_default" style="font-size:small"></span>OVERLAYFS_MAGIC && errno == EINVAL) {<br>
> > +               tst_resm(TCONF, "Swapfile on overlayfs not implemeted");<br>
> > +                       return;<br>
> > +       }<br>
> ><br>
> <br>
> The code looks correct.<br>
> <br>
> But it already has a test skipping for BTRFS, is there any possibility to<br>
> combine these filesystems check together?<br>
<br>
Good idea~  Sending V2.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">One more comment.</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">I just noticed that it also has an FS skipping list in setup(), do u think can we move BTRFS and OVERLAYFS to there?</div></div></div><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>