blob: 5f781ab79592fcf2d438b365955546dcc9682b9c [file] [log] [blame]
From c79173e5cc820921d95ae790ad242567e84bceb7 Mon Sep 17 00:00:00 2001
From: Ronnie Sahlberg <>
Date: Fri, 31 Jan 2020 05:52:51 +1000
Subject: [PATCH] cifs: fix soft mounts hanging in the reconnect code
commit c54849ddd832ae0a45cab16bcd1ed2db7da090d7 upstream.
RHBZ: 1795429
In recent DFS updates we have a new variable controlling how many times we will
retry to reconnect the share.
If DFS is not used, then this variable is initialized to 0 in:
static inline int
dfs_cache_get_nr_tgts(const struct dfs_cache_tgt_list *tl)
return tl ? tl->tl_numtgts : 0;
This means that in the reconnect loop in smb2_reconnect() we will immediately wrap retries to -1
and never actually get to pass this conditional:
if (--retries)
The effect is that we no longer reach the point where we fail the commands with -EHOSTDOWN
and basically the kernel threads are virtually hung and unkillable.
Fixes: a3a53b7603798fd8 (cifs: Add support for failover in smb2_reconnect())
Signed-off-by: Ronnie Sahlberg <>
Signed-off-by: Steve French <>
Reviewed-by: Paulo Alcantara (SUSE) <>
CC: Stable <>
Signed-off-by: Paul Gortmaker <>
diff --git a/fs/cifs/smb2pdu.c b/fs/cifs/smb2pdu.c
index 7dc36e4c462a..5ee890c03699 100644
--- a/fs/cifs/smb2pdu.c
+++ b/fs/cifs/smb2pdu.c
@@ -312,7 +312,7 @@ smb2_reconnect(__le16 smb2_command, struct cifs_tcon *tcon)
if (server->tcpStatus != CifsNeedReconnect)
- if (--retries)
+ if (retries && --retries)