diff options
| author | Shyam Prasad N <sprasad@microsoft.com> | 2024-01-21 04:32:45 +0100 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2024-01-24 03:23:28 +0100 |
| commit | a68106a6928e0a6680f12bcc7338c0dddcfe4d11 (patch) | |
| tree | 5ca2fd8ebc1c4860b2a244b42c67a98bc946ba0d /fs | |
| parent | cifs: cifs_pick_channel should try selecting active channels (diff) | |
| download | linux-a68106a6928e0a6680f12bcc7338c0dddcfe4d11.tar.xz linux-a68106a6928e0a6680f12bcc7338c0dddcfe4d11.zip | |
cifs: translate network errors on send to -ECONNABORTED
When the network stack returns various errors, we today bubble
up the error to the user (in case of soft mounts).
This change translates all network errors except -EINTR and
-EAGAIN to -ECONNABORTED. A similar approach is taken when
we receive network errors when reading from the socket.
The change also forces the cifsd thread to reconnect during
it's next activity.
Signed-off-by: Shyam Prasad N <sprasad@microsoft.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/smb/client/transport.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/fs/smb/client/transport.c b/fs/smb/client/transport.c index 8695c9961f5a..e00278fcfa4f 100644 --- a/fs/smb/client/transport.c +++ b/fs/smb/client/transport.c @@ -400,10 +400,17 @@ unmask: server->conn_id, server->hostname); } smbd_done: - if (rc < 0 && rc != -EINTR) + /* + * there's hardly any use for the layers above to know the + * actual error code here. All they should do at this point is + * to retry the connection and hope it goes away. + */ + if (rc < 0 && rc != -EINTR && rc != -EAGAIN) { cifs_server_dbg(VFS, "Error %d sending data on socket to server\n", rc); - else if (rc > 0) + rc = -ECONNABORTED; + cifs_signal_cifsd_for_reconnect(server, false); + } else if (rc > 0) rc = 0; out: cifs_in_send_dec(server); |