From df6c9d1821585c773297fbfffa1dd315b9c84359 Mon Sep 17 00:00:00 2001 From: Chuck Lever Date: Fri, 12 Feb 2010 13:04:14 -0500 Subject: [PATCH] text-based mount: Retry when server can't be reached We want new default behavior from mount.nfs when the server refuses a connection. Since connection refusal can be spurious (for example, if the server is rebooting), mount.nfs should retry. NFS shares that are automatically mounted by /etc/fstab at boot time may be problematic. The new behavior can be disabled by specifying the "retry=0" mount option, or these mounts can be changed to background mounts by specifying the "bg" option. A kernel code change is still required for the mount(2) system call to return ECONNREFUSED for NFSv4 mounts (see 2.6.33). For v2/v3, the version and transport negotiation logic in mount.nfs should drive a retry if the server's rpcbind can't be reached. Note that if a v2/v3 mount request encounters an unregistered NFS service, it will still fail immediately. That wouldn't be too hard to change as well, but there are many more corner cases there where failing immediately is appropriate. Signed-off-by: Chuck Lever Signed-off-by: Steve Dickson --- utils/mount/nfs.man | 6 +++++- utils/mount/stropts.c | 4 ++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/utils/mount/nfs.man b/utils/mount/nfs.man index 93bd642..c64de5f 100644 --- a/utils/mount/nfs.man +++ b/utils/mount/nfs.man @@ -363,7 +363,11 @@ The number of minutes that the command retries an NFS mount operation in the foreground or background before giving up. If this option is not specified, the default value for foreground mounts -is 2 minutes, and the default value for background mounts is 10000 minutes (80 minutes shy of one week). +is 2 minutes, and the default value for background mounts is 10000 minutes +(80 minutes shy of one week). +If a value of zero is specified, the +.BR mount (8) +command exits immediately after the first failure. .TP 1.5i .BI sec= mode The RPCGSS security flavor to use for accessing files on this mount point. diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c index 57a4b32..74224ff 100644 --- a/utils/mount/stropts.c +++ b/utils/mount/stropts.c @@ -515,6 +515,10 @@ nfs_rewrite_pmap_mount_options(struct mount_options *options) if (!nfs_probe_bothports(mnt_saddr, mnt_salen, &mnt_pmap, nfs_saddr, nfs_salen, &nfs_pmap)) { errno = ESPIPE; + if (rpc_createerr.cf_stat == RPC_PROGNOTREGISTERED) + errno = EOPNOTSUPP; + else if (rpc_createerr.cf_error.re_errno != 0) + errno = rpc_createerr.cf_error.re_errno; return 0; } -- 2.39.2