smb: client: Fix hang in smb2_reconnect
Test case:
mount -t cifs //192.168.0.1/test y -o
port=19999,ro,vers=2.1,sec=none,echo_interval=1
kill smbd with SIGSTOP
umount /tmp/y
Gives the following error:
INFO: task umount:466 blocked for more than 122 seconds.
Not tainted 6.8.2-axis9-devel #1
"echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this
message.
task:umount state:D stack:0 pid:466 tgid:466 ppid:464 flags:0x00000004
__schedule from schedule+0x34/0x108
schedule from schedule_preempt_disabled+0x24/0x34
schedule_preempt_disabled from __mutex_lock.constprop.0+0x330/0x8b0
__mutex_lock.constprop.0 from smb2_reconnect+0x278/0x8fc [cifs]
smb2_reconnect [cifs] from SMB2_open_init+0x54/0x9f4 [cifs]
SMB2_open_init [cifs] from smb2_query_info_compound+0x1a0/0x500[cifs]
smb2_query_info_compound [cifs] from smb2_queryfs+0x64/0x134 [cifs]
smb2_queryfs [cifs] from cifs_statfs+0xc8/0x318 [cifs]
cifs_statfs [cifs] from statfs_by_dentry+0x60/0x84
statfs_by_dentry from fd_statfs+0x30/0x74
fd_statfs from sys_fstatfs64+0x40/0x6c
sys_fstatfs64 from ret_fast_syscall+0x0/0x54
The umount task is blocked waiting on the session mutex. The reason it
never gets the session mutex is because 'kworker/0:3' is holding the
mutex and is waiting for response (see line 1209 in
fs/smb/client/smb2pdu.c.
Stack trace of 'kworker/0:3' just before calling wait_for_response:
CPU: 0 PID: 220 Comm: kworker/0:3 Not tainted 6.8.2-axis9-devel #1
Hardware name: Freescale i.MX6 SoloX (Device Tree)
Workqueue: cifsiod smb2_reconnect_server [cifs]
unwind_backtrace from show_stack+0x18/0x1c
show_stack from dump_stack_lvl+0x24/0x2c
dump_stack_lvl from compound_send_recv+0x7bc/0xac8 [cifs]
compound_send_recv [cifs] from cifs_send_recv+0x34/0x3c [cifs]
cifs_send_recv [cifs] from SMB2_negotiate+0x410/0x13dc [cifs]
SMB2_negotiate [cifs] from smb2_negotiate+0x4c/0x58 [cifs]
smb2_negotiate [cifs] from cifs_negotiate_protocol+0x9c/0x100 [cifs]
cifs_negotiate_protocol [cifs] from smb2_reconnect+0x418/0x8fc [cifs]
smb2_reconnect [cifs] from smb2_reconnect_server+0x1dc/0x514 [cifs]
smb2_reconnect_server [cifs] from process_one_work+0x188/0x3ec
process_one_work from worker_thread+0x1fc/0x430
worker_thread from kthread+0x110/0x130
kthread from ret_from_fork+0x14/0x28
Cc: stable@vger.kernel.org
Signed-off-by: Rickard x Andersson <rickaran@axis.com>
Signed-off-by: Steve French <stfrench@microsoft.com>