merged
On Mon, Feb 7, 2011 at 7:54 AM, Jeff Layton <jlayton@xxxxxxxxxx> wrote:
> ses->status is never set to CifsExiting, so these checks are
> always false.
>
> Tested-by: JG <jg@xxxxxx>
> Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>
> ---
> fs/cifs/cifssmb.c | 5 +----
> 1 files changed, 1 insertions(+), 4 deletions(-)
>
> diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c
> index 46c66ed..904aa47 100644
> --- a/fs/cifs/cifssmb.c
> +++ b/fs/cifs/cifssmb.c
> @@ -136,9 +136,6 @@ cifs_reconnect_tcon(struct cifsTconInfo *tcon, int smb_command)
> }
> }
>
> - if (ses->status == CifsExiting)
> - return -EIO;
> -
> /*
> * Give demultiplex thread up to 10 seconds to reconnect, should be
> * greater than cifs socket timeout which is 7 seconds
> @@ -156,7 +153,7 @@ cifs_reconnect_tcon(struct cifsTconInfo *tcon, int smb_command)
> * retrying until process is killed or server comes
> * back on-line
> */
> - if (!tcon->retry || ses->status == CifsExiting) {
> + if (!tcon->retry) {
> cFYI(1, "gave up waiting on reconnect in smb_init");
> return -EHOSTDOWN;
> }
> --
> 1.7.4
>
>
--
Thanks,
Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
