Bug 171712 - A NFS export mounted using version 4 and TCP shows up as UDP in /proc/mounts
Summary: A NFS export mounted using version 4 and TCP shows up as UDP in /proc/mounts
Alias: None
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: kernel
Version: 4.0
Hardware: All
OS: Linux
Target Milestone: ---
: ---
Assignee: Jeff Layton
QA Contact: Brian Brock
Depends On:
Blocks: 430698
TreeView+ depends on / blocked
Reported: 2005-10-25 15:13 UTC by Vinu K
Modified: 2018-10-19 23:05 UTC (History)
4 users (show)

Clone Of:
Last Closed: 2008-07-24 19:10:50 UTC

Attachments (Terms of Use)
Proposed patch (1.03 KB, patch)
2005-11-01 16:49 UTC, Peter Staubach
no flags Details | Diff
upstream kernel patch -- show correct options for nfsv4 (1.52 KB, patch)
2007-07-20 11:36 UTC, Jeff Layton
no flags Details | Diff
patch -- remove redundant protocol option from /proc/mounts (928 bytes, patch)
2007-07-23 11:54 UTC, Jeff Layton
no flags Details | Diff
updated patch -- add the option back, but correct it (1.49 KB, patch)
2007-09-17 13:11 UTC, Jeff Layton
no flags Details | Diff

External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2008:0665 normal SHIPPED_LIVE Moderate: Updated kernel packages for Red Hat Enterprise Linux 4.7 2008-07-24 16:41:06 UTC

Description Vinu K 2005-10-25 15:13:20 UTC
Description of problem:

A NFS export mounted using version 4 and TCP shows up as UDP in /proc/mounts

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:

[root@dhcp37-72 tmp]# uname -r
[root@dhcp37-72 tmp]# mount -t nfs4 -o proto=tcp dhcp37-74:/ rhel-4/
[root@dhcp37-72 tmp]# mount -t nfs4
dhcp37-74:/ on /tmp/rhel-4 type nfs4 (rw,proto=tcp,addr=
[root@dhcp37-72 tmp]# grep nfs4 /proc/mounts
dhcp37-74:/ /tmp/rhel-4 nfs4
rw,v4,rsize=32768,wsize=32768,hard,intr,udp,nolock,addr=dhcp37-74 0 0
[root@dhcp37-72 tmp]# netstat -tn
Active Internet connections (w/o servers)
Proto Recv-Q Send-Q Local Address               Foreign Address             State
tcp        0      0            

Actual results:
/proc/mounts shows that UDP is used

Expected results:
/proc/mounts should show TCP

Additional info:

Comment 3 Peter Staubach 2005-11-01 16:49:09 UTC
Created attachment 120611 [details]
Proposed patch

Comment 5 Jeff Layton 2007-07-20 11:32:55 UTC
Actually, it looks like upstream fixed this with a kernel patch. We have most of
the patch in RHEL4 already, but are missing the delta that removes ",udp" from
the output. Too late for 4.6, we'll try for 4.7.

Comment 6 Jeff Layton 2007-07-20 11:36:40 UTC
Created attachment 159643 [details]
upstream kernel patch -- show correct options for nfsv4

This is the patch that went upstream. We already have all but the first delta
that removes this line:

		{ NFS_MOUNT_TCP, ",tcp", ",udp" },

The question is, will removing this break anything in userspace? I tend to
think not, but maybe it's better to just fix it so that these strings are still
present, but correct...

Comment 7 RHEL Product and Program Management 2007-07-20 11:38:26 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release.  Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products.  This request is not yet committed for inclusion in an Update

Comment 8 Jeff Layton 2007-07-23 11:54:07 UTC
Created attachment 159776 [details]
patch -- remove redundant protocol option from /proc/mounts

This pulls in the missing delta from the earlier patch. This would bring us
into line with upstream, but I suppose there's a chance that this could break
applications that are looking for 'tcp' or 'udp' alone as an option.

I'll propose this first, but if it's deemed to risky I'll fall back and change
it to keep the old option (but fix it).

Comment 9 Jeff Layton 2007-07-31 15:13:48 UTC
Looks like umount actually looks for a 'tcp' or 'udp' mount opt. Before I can
propose this I'll need to check that hasmntopt(...,"udp") will still match for

Comment 10 Jeff Layton 2007-09-17 13:11:57 UTC
Created attachment 197391 [details]
updated patch -- add the option back, but correct it

This patch is similar to the last one, but also adds an extra ,tcp or ,udp
option in addition to the proto= option. At this late date, I'd hate to break
anything in userspace that's depending on seeing this option in /proc/mounts
and correcting it is fairly trivial.

Comment 14 Martin Jenner 2008-01-16 21:09:01 UTC
QE ack for RHEL 4.7

Comment 15 Vivek Goyal 2008-01-23 00:13:50 UTC
Committed in 68.8

Comment 19 errata-xmlrpc 2008-07-24 19:10:50 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.


Note You need to log in before you can comment on or make changes to this bug.