On 4/13/2015 8:16 PM, Benjamin Coddington wrote: > Hi Kinglong, I can't seem to figure out how to reproduce your problem, and > I don't see show this change fixes it. Your mount is falling back to a v3 > attempt without this patch? What's set in nfsmount.conf? Sorry for my missing notes for nfsmount.conf. My nfsmount.conf is default config, besides [ NFSMount_Global_Options ], others line are commented by "#", Defaultvers isn't set as, "# Defaultvers=4" thanks, Kinglong Mee > > Ben > > On Sun, 12 Apr 2015, Kinglong Mee wrote: > >> When mounting nfs with -ov4, -overs=4 or -onfsvers=4, fail as, >> >> # mount -t nfs -ov4 192.168.31.12:/ /testidr/ >> mount.nfs: access denied by server while mounting 192.168.31.12:/ >> >> Fixes: f980298853 "mount.nfs: configurable minor version defaults" >> Signed-off-by: Kinglong Mee <kinglongmee@xxxxxxxxx> >> --- >> utils/mount/stropts.c | 12 +++--------- >> 1 file changed, 3 insertions(+), 9 deletions(-) >> >> diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c >> index df67fac..2ae532e 100644 >> --- a/utils/mount/stropts.c >> +++ b/utils/mount/stropts.c >> @@ -108,12 +108,6 @@ static void nfs_default_version(struct nfsmount_info *mi) >> return; >> } >> >> - if (mi->version.v_mode == V_GENERAL && >> - config_default_vers.v_mode == V_DEFAULT) { >> - mi->version.v_mode = V_SPECIFIC; >> - return; >> - } >> - >> if (mi->version.v_mode == V_DEFAULT && >> config_default_vers.v_mode != V_DEFAULT) { >> mi->version.major = config_default_vers.major; >> @@ -121,9 +115,9 @@ static void nfs_default_version(struct nfsmount_info *mi) >> return; >> } >> >> - if (mi->version.v_mode == V_GENERAL && >> - config_default_vers.v_mode != V_DEFAULT) { >> - if (mi->version.major == config_default_vers.major) >> + if (mi->version.v_mode == V_GENERAL) { >> + if (config_default_vers.v_mode != V_DEFAULT && >> + mi->version.major == config_default_vers.major) >> mi->version.minor = config_default_vers.minor; >> return; >> } >> -- >> 2.3.5 >> >> > -- 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