mirror of
https://github.com/torvalds/linux.git
synced 2024-11-30 08:01:59 +00:00
cifs: Re-indent cifs_swn_reconnect()
This code is slightly nicer if we flip the cifs_sockaddr_equal() around and pull all the code in one tab. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Samuel Cabrero <scabrero@suse.de> Signed-off-by: Steve French <stfrench@microsoft.com>
This commit is contained in:
parent
eedf8e88e5
commit
0f2c66ae5c
@ -484,41 +484,43 @@ static int cifs_swn_reconnect(struct cifs_tcon *tcon, struct sockaddr_storage *a
|
|||||||
|
|
||||||
/* Store the reconnect address */
|
/* Store the reconnect address */
|
||||||
mutex_lock(&tcon->ses->server->srv_mutex);
|
mutex_lock(&tcon->ses->server->srv_mutex);
|
||||||
if (!cifs_sockaddr_equal(&tcon->ses->server->dstaddr, addr)) {
|
if (cifs_sockaddr_equal(&tcon->ses->server->dstaddr, addr))
|
||||||
ret = cifs_swn_store_swn_addr(addr, &tcon->ses->server->dstaddr,
|
goto unlock;
|
||||||
&tcon->ses->server->swn_dstaddr);
|
|
||||||
if (ret < 0) {
|
|
||||||
cifs_dbg(VFS, "%s: failed to store address: %d\n", __func__, ret);
|
|
||||||
goto unlock;
|
|
||||||
}
|
|
||||||
tcon->ses->server->use_swn_dstaddr = true;
|
|
||||||
|
|
||||||
/*
|
ret = cifs_swn_store_swn_addr(addr, &tcon->ses->server->dstaddr,
|
||||||
* Unregister to stop receiving notifications for the old IP address.
|
&tcon->ses->server->swn_dstaddr);
|
||||||
*/
|
if (ret < 0) {
|
||||||
ret = cifs_swn_unregister(tcon);
|
cifs_dbg(VFS, "%s: failed to store address: %d\n", __func__, ret);
|
||||||
if (ret < 0) {
|
goto unlock;
|
||||||
cifs_dbg(VFS, "%s: Failed to unregister for witness notifications: %d\n",
|
|
||||||
__func__, ret);
|
|
||||||
goto unlock;
|
|
||||||
}
|
|
||||||
|
|
||||||
/*
|
|
||||||
* And register to receive notifications for the new IP address now that we have
|
|
||||||
* stored the new address.
|
|
||||||
*/
|
|
||||||
ret = cifs_swn_register(tcon);
|
|
||||||
if (ret < 0) {
|
|
||||||
cifs_dbg(VFS, "%s: Failed to register for witness notifications: %d\n",
|
|
||||||
__func__, ret);
|
|
||||||
goto unlock;
|
|
||||||
}
|
|
||||||
|
|
||||||
spin_lock(&GlobalMid_Lock);
|
|
||||||
if (tcon->ses->server->tcpStatus != CifsExiting)
|
|
||||||
tcon->ses->server->tcpStatus = CifsNeedReconnect;
|
|
||||||
spin_unlock(&GlobalMid_Lock);
|
|
||||||
}
|
}
|
||||||
|
tcon->ses->server->use_swn_dstaddr = true;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Unregister to stop receiving notifications for the old IP address.
|
||||||
|
*/
|
||||||
|
ret = cifs_swn_unregister(tcon);
|
||||||
|
if (ret < 0) {
|
||||||
|
cifs_dbg(VFS, "%s: Failed to unregister for witness notifications: %d\n",
|
||||||
|
__func__, ret);
|
||||||
|
goto unlock;
|
||||||
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* And register to receive notifications for the new IP address now that we have
|
||||||
|
* stored the new address.
|
||||||
|
*/
|
||||||
|
ret = cifs_swn_register(tcon);
|
||||||
|
if (ret < 0) {
|
||||||
|
cifs_dbg(VFS, "%s: Failed to register for witness notifications: %d\n",
|
||||||
|
__func__, ret);
|
||||||
|
goto unlock;
|
||||||
|
}
|
||||||
|
|
||||||
|
spin_lock(&GlobalMid_Lock);
|
||||||
|
if (tcon->ses->server->tcpStatus != CifsExiting)
|
||||||
|
tcon->ses->server->tcpStatus = CifsNeedReconnect;
|
||||||
|
spin_unlock(&GlobalMid_Lock);
|
||||||
|
|
||||||
unlock:
|
unlock:
|
||||||
mutex_unlock(&tcon->ses->server->srv_mutex);
|
mutex_unlock(&tcon->ses->server->srv_mutex);
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user