From: Trond Myklebust <trond.myklebust@fys.uio.no>
To: Mike Waychison <Michael.Waychison@sun.com>
Cc: John S J Anderson <jacobs@genehack.org>,
linux-kernel@vger.kernel.org,
Alexander Viro <viro@parcelfarce.linux.theplanet.co.uk>,
Herbert Poetzl <herbert@13thfloor.at>
Subject: Re: bug with multiple mounts of filesystems in 2.6
Date: Mon, 26 Jul 2004 22:51:52 -0400 [thread overview]
Message-ID: <1090896712.6809.171.camel@lade.trondhjem.org> (raw)
In-Reply-To: <4105A858.7090209@sun.com>
På må , 26/07/2004 klokka 20:56, skreiv Mike Waychison:
> As an example where sharing the super_block is wrong (albeit probably
> just an oversight) is that the protocols (udp vs tcp) are not compared
> in nfs_compare_super. You could argue that the client fhandles should
> be different though, I'm not sure..
Not an oversight. It's just that we don't really have a model for
trunking a single cache over several different RPC connections.
Basically, it all boils down to the problem that inodes and dentries
have no idea of which namespace you used to access them, and hence even
if you did have space in the vfsmount in which to store an rpc_client
struct and perhaps rsize/wsize ... you still have a reconstruction job
to do.
> Another 'bind mount extension' that would be nice to change at the
> vfsmount level may be w/rsize, but that is probably a very intrusive
> change for nfs and probably not possible. Thoughts?
The "struct nfs_open_context" I introduce in the latest NFS4_ALL patches
could probably be modified to include vfsmount information.
My question is why do people need to do this? What problems does it
solve?
Normally, rsize/wsize are per-server parameters. Their optimal value
depends above all upon the quality of the network link between client
and server. Ditto for the choice of UDP vs TCP: why would you want to
choose to use both options against a given server?
> [1] - I haven't tested mounting nfs ro, and then mounting nfs rw using
> the bind extensions. Does nfs make any assumptions about the mount
> being ro?
Nope, however the VFS support for this is missing. That was part of what
Herbert was doing in his patches.
Cheers,
Trond
prev parent reply other threads:[~2004-07-27 2:52 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-07-26 16:29 John S J Anderson
2004-07-26 19:37 ` Trond Myklebust
2004-07-26 21:33 ` Mike Waychison
2004-07-26 22:30 ` Herbert Poetzl
2004-07-26 22:35 ` Trond Myklebust
2004-07-27 0:56 ` Mike Waychison
2004-07-27 2:51 ` Trond Myklebust [this message]
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=1090896712.6809.171.camel@lade.trondhjem.org \
--to=trond.myklebust@fys.uio.no \
--cc=Michael.Waychison@sun.com \
--cc=herbert@13thfloor.at \
--cc=jacobs@genehack.org \
--cc=linux-kernel@vger.kernel.org \
--cc=viro@parcelfarce.linux.theplanet.co.uk \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®