Re: [PATCH 3/3] ovl: redirect on rename-dir

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 26, 2016 at 2:26 PM, Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
>

...

>
> BTW I don't remember having tested this, so it might possibly be
> buggy.  Automatic multi-layer testing would really be good.  What we
> basically need is:
>
>  - create normal (two layer) overlay (with interesting constructs,
> whiteout, opaque dir, redirect)
>  - umount
>  - create three layer overlay where the two lower layers come from the
> previous upper/lower layers
>  - do more interesting things
>
> There's one such test in xfstests but it would be good to have more.
>

I have a POC for generic addition of layers for unionmount-testsuite
I pushed it to my github:
https://github.com/amir73il/unionmount-testsuite.git #ovl_rename_dir

remount(ctx) rotates upper to top of lower layers and adds a new upper layer
For now, remount(ctx) is called after successful mkdir and rename dir to
catch directory redirect bugs.

As it happens, it is very easy to find a bug, because it is enough to
just umount an mount the overlay without even adding a new layer
after directory rename to corrupt the overlayfs lookup.

Almost every rename-*-dir test demonstrates the problem

Going forward, I intend to hook the remount(ctx) event
in a cleaner way, so for every VFS op in every subtest,
it is possible to configure a test where the operation happens:
- without any remount
- with umount + mount
- with umount + mount with a new layer

Cheers,
Amir.
--
To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Devel]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux