diff options
| author |   Shyam Prasad N <sprasad@microsoft.com> | 2022-01-16 13:28:34 +0000 |
|---|---|---|
| committer |   Steve French <stfrench@microsoft.com> | 2022-01-19 11:10:55 -0600 |
| commit | ece0767641740c7eea7aee5a332728e115b00eab (patch) | |
| tree | bd652a7936713290f89fc9bfc6bdac1eb36731c6 /fs/cifs/connect.c | |
| parent | cifs: fix the cifs_reconnect path for DFS (diff) | |
| download | linux-dev-ece0767641740c7eea7aee5a332728e115b00eab.tar.xz linux-dev-ece0767641740c7eea7aee5a332728e115b00eab.zip | |
cifs: remove repeated state change in dfs tree connect
cifs_tree_connect checks and sets the tidStatus for the tcon.
cifs_tree_connect also calls a dfs specific tree connect
function, which also does similar checks. This should
not happen. Removing it with this change.
Signed-off-by: Shyam Prasad N <sprasad@microsoft.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifs/connect.c')
| -rw-r--r-- | fs/cifs/connect.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index 4a31a9b3f34f..6740a7c39df3 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -4295,16 +4295,6 @@ static int __tree_connect_dfs_target(const unsigned int xid, struct cifs_tcon *t struct dfs_cache_tgt_iterator *tit; bool target_match; - /* only send once per connect */ - spin_lock(&cifs_tcp_ses_lock); - if (tcon->tidStatus != CifsNew && - tcon->tidStatus != CifsNeedTcon) { - spin_unlock(&cifs_tcp_ses_lock); - return 0; - } - tcon->tidStatus = CifsInTcon; - spin_unlock(&cifs_tcp_ses_lock); - extract_unc_hostname(server->hostname, &tcp_host, &tcp_host_len); tit = dfs_cache_get_tgt_iterator(tl); |