linux-kernel - Re: [PULL for 3.18] overlay filesystem v24

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJfpeguWkWOeWYS1o2gW4P1DJhXn=enBGukPcfscd-guas7h8w@mail.gmail.com>
Date:	Tue, 30 Sep 2014 09:31:24 +0200
From:	Miklos Szeredi <miklos@...redi.hu>
To:	"J. R. Okajima" <hooanon05g@...il.com>
Cc:	David Howells <dhowells@...hat.com>,
	Al Viro <viro@...iv.linux.org.uk>,
	Christoph Hellwig <hch@...radead.org>,
	Linux-Fsdevel <linux-fsdevel@...r.kernel.org>,
	Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Stephen Rothwell <sfr@...b.auug.org.au>,
	linux-unionfs@...r.kernel.org
Subject: Re: [PULL for 3.18] overlay filesystem v24

On Tue, Sep 30, 2014 at 6:54 AM, J. R. Okajima <hooanon05g@...il.com> wrote:
>
> David Howells:
>> Miklos Szeredi <miklos@...redi.hu> wrote:
>>
>> > I'd like to propose overlayfs for inclusion into 3.18.
>> >
>> > Al, would you mind giving it a review?
>> >
>> > Git tree is here:
>> >
>> >   git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs.current
>>
>> Tested-by: David Howells <dhowells@...hat.com>
>
> Does it mean overlayfs passed all your unionmount-testsuite? And does
> the test suite contain tests for "inode-based" union? For example,
> - read(2) may get the obsoleted filedata (fstat(2) for metadata too).
> - fcntl(F_SETLK) may be broken by copy-up.
> - inotify may not work when it refers to the file before being
>   copied-up.
> - unnecessary copy-up may happen, for example mmap(MAP_PRIVATE) after
>   open(O_RDWR).
> - exporting via NFS and fhandle systemcalls will not work.

Most of this is explicitly documented.

>
> A few releases ago, OFD file-lock was introduced to improve the
> behaviour of POSIX lock. POSIX lock has made users confused and I am
> afraid that the similar story will come up because of the "name-based"
> union behaviour. Of course the story is not limited to the file-lock.
>
> If I remember correctly, are you the one who consitunes the development
> of UnionMount? Is the development totally stopped?
> Next paragraph is what I wrote several times.
>         AUFS is an "inode-based" stackable filesystem and solved them many years
>         ago. But I have to admit that AUFS is big. Yes it is grown up.
>         I don't stop including overlayfs into mainline, but if the development
>         of UnionMount is really stopped, then I'd ask people to consider merging
>         aufs as well as overlayfs.

Union-mounts are namespace based as well.  There's little semantic
difference between overlayfs and union-mounts.  The difference is
mostly in the implementation.

It would be good to have an inode based union as well.  I suggest (as
I suggested many times) to try slimming aufs to a bare minimum and
submit that.  It may be easier to just start from scratch instead
trying to drop features from the existing codebase.   I'd be happy to
review and generally help with such an effort.

Thanks,
Miklos
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/