From mboxrd@z Thu Jan 1 00:00:00 1970 From: Suresh Jayaraman Subject: Re: [PATCH] cifs: handle cifs_get_tcon() errors properly Date: Mon, 15 Nov 2010 22:21:48 +0530 Message-ID: <4CE16524.10307@suse.de> References: <1289825123-13716-1-git-send-email-sjayaraman@suse.de> <20101115082535.0d7200a5@tlielax.poochiereds.net> <4CE149B1.5090901@suse.de> <20101115102843.6f425b27@tlielax.poochiereds.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: Steve French , linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jeff Layton Return-path: In-Reply-To: <20101115102843.6f425b27-9yPaYZwiELC+kQycOl6kW4xkIHaj4LzF@public.gmane.org> Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On 11/15/2010 08:58 PM, Jeff Layton wrote: > On Mon, 15 Nov 2010 20:24:41 +0530 > Suresh Jayaraman wrote: > >> I agree that this is not an improvement (more of a cleanup), but I think >> it improves readability (given that the single function runs for more than >> hundred lines), and skips a couple of unneeded checks. >> >> Here's a fixed version (but this patch can be ignored if we feel so): >> >> >> cifs_get_tcon() could return any of the following errors: >> -ENOMEM/-ENODEV/EREMOTEIO. We should follow the DFS referral code path only >> when we get -EREMOTEIO (STATUS_PATH_NOT_COVERED) from the server and not >> otherwise. >> >> Compile-tested only. >> >> Signed-off-by: Suresh Jayaraman >> --- >> fs/cifs/connect.c | 5 ++++- >> 1 files changed, 4 insertions(+), 1 deletions(-) >> >> diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c >> index 251a17c..6dbc145 100644 >> --- a/fs/cifs/connect.c >> +++ b/fs/cifs/connect.c >> @@ -2780,7 +2780,10 @@ try_mount_again: >> if (IS_ERR(tcon)) { >> rc = PTR_ERR(tcon); >> tcon = NULL; >> - goto remote_path_check; >> + if (rc == -EREMOTE) >> + goto remote_path_check; >> + else >> + goto mount_fail_check; >> } >> >> /* do not care if following two calls succeed - informational */ > > I just don't the point here. cifs_mount is quite messy, but I don't > think this really does much to improve its readability. > Ok, let's drop this patch.. -- Suresh Jayaraman