tentatively merged into cifs-2.6.git for-next pending more review and testing (and temporarily backed out the larger cifs netfs integration patches series) On Mon, Jan 22, 2024 at 11:08 PM <nspmangalore@xxxxxxxxx> wrote: > > From: Shyam Prasad N <sprasad@xxxxxxxxxxxxx> > > A recent change moved the code that decides to skip > a channel or disable multichannel entirely, into a > helper function. > > During this, a mutex_unlock of the session_mutex > should have been removed. Doing that here. > > Fixes: f591062bdbf4 ("cifs: handle servers that still advertise multichannel after disabling") > Signed-off-by: Shyam Prasad N <sprasad@xxxxxxxxxxxxx> > --- > fs/smb/client/smb2pdu.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/fs/smb/client/smb2pdu.c b/fs/smb/client/smb2pdu.c > index 4f2cc8373b67..86f6f35b7f32 100644 > --- a/fs/smb/client/smb2pdu.c > +++ b/fs/smb/client/smb2pdu.c > @@ -195,7 +195,6 @@ cifs_chan_skip_or_disable(struct cifs_ses *ses, > pserver = server->primary_server; > cifs_signal_cifsd_for_reconnect(pserver, false); > skip_terminate: > - mutex_unlock(&ses->session_mutex); > return -EHOSTDOWN; > } > > -- > 2.34.1 > -- Thanks, Steve