From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chuck Lever Subject: Re: [PATCH] nfs-utils: mount: really return from errno test Date: Thu, 13 Oct 2011 12:37:38 -0400 Message-ID: References: <40D1A40C-BE0C-4AE0-857D-05531615748E@oracle.com> <4E971329.3000007@RedHat.com> Mime-Version: 1.0 (Apple Message framework v1084) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8BIT Cc: Hamo , linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Steve Dickson Return-path: In-Reply-To: <4E971329.3000007-AfCzQyP5zfLQT0dZR+AlfA@public.gmane.org> Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-fsdevel.vger.kernel.org "You are not authorized to access bug #744657." On Oct 13, 2011, at 12:34 PM, Steve Dickson wrote: > Yeah... I believe its this one > https://bugzilla.redhat.com/show_bug.cgi?id=744657 > > On 10/13/2011 10:53 AM, Chuck Lever wrote: >> What was the presenting problem? Is there a bugzilla report I can look at? >> >> On Oct 11, 2011, at 5:44 AM, Hamo wrote: >> >>> We should only try next address family if we meet ECONNREFUSED or EHOSTUNREACH >>> for v4 or ECONNREFUSED or EOPNOTSUPP or EHOSTUNREACH for v3v2. >>> Before, only a break in swich can not make the program out of for loop. >>> >>> Signed-off-by: Yang Bai >>> --- >>> utils/mount/stropts.c | 6 ++++-- >>> 1 files changed, 4 insertions(+), 2 deletions(-) >>> >>> diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c >>> index 314a806..4032bf3 100644 >>> --- a/utils/mount/stropts.c >>> +++ b/utils/mount/stropts.c >>> @@ -665,9 +665,10 @@ static int nfs_try_mount_v3v2(struct nfsmount_info *mi) >>> case EHOSTUNREACH: >>> continue; >>> default: >>> - break; >>> + goto out; >>> } >>> } >>> +out: >>> return ret; >>> } >>> >>> @@ -751,9 +752,10 @@ static int nfs_try_mount_v4(struct nfsmount_info *mi) >>> case EHOSTUNREACH: >>> continue; >>> default: >>> - break; >>> + goto out; >>> } >>> } >>> +out: >>> return ret; >>> } >>> >>> -- >>> 1.7.1 >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >>> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> -- Chuck Lever chuck[dot]lever[at]oracle[dot]com -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html