Address some unfortunate mount option parsing behavior by setting certain
transport-based defaults *after* option parsing is complete.
o Some options don't obey the "rightmost wins" rule. Jeff Layton
noticed that specifying the "proto=" mount option after the "retrans"
or "timeo" options will cause the retrans and timeo values to be
overwritten with default settings.
Allow these options to be specified in any order without unexpectedly
reverting retrans and timeo to their default.
o I've received several reports that text-based mounting through
firewalls that block UDP fails, even if "proto=tcp" is specified.
If a user specifies "proto=tcp" via the legacy mount API, the mount
command also uses TCP to contact the server's mount daemon. Ditto
for "proto=udp". We want the kernel's mount option to emulate this
behavior; however, we still want the default mount protocol to be UDP
if no transport options were specified.
Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx>
---
fs/nfs/super.c | 49 +++++++++++++++++++++++++++++-------------------
include/linux/nfs_fs.h | 5 +++++
2 files changed, 35 insertions(+), 19 deletions(-)
diff --git a/fs/nfs/super.c b/fs/nfs/super.c
index d59913b..964b17b 100644
--- a/fs/nfs/super.c
+++ b/fs/nfs/super.c
@@ -816,20 +816,14 @@ static int nfs_parse_mount_options(char *raw,
case Opt_udp:
mnt->flags &= ~NFS_MOUNT_TCP;
mnt->nfs_server.protocol = XPRT_TRANSPORT_UDP;
- mnt->timeo = 7;
- mnt->retrans = 5;
break;
case Opt_tcp:
mnt->flags |= NFS_MOUNT_TCP;
mnt->nfs_server.protocol = XPRT_TRANSPORT_TCP;
- mnt->timeo = 600;
- mnt->retrans = 2;
break;
case Opt_rdma:
mnt->flags |= NFS_MOUNT_TCP; /* for side protocols */
mnt->nfs_server.protocol = XPRT_TRANSPORT_RDMA;
- mnt->timeo = 600;
- mnt->retrans = 2;
break;
case Opt_acl:
mnt->flags &= ~NFS_MOUNT_NOACL;
@@ -1023,21 +1017,15 @@ static int nfs_parse_mount_options(char *raw,
case Opt_xprt_udp:
mnt->flags &= ~NFS_MOUNT_TCP;
mnt->nfs_server.protocol = XPRT_TRANSPORT_UDP;
- mnt->timeo = 7;
- mnt->retrans = 5;
break;
case Opt_xprt_tcp:
mnt->flags |= NFS_MOUNT_TCP;
mnt->nfs_server.protocol = XPRT_TRANSPORT_TCP;
- mnt->timeo = 600;
- mnt->retrans = 2;
break;
case Opt_xprt_rdma:
/* vector side protocols to TCP */
mnt->flags |= NFS_MOUNT_TCP;
mnt->nfs_server.protocol = XPRT_TRANSPORT_RDMA;
- mnt->timeo = 600;
- mnt->retrans = 2;
break;
default:
goto out_unrec_xprt;
@@ -1110,6 +1098,36 @@ static int nfs_parse_mount_options(char *raw,
nfs_set_port((struct sockaddr *)&mnt->nfs_server.address,
mnt->nfs_server.port);
+ /* Certain default settings are based on the specified transport */
+ switch (mnt->nfs_server.protocol) {
+ case XPRT_TRANSPORT_UDP:
+ if (mnt->mount_server.protocol == 0)
+ mnt->mount_server.protocol = XPRT_TRANSPORT_UDP;
+ if (mnt->timeo == 0)
+ mnt->timeo = NFS_DEF_UDP_TIMEO;
+ if (mnt->retrans == 0)
+ mnt->retrans = NFS_DEF_UDP_RETRANS;
+ break;
+ case XPRT_TRANSPORT_TCP:
+ case XPRT_TRANSPORT_RDMA:
+ if (mnt->mount_server.protocol == 0)
+ mnt->mount_server.protocol = XPRT_TRANSPORT_TCP;
+ if (mnt->timeo == 0)
+ mnt->timeo = NFS_DEF_TCP_TIMEO;
+ if (mnt->retrans == 0)
+ mnt->retrans = NFS_DEF_TCP_RETRANS;
+ break;
+ default:
+ mnt->nfs_server.protocol = XPRT_TRANSPORT_TCP;
+ if (mnt->mount_server.protocol == 0)
+ mnt->mount_server.protocol = XPRT_TRANSPORT_UDP;
+ if (mnt->timeo == 0)
+ mnt->timeo = NFS_DEF_TCP_TIMEO;
+ if (mnt->retrans == 0)
+ mnt->retrans = NFS_DEF_TCP_RETRANS;
+ break;
+ }
+
return 1;
out_nomem:
@@ -1223,16 +1241,12 @@ static int nfs_validate_mount_data(void *options,
args->flags = (NFS_MOUNT_VER3 | NFS_MOUNT_TCP);
args->rsize = NFS_MAX_FILE_IO_SIZE;
args->wsize = NFS_MAX_FILE_IO_SIZE;
- args->timeo = 600;
- args->retrans = 2;
args->acregmin = 3;
args->acregmax = 60;
args->acdirmin = 30;
args->acdirmax = 60;
args->mount_server.port = 0; /* autobind unless user sets port */
- args->mount_server.protocol = XPRT_TRANSPORT_UDP;
args->nfs_server.port = 0; /* autobind unless user sets port */
- args->nfs_server.protocol = XPRT_TRANSPORT_TCP;
switch (data->version) {
case 1:
@@ -1806,14 +1820,11 @@ static int nfs4_validate_mount_data(void *options,
args->rsize = NFS_MAX_FILE_IO_SIZE;
args->wsize = NFS_MAX_FILE_IO_SIZE;
- args->timeo = 600;
- args->retrans = 2;
args->acregmin = 3;
args->acregmax = 60;
args->acdirmin = 30;
args->acdirmax = 60;
args->nfs_server.port = NFS_PORT; /* 2049 unless user set port= */
- args->nfs_server.protocol = XPRT_TRANSPORT_TCP;
switch (data->version) {
case 1:
diff --git a/include/linux/nfs_fs.h b/include/linux/nfs_fs.h
index 27d6a8d..afc3956 100644
--- a/include/linux/nfs_fs.h
+++ b/include/linux/nfs_fs.h
@@ -12,6 +12,11 @@
#include <linux/magic.h>
/* Default timeout values */
+#define NFS_DEF_UDP_TIMEO (7)
+#define NFS_DEF_UDP_RETRANS (5)
+#define NFS_DEF_TCP_TIMEO (600)
+#define NFS_DEF_TCP_RETRANS (2)
+
#define NFS_MAX_UDP_TIMEOUT (60*HZ)
#define NFS_MAX_TCP_TIMEOUT (600*HZ)
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
