Bug 171712
Summary: | A NFS export mounted using version 4 and TCP shows up as UDP in /proc/mounts | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 4 | Reporter: | Vinu K <vkanakas> |
Component: | kernel | Assignee: | Jeff Layton <jlayton> |
Status: | CLOSED ERRATA | QA Contact: | Brian Brock <bbrock> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 4.0 | CC: | coughlan, jbaron, staubach, steved |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | RHSA-2008-0665 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-07-24 19:10:50 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 430698 | ||
Attachments: |
Description
Vinu K
2005-10-25 15:13:20 UTC
Created attachment 120611 [details]
Proposed patch
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. 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...
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 release. 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).
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 "proto=udp". 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.
QE ack for RHEL 4.7 Committed in 68.8 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. http://rhn.redhat.com/errata/RHSA-2008-0665.html |