+ statd: Remove vestigial "-w" option from man page synopsis
+
+ The synopsis of rpc.statd in its man page lists "-w" as a valid
+ option. There is currently no support in the source code for a "-w"
+ option.
+
+ BugLink: https://bugzilla.linux-nfs.org/show_bug.cgi?id=199
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit b57cd77c13831051ad974ae027d96cd88a8d0c59
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Apr 6 10:48:38 2011 -0400
+
+ mount.nfs: Don't leak socket in nfs_ca_sockname()
+
+ Ensure the test socket is always closed before nfs_ca_sockname()
+ returns. Otherwise it's orphaned.
+
+ BugLink: https://bugzilla.linux-nfs.org/show_bug.cgi?id=197
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 502eef09d8050ffb87d394397c2780e1ef042d68
+Author: Steve Dickson <steved@redhat.com>
+Date: Wed Apr 6 10:46:06 2011 -0400
+
+ Removed a warning from v4root.c
+
+ v4root.c:176:9: warning: variable 'ret' set but not used
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit c2fa189a0467c25666f014cf9ff2576a9f54d682
+Author: Steve Dickson <steved@redhat.com>
+Date: Wed Apr 6 10:39:10 2011 -0400
+
+ Removed a warning from exportfs.c
+
+ exportfs.c:280:29: warning: 'exp' may be used uninitialized in this function
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit b98ae9df8c4904289c9390288325058b24caa423
+Author: Steve Dickson <steved@redhat.com>
+Date: Wed Apr 6 10:36:30 2011 -0400
+
+ Removed a warning from conffile.c
+
+ conffile.c:258:19: warning: 'j' may be used uninitialized in this function
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 012e1a4bf2a002e8cd4d5be3478bfa20a91cbfed
+Author: Masatake YAMATO <yamato@redhat.com>
+Date: Mon Mar 7 08:36:19 2011 -0500
+
+ Update man pages for /etc/exports.d
+
+ Man page updates for /etc/exports.d.
+
+ Signed-off-by: Masatake YAMATO <yamato@redhat.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit c7427b57e2be8ef0d57ad0618d4590c062b130f5
+Author: Masatake YAMATO <yamato@redhat.com>
+Date: Mon Mar 7 08:18:51 2011 -0500
+
+ Read /etc/exports.d/*.export as extra export files
+
+ This patch adding a capability to read /etc/exports.d/*.exports as
+ extra export files to exportfs.
+
+ If one wants to add or remove an export entry in a script, currently
+ one may have to use sed or something tool for adding or removing the
+ line for the entry in /etc/exports file.
+
+ With the patch, adding and removing an entry from a script is much
+ easier.
+ cat<<EOF... or mv can be used for adding. rm can be used for removing.
+
+ Signed-off-by: Masatake YAMATO <yamato@redhat.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit edb9b7f2ab9806afb9af31eabeb505fe454c51df
+Author: Steve Dickson <steved@redhat.com>
+Date: Sat Mar 5 16:17:01 2011 -0500
+
+ Cleaned up a warning in rpcdispatch.c
+
+ rpcdispatch.c:40:20: warning: comparison between signed and unsigned
+ integer expressions
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 930323817b61877d61fb8ef57229013daa2e6091
+Author: Steve Dickson <steved@redhat.com>
+Date: Sat Mar 5 16:13:01 2011 -0500
+
+ mount: Remove MOUNT_CONFIG warnings
+
+ The following changes are needed to remove compile warnings when
+ MOUNT_CONFIG is not defined
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 3ef3dc8f1e87ba7a6eaa3c2a6965aff6c80ba414
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Thu Mar 3 17:26:33 2011 -0500
+
+ mount: Recognize zero as a valid value for the port= option
+
+ While zero is not a valid IP port number, zero does represent a valid
+ value for "port=". It means "query rpcbind to discover the actual
+ non-zero port number to use". So the parsing functions that handle
+ "port=" should not flag zero as an invalid value.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit b3a4dbfb61dad59829f5191d727267b2ea45937a
+Author: Mi Jinlong <mijinlong@cn.fujitsu.com>
+Date: Wed Feb 9 11:29:42 2011 -0500
+
+ Gssd: modify wrong err message at handle_gssd_upcall
+
+ Modify wrong err message at handle_gssd_upcall when
+ sscanf encryption types fail.
+
+ Signed-off-by: Mi Jinlong <mijinlong@cn.fujitsu.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 45e4597bd570ed40221f51887cde7d7f096f55e7
+Author: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
+Date: Wed Feb 9 11:27:19 2011 -0500
+
+ Support AD style kerberos automatically in rpc.gss
+
+ An Active Directory KDC will only grant a TGT for UPNs, getting
+ a TGT for SPNs is not possible:
+
+ $ kinit -k host/ib5@ADS.ORCORP.CA
+ kinit: Client not found in Kerberos database while getting initial
+ credentials
+
+ The correct thing to do for machine credentials is to get a TGT
+ for the computer UPN <HOSTNAME>$@REALM:
+ $ kinit -k IB5\$
+ $ klist
+ 12/22/10 11:43:47 12/22/10 21:43:47 krbtgt/ADS.ORCORP.CA@ADS.ORCORP.CA
+
+ Samba automatically creates /etc/krb5.keytab entry for the computer UPN,
+ this patch makes gssd_refresh_krb5_machine_credential prefer it above
+ the SPNs if it is present.
+
+ The net result is that nfs client works automatically out of the box
+ if samba has been used to setup kerberos via 'net ads join' 'net ads
+ keytab create'
+
+ Tested using Windows Server 2003 R2 as the AD server.
+
+ Signed-off-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 730f6986f86873513fa021a450eb55ccd0f2fbff
+Author: Steve Dickson <steved@redhat.com>
+Date: Wed Jan 26 07:49:19 2011 -0500
+
+ Fixed segfault in rpc.mountd
+
+ A unallocated piece of memory, instead of a NULL point, was being
+ used to initialize a ->next point in the mount link list which
+ caused a segfault after a few remote accesses via the showmount
+ command.
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 544ed73d5ab27c1390833d5cf93b9585c151667d
+Author: Steve Dickson <steved@redhat.com>
+Date: Fri Jan 14 10:12:28 2011 -0500
+
+ Improve debugging in svcgssd
+
+ Added in gss_display_error() which translates the GSS error into the
+ actual GSS macro name. Currently only the translation of these errors
+ are logged. Since those translations are buried deep in the kerberos
+ library code, having the actual GSS macro name makes it easier to
+ follow the code.
+
+ Moved the nfs4_init_name_mapping() call into main() so if debug is
+ enabled the DNS name and realms will be logged during start up.
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 57be18b9ab08148a1cc9d5af588119885720be8b
+Author: Mi Jinlong <mijinlong@cn.fujitsu.com>
+Date: Tue Jan 4 11:16:45 2011 -0500
+
+ libnsm.a: modify return value to false from 0 at nsm_drop_privileges()
+
+ At nsm_drop_privileges(), for improving readability, unify
+ the return value.
+
+ Signed-off-by: Mi Jinlong <mijinlong@cn.fujitsu.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 5c498280fd9353ded3ea169841079bdae23418e2
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Mon Dec 13 14:50:45 2010 -0500
+
+ libnsm.a: sm-notify sometimes ignores monitored hosts
+
+ Monitored host information is stored in files under /var/lib/nfs.
+ When visiting entries in the monitored hosts directory, libnsm.a
+ examines the value of dirent.d_type to determine if an entry is a
+ regular file.
+
+ According to readdir(3), the d_type field is not supported by all
+ file system types. My root file system happens to be one where d_type
+ isn't supported. Typical installations that use an ext-derived root
+ file system are not exposed to this issue, but those who use xfs, for
+ instance, are.
+
+ On such file systems, not only are remote peers not notified of
+ reboots, but the NSM state number is never incremented. A statd warm
+ restart would not re-monitor any hosts that were monitored before
+ the restart.
+
+ When writing support/nsm/file.c, I copied the use of d_type from the
+ original statd code, so this has likely been an issue for some time.
+
+ Replace the use of d_type in support/nsm/file.c with a call to
+ lstat(2). It's extra code, but is guaranteed to work on all file
+ system types.
+
+ Note there is a usage of d_type in gssd. I'll let gssd and rpcpipefs
+ experts decide whether that's worth changing.
+
+ Fix for:
+
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=193
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 089df7c754d9ebab0a7b804f396626ac95fee2e6
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Mon Dec 13 14:47:42 2010 -0500
+
+ libnsm.a: Replace __attribute_noinline__
+
+ Replace the __attribute_noinline__ form with
+
+ __attribute__((__noinline__)).
+
+ Even though the compiler didn't complain about __attribute_malloc__,
+ also replace those in order to maintain consistent style throughout the
+ source file.
+
+ Fix for:
+
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=194
+
+ Reported-by: "Gabor Z. Papp" <gzp@papp.hu>
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 7869a76207d3f4b3bd4ab57b4a7a8807ac2ff0c6
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Mon Dec 13 14:36:15 2010 -0500
+
+ sm-notify: Make use of AI_NUMERICSERV conditional
+
+ Gabor Papp reports nfs-utils-1.2.3 doesn't build on his system that
+ uses glibc-2.2.5:
+
+ make[3]: Entering directory
+ `/home/gzp/src/nfs-utils-1.2.3/utils/statd'
+ gcc -DHAVE_CONFIG_H -I. -I../../support/include -D_GNU_SOURCE -Wall
+ -Wextra -Wstrict-prototypes -pipe -g -O2 -MT sm-notify.o -MD
+ -MP -MF .deps/sm-notify.Tpo -c -o sm-notify.o sm-notify.c
+ sm-notify.c: In function 'smn_bind_address':
+ sm-notify.c:247: error: 'AI_NUMERICSERV' undeclared (first use in this
+ function)
+ sm-notify.c:247: error: (Each undeclared identifier is reported only
+ once
+ sm-notify.c:247: error: for each function it appears in.)
+ make[3]: *** [sm-notify.o] Error 1
+
+ According to the getaddrinfo(3) man page, AI_NUMERICSERV is available
+ only since glibc 2.3.4. getaddrinfo(3) seems to convert strings
+ containing a number to the right port value without the use of
+ AI_NUMERICSERV, so I think we can survive on older glibc's without it.
+ It will allow admins to specify service names as well as port numbers
+ on those versions.
+
+ There are uses of AI_NUMERICSERV in gssd and in nfs_svc_create(). The
+ one in nfs_svc_create() is behind HAVE_LIBTIRPC, and the other is a
+ issue only for those who want to deploy Kerberos -- likely in both
+ cases, a more modern glibc will be present. I'm going to leave those
+ two.
+
+ Fix for:
+
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=195
+
+ Reported-by: "Gabor Z. Papp" <gzp@papp.hu>
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit e8dbaddc8465dcd07b53f8e80a537703dd0248ca
+Author: Sid Moore <learnmost@gmail.com>
+Date: Fri Dec 3 09:19:06 2010 -0500
+
+ rpc.mountd: Checking RPC Procedure ID before process it
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 3c6973c595d62dc6452967d50ae8abe69f9f8bad
+Author: Mi Jinlong <mijinlong@cn.fujitsu.com>
+Date: Mon Nov 29 10:59:10 2010 -0500
+
+ libnfs.a: fix a bug when parse section's arg
+
+ When parsing section's arg at configure file, the pointer
+ should stop when fetch ']', and give the warning message.
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 86f7be64cafd17d4a3f164603484eaedb4757431
+Author: Harshula Jayasuriya <harshula@redhat.com>
+Date: Mon Nov 22 11:22:31 2010 -0500
+
+ nfs-utils: nfsstat: has_stats() does not function correctly for NFSv4 client stats
+
+ The NFSv4 client procs/ops in "struct rpc_procinfo nfs4_procedures" is
+ used to generate the NFS client stats interface:
+ ------------------------------------------------------------
+ net 0 0 0 0
+ rpc 15 0 0
+ proc2 18 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0
+ proc3 22 0 2 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 1 2 1 0
+ proc4 42 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 1 0 0 0 0 0 0 0 0 0 0 0
+ 0 0 0
+ 0 0 0 0 0 0 0
+ ------------------------------------------------------------
+ Note, for proc4, the number 42. That is the number of stats that follow
+ on the same line. Currently nfsstat's has_stats() relies on this number
+ to be equal to CLTPROC4_SZ. Unfortunately this is not the case. I have
+ changed has_stats() not to rely on these two values being equal. This
+ should also allow nfsstat to work with different kernel versions that
+ expose a different number of NFS client ops.
+
+ * Fix has_stats()
+ * Stop print_clnt_list() printing server stats!
+ * Describe the option -3 and -4 completely in the nfsstat manpage.
+
+ Signed-off-by: Harshula Jayasuriya <harshula@redhat.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 0868dcccb9a3bf3d022a32ff31311fe371484e77
+Author: Steve Dickson <steved@redhat.com>
+Date: Sat Nov 20 15:01:21 2010 -0500
+
+ Enable nfsidmap to compile
+
+ Only enable the compilation of nfsidmap when libnfsidmap support it.
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 6f07548141e710767d425e119d9823691293771d
+Author: Bryan Schumaker <bjschuma@netapp.com>
+Date: Fri Nov 19 12:01:10 2010 -0500
+
+ Add the new nfsidmap program
+
+ This patch adds the nfsidmap program to nfs-utils. This program is
+ called by the nfs idmapper through request-keys to map between
+ uid / user name and gid / group name.
+
+ Signed-off-by: Bryan Schumaker <bjschuma@netapp.com>
+ Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 409487978593de13ae36be0ee56d8111ad6b3319
+Author: Steve Dickson <steved@redhat.com>
+Date: Mon Nov 22 11:33:37 2010 -0500
+
+ Removed a couple warnings from utils/mount/stropts.c
+
+ stropts.c:740:6: warning: 'ret' may be used uninitialized in this function
+ stropts.c:653:6: warning: 'ret' may be used uninitialized in this function
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit f4968a724c1d4162a8e2b9f6a19c460cc56c95f7
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Fri Oct 29 12:56:21 2010 -0400
+
+ nfs(5): Document remount behavior
+
+ It appears that, for a long while, NFS "remount" mounts have
+ completely wiped the existing mount options in /etc/mtab for a given
+ mount point. This is a problem for umount.nfs, since it reads its
+ options out of /etc/mtab to find out how to do the unmount.
+
+ The mount(8) command provides the NFS mount subcommand with the mount
+ options to perform the remount. There are four cases to consider:
+
+ 1. Both the device and mount directory are specified on the
+ command line, and the target mount point is in /etc/fstab
+
+ 2. Only one of the device and mount directory is specified on
+ the command line, and the target mount point is in
+ /etc/fstab
+
+ 3. Both the device and mount directory are specified on the
+ command line, and the target mount point is not in /etc/fstab
+
+ 4. Only one of the device and mount directory is specified on
+ the command line, and the target mount point is not in
+ /etc/fstab
+
+ Currently only case 4 works correctly. In that case, mount(8)
+ provides the correct set of mount options to the mount.nfs
+ subcommand and it can update /etc/mtab correctly.
+
+ Cases 1 and 3 replace all mount options in /etc/mtab with the options
+ provided on the command line during a remount. Case 2 replaces the
+ mount options in /etc/mtab with a mix of options from /etc/fstab and
+ /etc/mtab.
+
+ Cases 1 and 3 are historical behavior. Basically this is a formal
+ interface to allow administrators to replace the mount options in
+ /etc/mtab completely, instead of merging in new ones. The present
+ patch documents that behavior in nfs(5), and provides best practice
+ for remounting NFS mount points.
+
+ There are near-term plans to address case 2 by fixing mount(8)
+ (provided by utils-linux-ng in most distributions).
+
+ This is a partial fix for:
+
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=188
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 6d1a82b005994f759f2c847c0354413a24643da5
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Thu Oct 28 13:15:22 2010 -0400
+
+ nfs(5): Grammar and style fixes
+
+ Clean up grammar and style issues introduced by recent updates. Also,
+ I'm not certain inappropriate options are always ignored.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit ab2cdb859f738a25e2567a2ec674cfa78a0a175d
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Thu Oct 28 13:13:19 2010 -0400
+
+ mount.nfs: mnt_freq and mnt_pass are always zero
+
+ Clean up.
+
+ No need to pass constant zeros to add_mtab() from its only call site.
+ Ensure that initialization of a struct mntent is consistent in both
+ places that it is done.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit bc4a0c42570d5620cc1bb32428e16b9c9b5f3863
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Thu Oct 28 13:10:48 2010 -0400
+
+ mount.nfs: Fix memory leak in nfs_sys_mount()
+
+ This appears to have been left behind by last year's adjustments to
+ how the extra_opts string is constructed.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 1f237ac72e6f563908b350e11fd2bb866c003028
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Thu Oct 28 13:09:38 2010 -0400
+
+ mount: Fix compiler warning in nfs_parse_retry_option()
+
+ stropts.c: In function nfs_parse_retry_option:
+ stropts.c:131: warning: conversion to unsigned int from long int may
+ alter its value
+
+ Make it more clear what the second argument is for, and flag the
+ switch fallthrough case.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 1ea2c3be33f2eb4630c5cdb78edf2bb670b294ab
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Thu Oct 28 12:12:12 2010 -0400
+
+ nfs-utils: Remove all uses of AI_ADDRCONFIG
+
+ It was reported that, if only "lo" is up,
+
+ mount.nfs 127.0.0.1:/export /mount
+
+ fails with "Name or service not known".
+
+ "man 3 getaddrinfo" says this:
+
+ If hints.ai_flags includes the AI_ADDRCONFIG flag, then IPv4
+ addresses are returned in the list pointed to by res only if the
+ local system has at least one IPv4 address configured, and IPv6
+ addresses are only returned if the local system has at least
+ one IPv6 address configured.
+
+ The man page oversimplifies here. A review of glibc shows that
+ getaddrinfo(3) explicitly ignores loopback addresses when deciding
+ whether an IPv4 or IPv6 address is configured.
+
+ This behavior around loopback is a problem not just for mount.nfs,
+ but also for RPC daemons that have to start up before a system's
+ networking is fully configured and started. Given the history of
+ other problems with AI_ADDRCONFIG and the unpredictable behavior it
+ introduces, let's just remove it everywhere in nfs-utils.
+
+ This fix addresses:
+
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=191
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit f8e315543b7f1db7f37a4bfe8ede3020cef62868
+Author: Jeff Layton <jlayton@redhat.com>
+Date: Thu Oct 28 09:18:33 2010 -0400
+
+ nfs-utils: fix default value for --enable-tirpc
+
+ We need $enable_tirpc to be a tristate. 'yes' means that someone
+ explicitly requested building with tirpc. 'no' means that it was
+ explicitly disabled. Anything else means that no one specified a value.
+
+ Fix it by setting the value to a blank string so that the default is
+ properly undefined.
+
+ Reported-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Jeff Layton <jlayton@redhat.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit c62d756402509ca5d07c1fd4d2e5a9d78dc4171b
+Author: Steve Dickson <steved@redhat.com>
+Date: Tue Oct 19 15:54:35 2010 -0400
+
+ Updated rpc.mountd man page
+
+ Updated the rpc.mountd man page to no longer reference
+ v3 as the "newer" version and also mentioned v4 as
+ a supported version.
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 79e9079e9af4e5c2aa1d77815df1147b26876eb8
+Author: Steve Dickson <steved@redhat.com>
+Date: Tue Oct 19 15:54:04 2010 -0400
+
+ Cleared up the sync option in exportfs man page
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 6f228ea26be06572de245aed5496aaa122cca5a8
+Author: Steve Dickson <steved@redhat.com>
+Date: Fri Oct 15 17:20:28 2010 -0400
+
+ Removed duplicate entries in export man page
+
+ The man page's paragraphs about "refer=" and "replicas="
+ each appear twice.
+
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 849b7072a04975bb5da09245fbcacb0cb754a909
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Thu Oct 14 10:33:25 2010 -0400
+
+ mountd: Clear mountd registrations at start up
+
+ Clear stale MNT registrations before mountd tries to create fresh
+ listeners, to ensure that mountd starts. This is also what statd does.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 93dcf64cc4a9e67f693aea35c8193428015f4a30
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 13:57:52 2010 -0400
+
+ behavior as file systems that use the monolithic /sbin/mount command.
+ See the MS_NOMTAB macro in utils-linux-ng/mount/mount.c.
+
+ Note that mount(8) has MS_USERS and MS_USER in the "nomtab" category
+ as well, but mount.nfs needs to record those values so that unmounting
+ a user-mounted NFS file system can work.
+
+ While we're here, fix some white space damage in fix_opts_string().
+
+ This is a partial fix for:
+
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=188
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit dc08c702a6c7f824f317af561f491635ee898a71
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 13:55:10 2010 -0400
+
+ umount.nfs: Distinguish between nfs4 and nfs mounts
+
+ Neil Brown reports that umount.nfs is still confused by "-t nfs -o
+ vers=4" mounts.
+
+ /etc/mtab can be confused. /proc/mounts is authoritative on the
+ fstype of a mount. Have umount.nfs consult it to determine which
+ mechanism to use for unmounting. The code to read /proc/mounts was
+ lifted from the nfsstat command.
+
+ The code introduced by this patch may look like belt-n-suspenders, but
+ we have two use cases to consider:
+
+ 1. Old kernels don't support the "vers=4" mount option, so
+ umount.nfs must look for the "nfs4" fstype
+ 2. Upcoming kernels may eliminate support the "nfs4" fstype, so
+ umount.nfs must look for the "vers=4" mount option
+
+ Thus this logic checks for "nfs4" first then looks for the NFS version
+ setting.
+
+ Note that we could handle unmounting entirely in the kernel, but that
+ won't help older kernels that have this issue.
+
+ See:
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=189
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 17962b82afb68ca8e6b0d3f432d36c6c7c4980ea
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 13:01:51 2010 -0400
+
+ mount.nfs: mountproto does not support RDMA
+
+ Clean up. Our client does not support the MNT protocol on RDMA.
+
+ nfs_mount_protocol() isn't invoked for RDMA mounts (they are shunted
+ off before nfs_options2pmap() is invoked). But in case it ever is,
+ it should return the expected response.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 73c61fa5cd114fa6eae0e095724ed63aa66a4a6b
+Author: NeilBrown <neilb@suse.de>
+Date: Wed Oct 13 12:08:41 2010 -0400
+
+ gcc complained:
+
+ client.c: In function 'init_netmask6':
+ client.c:181:1: warning: no return statement in function returning
+ non-void
+
+ and Suse' build system complained
+
+ I: Program returns random data in a function
+ E: nfs-utils no-return-in-nonvoid-function client.c:181
+
+ when I built without --enable-ipv6
+
+ Reviewed-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: NeilBrown <neilb@suse.de>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 7e90281b88c05b01c61152b54a0cf2faec45b09c
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 12:02:32 2010 -0400
+
+ mount.nfs: Eliminate compiler warnings in utils/mount/network.c
+
+ Clean up.
+
+ network.c: In function get_socket:
+ network.c:431: warning: dereferencing type-punned pointer might break
+ strict-aliasing rules
+
+ network.c: In function probe_bothports:
+ network.c:759: warning: dereferencing type-punned pointer might break
+ strict-aliasing rules
+ network.c:762: warning: dereferencing type-punned pointer might break
+ strict-aliasing rules
+
+ network.c: In function nfs_probe_statd:
+ network.c:775: warning: dereferencing type-punned pointer might break
+ strict-aliasing rules
+
+ network.c: In function nfs_call_umount:
+ network.c:904: warning: dereferencing type-punned pointer might break
+ strict-aliasing rules
+
+ network.c: In function nfs_ca_sockname:
+ network.c:1106: warning: dereferencing type-punned pointer might break
+ strict-aliasing rules
+ network.c:1112: warning: dereferencing type-punned pointer might break
+ strict-aliasing rules
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 57385cf87790c0cbdfddfccdde66bd2c8da45923
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:59:30 2010 -0400
+
+ mount.nfs: Eliminate compiler warning in utils/mount/parse_opt.c
+
+ parse_opt.c: In function po_rightmost:
+ parse_opt.c:517: warning: conversion to int from unsigned int may
+ change the sign of the result
+
+ "i" contains the function's result value, so it should be defined as
+ the same type as the function's return type.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 1ee10ef034cbca86da4df271ac4097a948e7ab59
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:58:27 2010 -0400
+
+ mount.nfs: Eliminate compiler warning in utils/mount/nfsumount.c
+
+ Clean up.
+
+ nfsumount.c:374: warning: ISO C forbids omitting the middle term of
+ a ?: expression
+
+ This is also probably harmless, but let's make the code unambiguous.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit e9c97e4f7075e563d7a442ca8298ac56bafba0d5
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:56:58 2010 -0400
+
+ mount.nfs: Eliminate compiler warning in utils/mount/nfsumount.c
+
+ Clean up.
+
+ nfsumount.c:265: warning: no previous prototype for nfsumount
+
+ It's also a good idea if the compiler can ensure that the prototype
+ in nfsmount.h matches the actual function defined in nfsumount.c.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit e2b6d9cbaf20df26dd371a715fce3ae158f37126
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:54:49 2010 -0400
+
+ mount.nfs: Eliminate compiler warnings in utils/mount/mount.c
+
+ Clean up.
+
+ mount.c: In function parse_opt:
+ mount.c:354: warning: conversion to size_t from int may change the
+ sign of the result
+ mount.c:354: warning: conversion to int from size_t may change the
+ sign of the result
+ mount.c:359: warning: conversion to size_t from int may change the
+ sign of the result
+ mount.c:359: warning: conversion to int from size_t may change the
+ sign of the result
+ mount.c: In function parse_opts:
+ mount.c:374: warning: conversion to int from size_t may change the
+ sign of the result
+ mount.c:377: warning: conversion to size_t from int may change the
+ sign of the result
+
+ Character string lengths are usually size_t anyway. We can easily
+ avoid the implicit type cast here.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 013e8ec9ffb9f28f97e58299719023faf846a029
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:53:44 2010 -0400
+
+ mount.nfs: Eliminate compiler warning in utils/mount/mount.c
+
+ Clean up.
+
+ mount.c: At top level:
+ mount.c:324: warning: no previous prototype for ?mount_usage?
+
+ mount_usage() has no callers outside of utils/mount/mount.c and no
+ prototype is provided in a header file. Make it static.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 47480475c99335c1203e81662f815b62573c19e8
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:50:57 2010 -0400
+
+ mount.nfs: Eliminate compiler warnings in utils/mount/version.h
+
+ Clean up.
+
+ In file included from mount.c:50:
+ version.h: In function linux_version_code:
+ version.h:48: warning: conversion to unsigned int from int may
+ change the sign of the result
+ version.h:48: warning: conversion to unsigned int from int may
+ change the sign of the result
+ version.h:48: warning: conversion to unsigned int from int may
+ change the sign of the result
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 00885013dccbe00f5cc4e19223cf18e85a8e616a
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:44:23 2010 -0400
+
+ mount.nfs: Eliminate compiler warning in utils/mount/mount.c
+
+ Clean up.
+
+ In file included from mount.c:41:
+ mount_config.h:35: warning: no previous prototype for mount_config_opts
+
+ Functions defined in include files are usually declared as "static
+ inline," eliminating the need for a forward declaration.
+
+ While I was there, I also fixed the macro that prevents including
+ mount_config.h multiple times, and fixed some white space damage.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit ffa577350b03ddd421d455a8cd4cff86e3718264
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:38:22 2010 -0400
+
+ mount.nfs: Eliminate compiler warnings
+
+ Clean up.
+
+ fstab.c: In function ?lock_mtab?:
+ fstab.c:385: warning: declaration of ?errsv? shadows a previous local
+ fstab.c:367: warning: shadowed declaration is here
+ fstab.c:407: warning: declaration of ?errsv? shadows a previous local
+ fstab.c:367: warning: shadowed declaration is here
+ fstab.c:417: warning: declaration of ?tries? shadows a previous local
+ fstab.c:325: warning: shadowed declaration is here
+ fstab.c:422: warning: declaration of ?errsv? shadows a previous local
+ fstab.c:367: warning: shadowed declaration is here
+
+ These are probably harmless. Reusing a variable name, however, is a
+ little confusing to follow when reading the code.
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 5fe118b838254023d83424c5010ae73a91ec267d
+Author: Trond Myklebust <Trond.Myklebust@netapp.com>
+Date: Wed Oct 13 11:27:21 2010 -0400
+
+ export: Ensure that we free struct exportent->e_uuid
+
+ Currently, the exportent->e_uuid is initialised in
+ support/nfs/exports.c:parseopts(), but it is never freed.
+
+ Also ensure that exportent->e_uuid is duplicated correctly in
+ dupexportent().
+
+ Adjusted to account for the new export_free() helper.
+
+ Also, e_uuid points to memory that is always allocated with strdup(3),
+ not with xstrdup(). Thus it must be freed via free(3) and not via
+ xfree().
+
+ Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 656028f9925f5817c5a37565d27159973db84ec3
+Author: Chuck Lever <chuck.lever@oracle.com>
+Date: Wed Oct 13 11:22:07 2010 -0400
+
+ libnfs.a: Allow multiple RPC listeners to share listener port number
+
+ Normally, when "-p" is not specified on the mountd command line, the
+ TI-RPC library chooses random port numbers for each listener. If a
+ port number _is_ specified on the command line, all the listeners
+ will get the same port number, so SO_REUSEADDR needs to be set on
+ each socket.
+
+ Thus we can't let TI-RPC create the listener sockets for us in this
+ case; we must create them ourselves and then set SO_REUSEADDR (and
+ other socket options) by hand.
+
+ Different versions of the same RPC program have to share the same
+ listener and SVCXPRT, so we have to cache xprts we create, and re-use
+ them when additional requests for registration come from the
+ application.
+
+ Though it doesn't look like it, this fix was "copied" from the legacy
+ rpc_init() function. It's more complicated for TI-RPC, of course,
+ since a TI-RPC application can set up listeners with a nearly
+ arbitrary number of address families and socket types, not just the
+ two listeners that legacy RPC applications can set up (one for AF_INET
+ UDP and one for AF_INET TCP).
+
+ See:
+ https://bugzilla.linux-nfs.org/show_bug.cgi?id=190
+
+ Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 1296be71ebae4c0d7da61cc1077d97562d3bc549
+Author: Steve Dickson <steved@redhat.com>
+Date: Wed Oct 13 10:17:58 2010 -0400
+
+ nfs-utils: Fixed typo in NFS man page
+
+ Chuck pointed out there was a grammar typo in addition to the spelling
+ typo. Here is a revised version of the patch.
+
+ Signed-off-by: Jim Rees <rees@umich.edu>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 9afbdbad4436df6f7a5a501c6e4db04c3a5bbb08
+Author: Steve Dickson <steved@redhat.com>
+Date: Wed Oct 13 10:15:12 2010 -0400
+
+ Fix style nits in atomicio.c
+
+ Signed-off-by: Jim Rees <rees@umich.edu>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit c117b7a1f29db65d139824ba5bab2a58bf5609e2
+Author: Steve Dickson <steved@redhat.com>
+Date: Wed Oct 13 10:09:53 2010 -0400
+
+ nfs-utils: Move common code into support
+
+ There are several source files and headers present in the ./utils/idmapd
+ directory which are also usable in a doimapd daemon. Because of this we
+ move that support into the support directory such that it can be shared by
+ both daemons.
+
+ Signed-off-by: Jim Rees <rees@umich.edu>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 8c217b9623c8304608196aeb2f7360abfdf987c8
+Author: Suresh Jayaraman <sjayaraman@suse.de>
+Date: Wed Sep 29 07:14:14 2010 -0400
+
+ The kernel 2.6.37 has a add new mount option: local_lock.
+ Document the new option in the nfs(5) man page.
+
+ Signed-off-by: Suresh Jayaraman <sjayaraman@suse.de>
+ Signed-off-by: Steve Dickson <steved@redhat.com>
+
+commit 554df6bc8456c6971c1c597f70d3b9131e4e5e11
+Author: Steve Dickson <steved@redhat.com>
+Date: Tue Sep 28 08:24:16 2010 -0400
+
+ Revert "nfs-iostat.py: don't wait for an extra interval when given a count"
+
+ This reverts commit 837796686ad8f9178c7b6855ada728a53ae511e3.