cifs: Fix potential softlockups while refreshing DFS cache
authorPaulo Alcantara (SUSE) <pc@cjr.nz>
Fri, 22 Nov 2019 15:30:53 +0000 (12:30 -0300)
committerSteve French <stfrench@microsoft.com>
Mon, 25 Nov 2019 15:33:04 +0000 (09:33 -0600)
commit84a1f5b1cc6fd7f6cd99fc5630c36f631b19fa60
tree58346cce7b4f68ef8dc41b39c09c9885436750c4
parentdf3df923b31d298c3d3653a0380202b9f2df9864
cifs: Fix potential softlockups while refreshing DFS cache

We used to skip reconnects on all SMB2_IOCTL commands due to SMB3+
FSCTL_VALIDATE_NEGOTIATE_INFO - which made sense since we're still
establishing a SMB session.

However, when refresh_cache_worker() calls smb2_get_dfs_refer() and
we're under reconnect, SMB2_ioctl() will not be able to get a proper
status error (e.g. -EHOSTDOWN in case we failed to reconnect) but an
-EAGAIN from cifs_send_recv() thus looping forever in
refresh_cache_worker().

Fixes: e99c63e4d86d ("SMB3: Fix deadlock in validate negotiate hits reconnect")
Signed-off-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Suggested-by: Aurelien Aptel <aaptel@suse.com>
Reviewed-by: Aurelien Aptel <aaptel@suse.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/cifs/smb2pdu.c