linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] overlay filesystem fixes for 3.18
@ 2014-11-20 16:45 Miklos Szeredi
  2014-11-20 21:00 ` Al Viro
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Miklos Szeredi @ 2014-11-20 16:45 UTC (permalink / raw)
  To: Al Viro; +Cc: Linus Torvalds, linux-fsdevel, linux-kernel, linux-unionfs

Hi Al,

Please pull from

  git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs-current

The biggest change is to rename the filesystem from "overlayfs" to "overlay".
This will allow legacy overlayfs to be easily carried by distros alongside the
new mainline one.  Also fix a couple of copy-up races and allow escaping comma
character in filenames.

Thanks,
Miklos

---
Miklos Szeredi (8):
      ovl: rename filesystem type to "overlay"
      ovl: fix remove/copy-up race
      ovl: fix race in private xattr checks
      ovl: allow filenames with comma
      ovl: use lockless_dereference() for upperdentry
      ovl: pass dentry into ovl_dir_read_merged()
      ovl: update MAINTAINERS
      ovl: ovl_dir_fsync() cleanup

---
 Documentation/filesystems/overlayfs.txt |  2 +-
 MAINTAINERS                             |  7 ++--
 fs/Makefile                             |  2 +-
 fs/overlayfs/Kconfig                    |  2 +-
 fs/overlayfs/Makefile                   |  4 +--
 fs/overlayfs/dir.c                      | 31 ++++++++++-------
 fs/overlayfs/inode.c                    | 27 ++++++++++-----
 fs/overlayfs/readdir.c                  | 39 +++++++++------------
 fs/overlayfs/super.c                    | 61 ++++++++++++++++++++++++++-------
 9 files changed, 111 insertions(+), 64 deletions(-)

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] overlay filesystem fixes for 3.18
  2014-11-20 16:45 [GIT PULL] overlay filesystem fixes for 3.18 Miklos Szeredi
@ 2014-11-20 21:00 ` Al Viro
  2014-11-21 10:43 ` Sedat Dilek
  2014-11-21 11:32 ` Hans-Peter Jansen
  2 siblings, 0 replies; 8+ messages in thread
From: Al Viro @ 2014-11-20 21:00 UTC (permalink / raw)
  To: Miklos Szeredi; +Cc: Linus Torvalds, linux-fsdevel, linux-kernel, linux-unionfs

On Thu, Nov 20, 2014 at 05:45:31PM +0100, Miklos Szeredi wrote:
> Hi Al,
> 
> Please pull from
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs-current
> 
> The biggest change is to rename the filesystem from "overlayfs" to "overlay".
> This will allow legacy overlayfs to be easily carried by distros alongside the
> new mainline one.  Also fix a couple of copy-up races and allow escaping comma
> character in filenames.

Grabbed, will push after I get a bit of sleep (and the same regarding the
review and comments on the other branch).

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] overlay filesystem fixes for 3.18
  2014-11-20 16:45 [GIT PULL] overlay filesystem fixes for 3.18 Miklos Szeredi
  2014-11-20 21:00 ` Al Viro
@ 2014-11-21 10:43 ` Sedat Dilek
  2014-11-21 10:50   ` Miklos Szeredi
  2014-11-21 11:32 ` Hans-Peter Jansen
  2 siblings, 1 reply; 8+ messages in thread
From: Sedat Dilek @ 2014-11-21 10:43 UTC (permalink / raw)
  To: Miklos Szeredi
  Cc: Al Viro, Linus Torvalds, linux-fsdevel, LKML, linux-unionfs

On Thu, Nov 20, 2014 at 5:45 PM, Miklos Szeredi <miklos@szeredi.hu> wrote:
> Hi Al,
>
> Please pull from
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs-current
>

This seems to be the place where overlayfs-fixes are collected for a
git-pull request.
Can you add a "T:" line with the Git tree into MAINTAINERS file if
this is the case?
If not please let the people know about current situation.
Thanks.

- Sedat -

> The biggest change is to rename the filesystem from "overlayfs" to "overlay".
> This will allow legacy overlayfs to be easily carried by distros alongside the
> new mainline one.  Also fix a couple of copy-up races and allow escaping comma
> character in filenames.
>
> Thanks,
> Miklos
>
> ---
> Miklos Szeredi (8):
>       ovl: rename filesystem type to "overlay"
>       ovl: fix remove/copy-up race
>       ovl: fix race in private xattr checks
>       ovl: allow filenames with comma
>       ovl: use lockless_dereference() for upperdentry
>       ovl: pass dentry into ovl_dir_read_merged()
>       ovl: update MAINTAINERS
>       ovl: ovl_dir_fsync() cleanup
>
> ---
>  Documentation/filesystems/overlayfs.txt |  2 +-
>  MAINTAINERS                             |  7 ++--
>  fs/Makefile                             |  2 +-
>  fs/overlayfs/Kconfig                    |  2 +-
>  fs/overlayfs/Makefile                   |  4 +--
>  fs/overlayfs/dir.c                      | 31 ++++++++++-------
>  fs/overlayfs/inode.c                    | 27 ++++++++++-----
>  fs/overlayfs/readdir.c                  | 39 +++++++++------------
>  fs/overlayfs/super.c                    | 61 ++++++++++++++++++++++++++-------
>  9 files changed, 111 insertions(+), 64 deletions(-)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] overlay filesystem fixes for 3.18
  2014-11-21 10:43 ` Sedat Dilek
@ 2014-11-21 10:50   ` Miklos Szeredi
  2014-11-22  8:40     ` Sedat Dilek
  0 siblings, 1 reply; 8+ messages in thread
From: Miklos Szeredi @ 2014-11-21 10:50 UTC (permalink / raw)
  To: Sedat Dilek; +Cc: Al Viro, Linus Torvalds, linux-fsdevel, LKML, linux-unionfs

On Fri, Nov 21, 2014 at 11:43 AM, Sedat Dilek <sedat.dilek@gmail.com> wrote:
>>   git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs-current
>>
>
> This seems to be the place where overlayfs-fixes are collected for a
> git-pull request.
> Can you add a "T:" line with the Git tree into MAINTAINERS file if
> this is the case?

Done that already.  See "ovl: update MAINTAINERS" patch in that series.

Thanks,
Miklos

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] overlay filesystem fixes for 3.18
  2014-11-20 16:45 [GIT PULL] overlay filesystem fixes for 3.18 Miklos Szeredi
  2014-11-20 21:00 ` Al Viro
  2014-11-21 10:43 ` Sedat Dilek
@ 2014-11-21 11:32 ` Hans-Peter Jansen
  2014-11-21 12:41   ` Miklos Szeredi
  2 siblings, 1 reply; 8+ messages in thread
From: Hans-Peter Jansen @ 2014-11-21 11:32 UTC (permalink / raw)
  To: Miklos Szeredi
  Cc: Al Viro, Linus Torvalds, linux-fsdevel, linux-kernel, linux-unionfs

Dear Miklos,

On Donnerstag, 20. November 2014 17:45:31 Miklos Szeredi wrote:
> 
> The biggest change is to rename the filesystem from "overlayfs" to
> "overlay". This will allow legacy overlayfs to be easily carried by distros
> alongside the new mainline one.

Would you kindly give a firm elaboration of this rename? 

Why would a distro kernel maintainer want to keep the external overlayfs 
patches together with the now in kernel module?

Isn't it enough to drop the external patches from distro kernels in order to 
use it successfully or are there any semantic changes hidden that an admin 
should know about?

BTW: congrats to all involved parties. A layered filesystem finally arrived in 
the Linux kernel! This is a quantum leap for creative system architects.

Cheers,
Pete

proud user of aufs (and formerly unionfs) based fat diskless systems since 
about a decade.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] overlay filesystem fixes for 3.18
  2014-11-21 11:32 ` Hans-Peter Jansen
@ 2014-11-21 12:41   ` Miklos Szeredi
  2014-11-21 14:51     ` Hans-Peter Jansen
  0 siblings, 1 reply; 8+ messages in thread
From: Miklos Szeredi @ 2014-11-21 12:41 UTC (permalink / raw)
  To: Hans-Peter Jansen
  Cc: Al Viro, Linus Torvalds, Linux-Fsdevel, Kernel Mailing List,
	linux-unionfs

On Fri, Nov 21, 2014 at 12:32 PM, Hans-Peter Jansen <hpj@urpla.net> wrote:
> Dear Miklos,
>
> On Donnerstag, 20. November 2014 17:45:31 Miklos Szeredi wrote:
>>
>> The biggest change is to rename the filesystem from "overlayfs" to
>> "overlay". This will allow legacy overlayfs to be easily carried by distros
>> alongside the new mainline one.
>
> Would you kindly give a firm elaboration of this rename?
>
> Why would a distro kernel maintainer want to keep the external overlayfs
> patches together with the now in kernel module?
>
> Isn't it enough to drop the external patches from distro kernels in order to
> use it successfully or are there any semantic changes hidden that an admin
> should know about?

See this thread:

https://lkml.org/lkml/2014/11/18/433

Thanks,
Miklos

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] overlay filesystem fixes for 3.18
  2014-11-21 12:41   ` Miklos Szeredi
@ 2014-11-21 14:51     ` Hans-Peter Jansen
  0 siblings, 0 replies; 8+ messages in thread
From: Hans-Peter Jansen @ 2014-11-21 14:51 UTC (permalink / raw)
  To: Miklos Szeredi
  Cc: Al Viro, Linus Torvalds, Linux-Fsdevel, Kernel Mailing List,
	linux-unionfs

On Freitag, 21. November 2014 13:41:57 Miklos Szeredi wrote:
> On Fri, Nov 21, 2014 at 12:32 PM, Hans-Peter Jansen <hpj@urpla.net> wrote:
> > Dear Miklos,
> > 
> > On Donnerstag, 20. November 2014 17:45:31 Miklos Szeredi wrote:
> >> The biggest change is to rename the filesystem from "overlayfs" to
> >> "overlay". This will allow legacy overlayfs to be easily carried by
> >> distros
> >> alongside the new mainline one.
> > 
> > Would you kindly give a firm elaboration of this rename?
> > 
> 
> See this thread:
> 
> https://lkml.org/lkml/2014/11/18/433

Okay, thanks. I  must have missed this one. "overlay" is as good as a name can 
be, IMHO.

Cheers,
Pete

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [GIT PULL] overlay filesystem fixes for 3.18
  2014-11-21 10:50   ` Miklos Szeredi
@ 2014-11-22  8:40     ` Sedat Dilek
  0 siblings, 0 replies; 8+ messages in thread
From: Sedat Dilek @ 2014-11-22  8:40 UTC (permalink / raw)
  To: Miklos Szeredi
  Cc: Al Viro, Linus Torvalds, linux-fsdevel, LKML, linux-unionfs

On Fri, Nov 21, 2014 at 11:50 AM, Miklos Szeredi <miklos@szeredi.hu> wrote:
> On Fri, Nov 21, 2014 at 11:43 AM, Sedat Dilek <sedat.dilek@gmail.com> wrote:
>>>   git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs-current
>>>
>>
>> This seems to be the place where overlayfs-fixes are collected for a
>> git-pull request.
>> Can you add a "T:" line with the Git tree into MAINTAINERS file if
>> this is the case?
>
> Done that already.  See "ovl: update MAINTAINERS" patch in that series.
>

I asked for that a while ago.
"Done" is good to hear.

What about the other stuff (see my questions in [1])?

Thanks.

- Sedat -

[1] http://marc.info/?l=linux-kernel&m=141406538905679&w=1

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2014-11-22  8:40 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-20 16:45 [GIT PULL] overlay filesystem fixes for 3.18 Miklos Szeredi
2014-11-20 21:00 ` Al Viro
2014-11-21 10:43 ` Sedat Dilek
2014-11-21 10:50   ` Miklos Szeredi
2014-11-22  8:40     ` Sedat Dilek
2014-11-21 11:32 ` Hans-Peter Jansen
2014-11-21 12:41   ` Miklos Szeredi
2014-11-21 14:51     ` Hans-Peter Jansen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).