nfs-utils.git/support/include, branch gss-fixes

nfsd: fix minorversion-choosing interface

2014-01-07T20:57:48+00:00

From: "J. Bruce Fields" 

By unconditionally adding ?4.2 to the version string written to the
kernel we make nfs-utils incompatible with pre-4.2-supporting kernels.

Ditto for 4.1.  This problem was introduced by
12a590f8d556c00a9502eeebaa763d906222d521 "rpc.nfsd: Allow v4.2 server
support with the -V option", which also change nfsd to unconditionally
pass ?4.2.

Instead, just don't mention 4.1 or 4.2 unless the commandline has
specifically requested that one or the other be turned on or off.

Tested-by: Joakim Tjernlund 
Reported-by: Joakim Tjernlund 
Signed-off-by: J. Bruce Fields 
Signed-off-by: Steve Dickson

nfs-utils: consolidate mydaemon() and release_parent() implementations

2013-11-20T20:04:47+00:00

We currently have 2 cut-and-paste versions of this code. One for idmapd
and one for svcgssd.[1]

The two are basically equivalent but there are some small differences,
mostly related to how errors in that function are logged. svcgssd uses
printerr() with a priority of 1, which only prints errors if -v was
specified. That doesn't seem to be quite right. Daemonizing errors are
necessarily fatal and should be logged as such. The one for idmapd uses
err(), which always prints to stderr even though we have the xlog
facility set up. Since both have xlog configured at this point, log the
errors using xlog_err() instead.

The only other significant difference I see is that the idmapd version
will open "/" if it's unable to open "/dev/null". I believe that however
was a holdover from an earlier version of that function that did not
error out when we were unable to open a file descriptor. Since the
function does that now, I don't believe we need that fallback anymore.

[1]: technically, we have a third in statd too, but it's different
     enough that I don't want to touch it here.

Signed-off-by: Jeff Layton 
Signed-off-by: Steve Dickson

mountd: Use protocol bit fields to turn protocols off.

2013-11-05T19:10:05+00:00

Convert the current code to used the NFSCTL_XXX macros
to turn off the TCP listener.

Signed-off-by: Steve Dickson

exportfs: exit with error code if there was any error (take 2).

2013-11-05T19:03:53+00:00

exportfs currently exits with a non-zero error for some errors,
but not for others.

It does this by having various support routines set the global
variable "export_errno".

Change this to have 'xlog' set export_errno if an ERROR is
reported.  That way all errors will be caught.

Note that the exit error code is changed from 22 (EINVAL)
to the more traditional '1'.

Signed-off-by: NeilBrown 
Signed-off-by: Steve Dickson

mount.nfs: dont pass options from configuration sections that are not relevant

2013-08-19T17:04:52+00:00

The nfsmount.conf file has the following format:

[ section "arg" ]
	tag = value

conf_get_tag_list() currently doesn't check the arg field so we wind up
getting all the options that fall under a particular section value,
instead of just the ones that match the specific "arg" field.  As a
result, we wind up passing options to the mount syscall from sections
that aren't even relevant to the mount operation that is being
performed.

For example, if we have three different server sections, and each
section has an Nfsvers tag, then the string we pass to the mount syscall
will have two extra occurrences of the nfsvers option.  Each option
should appear at most 4 times -- once for the system section, once for
the server-specific section, once for the mount-specific section, and
once for the command line mount options.

Acked-by: Chuck Lever 
Signed-off-by: Scott Mayhew 
Signed-off-by: Steve Dickson

rpc.nfsd: Allow v4.2 server support with the -V option

2013-08-19T17:04:50+00:00

Signed-off-by: Steve Dickson

rpc.nfsd: No longer advertise NFS v2 support.

2013-08-19T17:04:49+00:00

Changed the default protocol versions that rpc.nfsd
register with rpcbind to just 3 and 4. Version 2
can still be enabled with the '-V' flag, but it
will not be on by default.

Signed-off-by: Steve Dickson

statd: exit if a statd is already running

2013-05-07T15:47:35+00:00

Moves nfs_probe_statd from mount to nfs support lib to share with statd.

Acked-by: Chuck Lever 
Signed-off-by: Weston Andros Adamson 
Signed-off-by: Steve Dickson

mountd: fix exporting of "/" with sec= setting.

2013-04-22T16:47:20+00:00

Commit 91bb95f2689e84856ecdf6fac365489d36709cf9
   4set_root: force "fsid=0" for all exports of '/'

set NFSEXP_FSID for the export of "/" if nothing else had any fsid set,
however it didn't also set the flag for all security flavours.  So the
kernel complains that the flags on the security flavours don't match and
it rejects the export.

So call fix_pseudoflavor_flags() in write_secinfo() to make sure that
any fiddling that has been done to e_flags gets copied to e_secinfo.

Signed-off-by: NeilBrown 
Signed-off-by: Steve Dickson

nfsd: Add support for the -V and --nfs-version optional arguments

2013-03-25T20:07:59+00:00

Add command line options to enable those NFS versions that are
currently disabled by default. We choose to use the options '-V'
and '--nfs-version' for compatibility with rpc.mountd.

Acked-by: J. Bruce Fields 
Signed-off-by: Trond Myklebust 
Signed-off-by: Steve Dickson