All of lore.kernel.org
 help / color / mirror / Atom feed
From: Al Viro <viro@ZenIV.linux.org.uk>
To: Ian Kent <raven@themaw.net>
Cc: Valerie Aurora <vaurora@redhat.com>,
	linux-fsdevel@vger.kernel.org, Jan Blunck <jblunck@suse.de>,
	autofs@linux.kernel.org, sfrench@samba.org,
	Trond.Myklebust@netapp.com, David Howells <dhowells@redhat.com>
Subject: Re: [PATCH 1/7] autofs4: Save autofs trigger's vfsmount in super block info
Date: Sun, 17 Jan 2010 17:57:23 +0000	[thread overview]
Message-ID: <20100117175723.GP19799@ZenIV.linux.org.uk> (raw)
In-Reply-To: <20100116101714.GN19799@ZenIV.linux.org.uk>

On Sat, Jan 16, 2010 at 10:17:14AM +0000, Al Viro wrote:
> 	* [unsolved, to be dealt along with per-superblock write counts]
> do_remount() plays fast and loose with MNT_READONLY for !MS_BIND case.
> 	* [*really* unsolved] it remains to be seen whether we want to
> propagate modifications of mount flags via shared subtree stuff.  For most
> of those it's trivial (and arguably the right thing to do), but ro/rw is
> really nasty.  Nick's mnt_want_write() implementation will need very careful
> analysis.

	Speaking of autofs4, what the hell is going on in
autofs_dev_ioctl_ismountpoint()?  Checks in there make no sense,
both the "could that dentry be negative?" and whatever the hell
it is trying to do with mnt_mountpoint.  Ian?

  reply	other threads:[~2010-01-17 17:57 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-23 23:36 [PATCH 0/7] VFS prep for union mounts/writable overlays Valerie Aurora
2009-12-23 23:36 ` [PATCH 1/7] autofs4: Save autofs trigger's vfsmount in super block info Valerie Aurora
2009-12-23 23:36   ` [PATCH 2/7] VFS: Make lookup_hash() return a struct path Valerie Aurora
2009-12-23 23:36     ` [PATCH 3/7] VFS: Make real_lookup() " Valerie Aurora
2009-12-23 23:37       ` [PATCH 4/7] VFS: Propagate mnt_flags into do_loopback Valerie Aurora
2009-12-23 23:37         ` [PATCH 5/7] VFS: Add read-only users count to superblock Valerie Aurora
2009-12-23 23:37           ` [PATCH 6/7] VFS: BUG_ON() rehash of an already hashed dentry Valerie Aurora
2009-12-23 23:37             ` [PATCH 7/7] VFS: Remove unnecessary micro-optimization in cached_lookup() Valerie Aurora
2010-01-02  0:44   ` [PATCH 1/7] autofs4: Save autofs trigger's vfsmount in super block info Ian Kent
2010-01-14  5:43     ` Al Viro
2010-01-14 19:18       ` Valerie Aurora
2010-01-15  6:05         ` Ian Kent
2010-01-15  8:03           ` Al Viro
2010-01-15 14:55             ` David Howells
2010-01-15 16:58               ` Al Viro
2010-01-15 17:08                 ` David Howells
2010-01-15 17:26                   ` Al Viro
2010-01-16 10:17                     ` Al Viro
2010-01-17 17:57                       ` Al Viro [this message]
2010-01-18  4:21                         ` Ian Kent
2010-01-18  5:59                           ` Al Viro
2010-01-18  9:14                             ` Ian Kent
2010-01-18 10:27                               ` Al Viro
2010-01-18 19:35                                 ` Trond Myklebust
2010-01-25  8:16                                   ` H. Peter Anvin
2010-01-19  7:05                                 ` Ian Kent
2010-01-15 17:36             ` Steve French
2010-01-18  5:08             ` Ian Kent

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100117175723.GP19799@ZenIV.linux.org.uk \
    --to=viro@zeniv.linux.org.uk \
    --cc=Trond.Myklebust@netapp.com \
    --cc=autofs@linux.kernel.org \
    --cc=dhowells@redhat.com \
    --cc=jblunck@suse.de \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=raven@themaw.net \
    --cc=sfrench@samba.org \
    --cc=vaurora@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.