diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2007-08-29 17:58:57 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2007-09-01 10:14:38 -0400 |
| commit | fdb66ff4ace3c4e18809b9b71dfcce1692143147 (patch) | |
| tree | 8e7db455c604ce2cfe27dcf4332fca8f4c9f8b46 | |
| parent | deee9369b993b52a8e2f25683b4a67be7a65e8ae (diff) | |
NFS: mount option parser chokes on proto=
The new text-based NFS mount option parsing logic doesn't recognize any
valid transport protocols due to a silly mistake in the protocol token
matching logic. This prevents basic mount requests such as:
mount.nfs server:/export /mnt -o proto=tcp
from working with the new text-based NFS mount API.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
| -rw-r--r-- | fs/nfs/super.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fs/nfs/super.c b/fs/nfs/super.c index 46139003ea0c..66a223b22a80 100644 --- a/fs/nfs/super.c +++ b/fs/nfs/super.c | |||
| @@ -911,13 +911,13 @@ static int nfs_parse_mount_options(char *raw, | |||
| 911 | kfree(string); | 911 | kfree(string); |
| 912 | 912 | ||
| 913 | switch (token) { | 913 | switch (token) { |
| 914 | case Opt_udp: | 914 | case Opt_xprt_udp: |
| 915 | mnt->flags &= ~NFS_MOUNT_TCP; | 915 | mnt->flags &= ~NFS_MOUNT_TCP; |
| 916 | mnt->nfs_server.protocol = IPPROTO_UDP; | 916 | mnt->nfs_server.protocol = IPPROTO_UDP; |
| 917 | mnt->timeo = 7; | 917 | mnt->timeo = 7; |
| 918 | mnt->retrans = 5; | 918 | mnt->retrans = 5; |
| 919 | break; | 919 | break; |
| 920 | case Opt_tcp: | 920 | case Opt_xprt_tcp: |
| 921 | mnt->flags |= NFS_MOUNT_TCP; | 921 | mnt->flags |= NFS_MOUNT_TCP; |
| 922 | mnt->nfs_server.protocol = IPPROTO_TCP; | 922 | mnt->nfs_server.protocol = IPPROTO_TCP; |
| 923 | mnt->timeo = 600; | 923 | mnt->timeo = 600; |
| @@ -936,10 +936,10 @@ static int nfs_parse_mount_options(char *raw, | |||
| 936 | kfree(string); | 936 | kfree(string); |
| 937 | 937 | ||
| 938 | switch (token) { | 938 | switch (token) { |
| 939 | case Opt_udp: | 939 | case Opt_xprt_udp: |
| 940 | mnt->mount_server.protocol = IPPROTO_UDP; | 940 | mnt->mount_server.protocol = IPPROTO_UDP; |
| 941 | break; | 941 | break; |
| 942 | case Opt_tcp: | 942 | case Opt_xprt_tcp: |
| 943 | mnt->mount_server.protocol = IPPROTO_TCP; | 943 | mnt->mount_server.protocol = IPPROTO_TCP; |
| 944 | break; | 944 | break; |
| 945 | default: | 945 | default: |
