Return-Path: Message-ID: <554A191A.6090301@gmail.com> Date: Wed, 06 May 2015 21:37:30 +0800 From: Kinglong Mee MIME-Version: 1.0 To: Steve Dickson CC: Benjamin Coddington , "linux-nfs@vger.kernel.org" Subject: Re: [PATCH] mount: Writes signal number to kernel as command line References: <552BB264.8010104@gmail.com> In-Reply-To: Content-Type: text/plain; charset=utf-8 List-ID: On 4/13/2015 8:53 PM, Benjamin Coddington wrote: > On Mon, 13 Apr 2015, Kinglong Mee wrote: > >> When mounting nfs with -overs=4,minorversion=2, want getting >> nfs mounts with vers=4.2, but got vers=4.0 as, >> >> # mount -t nfs -onfsvers=4,minorversion=2 127.0.0.1:/ /mnt/ >> # cat /proc/mounts | grep vers >> 127.0.0.1:/ /mnt nfs4 rw,relatime,vers=4.0,rsize=262144,wsize=262144,namlen=255,hard,proto=tcp,timeo=600,retrans=2,sec=sys,clientaddr=127.0.0.1,local_lock=none,addr=127.0.0.1 0 0 >> >> It's caused by mount.nfs writing bad vers to kernel. This patch >> lets mount.nfs writing signal number to kernel as command line. >> >> Noteļ¼š This patch is based on my last patch, >> "mount: make sure mounting nfs with v4,vers=4 and nfsvers=4" >> >> Signed-off-by: Kinglong Mee > > Looks good. > > Reviewed-by: Benjamin Coddington Add this one? thanks, Kinglong Mee >> --- >> utils/mount/stropts.c | 9 +++++++-- >> 1 file changed, 7 insertions(+), 2 deletions(-) >> >> diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c >> index 2ae532e..c8f5a6d 100644 >> --- a/utils/mount/stropts.c >> +++ b/utils/mount/stropts.c >> @@ -737,8 +737,13 @@ static int nfs_do_mount_v4(struct nfsmount_info *mi, >> } >> >> if (mi->version.v_mode != V_SPECIFIC) { >> - snprintf(version_opt, sizeof(version_opt) - 1, >> - "vers=%lu.%lu", mi->version.major, mi->version.minor); >> + if (mi->version.v_mode == V_GENERAL) >> + snprintf(version_opt, sizeof(version_opt) - 1, >> + "vers=%lu", mi->version.major); >> + else >> + snprintf(version_opt, sizeof(version_opt) - 1, >> + "vers=%lu.%lu", mi->version.major, >> + mi->version.minor); >> >> if (po_append(options, version_opt) == PO_FAILED) { >> errno = EINVAL; >> -- >> 2.3.5 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >