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

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

 



On Wed, Mar 13, 2013 at 3:31 PM, Sedat Dilek <sedat.dilek@xxxxxxxxx> wrote:
> On Wed, Mar 13, 2013 at 3:16 PM, Miklos Szeredi <miklos@xxxxxxxxxx> 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@xxxxxxxxxx>");
+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-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux