linux-kernel - Re: [PATCH 0/9] overlay filesystem: request for inclusion (v17)

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]
Date:	Wed, 13 Mar 2013 16:13:05 +0100
From:	Sedat Dilek <sedat.dilek@...il.com>
To:	Miklos Szeredi <miklos@...redi.hu>
Cc:	viro@...iv.linux.org.uk, torvalds@...ux-foundation.org,
	linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org,
	hch@...radead.org, akpm@...ux-foundation.org, apw@...onical.com,
	nbd@...nwrt.org, neilb@...e.de, jordipujolp@...il.com,
	ezk@....cs.sunysb.edu, dhowells@...hat.com,
	sedat.dilek@...glemail.com, hooanon05@...oo.co.jp, mszeredi@...e.cz
Subject: Re: [PATCH 0/9] overlay filesystem: request for inclusion (v17)

On Wed, Mar 13, 2013 at 3:31 PM, Sedat Dilek <sedat.dilek@...il.com> wrote:
> On Wed, Mar 13, 2013 at 3:16 PM, Miklos Szeredi <miklos@...redi.hu> wrote:
>> Here's another version with the comments addressed plus a small bugfix and some
>> checkpatch cleanups.
>>
>> Changes in v17:
>>
>>  - fix wrong return value in a failure path in ovl_link()
>>  - fix subjects
>>  - use file_inode() and MODULE_ALIAS_FS()
>>  - fold bugfix patches
>>  - checkpatch cleanups
>>
>> Git tree is here:
>>
>>   git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs.current
>>
>
> Hi,
>
> I pulled in v17 (current) into Linux-Next (next-20130313) and built
> OverlayFS as a module.
>
> Unfortunately, I do not see any success message on loading it.
>
> --- dmesg_3.9.0-rc2-next20130313-3-iniza-small.txt      2013-03-13
> 15:21:19.578712536 +0100
> +++ dmesg_3.9.0-rc2-next20130313-3-iniza-small_after-overlayfs-test.txt
> 2013-03-13 15:22:14.658238998 +0100
> @@ -806,3 +806,8 @@
>  [   25.517154] wlan0: associated
>  [   25.517214] IPv6: ADDRCONF(NETDEV_CHANGE): wlan0: link becomes ready
>  [   54.149536] usb 2-1.5: USB disconnect, device number 3
> +[   86.502215] squashfs: version 4.0 (2009/01/31) Phillip Lougher
> +[   87.007082] EXT4-fs (loop4): mounted filesystem with ordered data
> mode. Opts: (null)
> +[   87.311998] EXT4-fs (loop4): re-mounted. Opts: data=ordered
> +[   87.657291] EXT4-fs (loop4): re-mounted. Opts: data=ordered
> +[   88.057251] EXT4-fs (loop4): re-mounted. Opts: data=ordered
>
> Highly appreciated to see such a message!
>
> Test-case script attached (needs an additional patch on top of
> overlayfs.current, see attachments).
>

Looks like this is missing (or intended?):

diff --git a/fs/overlayfs/super.c b/fs/overlayfs/super.c
index 482c26f..f23ebfc 100644
--- a/fs/overlayfs/super.c
+++ b/fs/overlayfs/super.c
@@ -684,3 +684,6 @@ static void __exit ovl_exit(void)

 module_init(ovl_init);
 module_exit(ovl_exit);
+MODULE_DESCRIPTION("overlayfs v17: provides overlay-filesystem functionality");
+MODULE_AUTHOR("Miklos Szeredi <miklos@...redi.hu>");
+MODULE_LICENSE("GPL");

Feel free to correct if I fill in bullshit...

- Sedat -

> - Sedat -
>
>> Thanks,
>> Miklos
>>
>> ---
>> Andy Whitcroft (1):
>>       overlayfs: add statfs support
>>
>> Erez Zadok (1):
>>       overlayfs: implement show_options
>>
>> Miklos Szeredi (6):
>>       vfs: add i_op->dentry_open()
>>       vfs: export do_splice_direct() to modules
>>       vfs: export __inode_permission() to modules
>>       vfs: introduce clone_private_mount()
>>       overlay filesystem
>>       fs: limit filesystem stacking depth
>>
>> Neil Brown (1):
>>       overlay: overlay filesystem documentation
>>
>> ---
>>  Documentation/filesystems/Locking       |    2 +
>>  Documentation/filesystems/overlayfs.txt |  199 +++++++++
>>  Documentation/filesystems/vfs.txt       |    7 +
>>  MAINTAINERS                             |    7 +
>>  fs/Kconfig                              |    1 +
>>  fs/Makefile                             |    1 +
>>  fs/ecryptfs/main.c                      |    7 +
>>  fs/internal.h                           |    5 -
>>  fs/namei.c                              |   10 +-
>>  fs/namespace.c                          |   18 +
>>  fs/open.c                               |   23 +-
>>  fs/overlayfs/Kconfig                    |   10 +
>>  fs/overlayfs/Makefile                   |    7 +
>>  fs/overlayfs/copy_up.c                  |  385 +++++++++++++++++
>>  fs/overlayfs/dir.c                      |  605 +++++++++++++++++++++++++++
>>  fs/overlayfs/inode.c                    |  372 +++++++++++++++++
>>  fs/overlayfs/overlayfs.h                |   70 ++++
>>  fs/overlayfs/readdir.c                  |  566 +++++++++++++++++++++++++
>>  fs/overlayfs/super.c                    |  686 +++++++++++++++++++++++++++++++
>>  fs/splice.c                             |    1 +
>>  include/linux/fs.h                      |   14 +
>>  include/linux/mount.h                   |    3 +
>>  22 files changed, 2989 insertions(+), 10 deletions(-)
>>  create mode 100644 Documentation/filesystems/overlayfs.txt
>>  create mode 100644 fs/overlayfs/Kconfig
>>  create mode 100644 fs/overlayfs/Makefile
>>  create mode 100644 fs/overlayfs/copy_up.c
>>  create mode 100644 fs/overlayfs/dir.c
>>  create mode 100644 fs/overlayfs/inode.c
>>  create mode 100644 fs/overlayfs/overlayfs.h
>>  create mode 100644 fs/overlayfs/readdir.c
>>  create mode 100644 fs/overlayfs/super.c
>>
--
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/