diff options
| author |   Steve French <stfrench@microsoft.com> | 2021-11-15 21:00:08 -0600 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2021-11-16 10:46:09 -0600 |
| commit | 0226487ad8146433be74d20955448e12d85fd251 (patch) | |
| tree | 9e735ee6d075008838385eeb532d2cacf824f76e /fs/cifs | |
| parent | Linux 5.16-rc1 (diff) | |
| download | linux-dev-0226487ad8146433be74d20955448e12d85fd251.tar.xz linux-dev-0226487ad8146433be74d20955448e12d85fd251.zip | |
cifs: move debug print out of spinlock
It is better to print debug messages outside of the chan_lock
spinlock where possible.
Reviewed-by: Shyam Prasad N <sprasad@microsoft.com>
Addresses-Coverity: 1493854 ("Thread deadlock")
Reviewed-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs')
| -rw-r--r-- | fs/cifs/sess.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/sess.c b/fs/cifs/sess.c index 2c10b186ed6e..7db8b22edac9 100644 --- a/fs/cifs/sess.c +++ b/fs/cifs/sess.c @@ -95,9 +95,9 @@ int cifs_try_adding_channels(struct cifs_sb_info *cifs_sb, struct cifs_ses *ses) } if (!(ses->server->capabilities & SMB2_GLOBAL_CAP_MULTI_CHANNEL)) { - cifs_dbg(VFS, "server %s does not support multichannel\n", ses->server->hostname); ses->chan_max = 1; spin_unlock(&ses->chan_lock); + cifs_dbg(VFS, "server %s does not support multichannel\n", ses->server->hostname); return 0; } spin_unlock(&ses->chan_lock); |