]> git.decadent.org.uk Git - nfs-utils.git/commit
mountd: prepend '$' to make use_ipaddr clients self-describing
authorJ. Bruce Fields <bfields@redhat.com>
Thu, 3 May 2012 19:06:21 +0000 (15:06 -0400)
committerSteve Dickson <steved@redhat.com>
Thu, 3 May 2012 19:12:10 +0000 (15:12 -0400)
commitc2544b77566690ebec32a2d47c9249548b1a0941
treeee5ad75e30a6036194e6c5735a046587e1d3e005
parent236025253c6ec534cd816390464435ec18c499f7
mountd: prepend '$' to make use_ipaddr clients self-describing

From: "J. Bruce Fields" <bfields@redhat.com>

Mountd is responsible for filling three interrelated kernel caches:

- auth_unix_ip maps an incoming ip addresses to a "domain".
- nfsd_fh maps (domain, filehandle-fragment) pairs to paths.
- nfsd_export maps (domain, path) pairs to export options.

Note that each export is assocated with a "client" string--the part
before the parentheses in an /etc/export line--which may be a domain
name, a netgroup, etc.

The "domain" string in the above three caches may be either:

- in the !use_ipaddr case, a comma-separated list of client
  strings.
- in the use_ipaddr case, an ip address.

In the former case, mountd does the hard work of matching an ip address
to the clients when doing the auth_unix_ip mapping.  In the latter case,
it delays that until the nfsd_fh or nfsd_export upcall.

We're currently depending on being able to flush the kernel caches
completely when switching between the use_ipaddr and !use_ipaddr cases.
However, the kernel's cache-flushing doesn't really provide reliable
guarantees on return; it's still possible we could see nfsd_fh or
nfsd_export upcalls with the old domain-type after flushing.

So, instead, make the two domain types self-describing by prepending a
"$" in the use_ipaddr case.

Reviewed-by: NeilBrown <neilb@suse.de>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
utils/mountd/auth.c
utils/mountd/cache.c
utils/mountd/mountd.h