summaryrefslogtreecommitdiffstats
path: root/net/ceph
diff options
context:
space:
mode:
authorIlya Dryomov <idryomov@gmail.com>2020-11-06 17:09:24 +0100
committerIlya Dryomov <idryomov@gmail.com>2020-12-14 23:21:48 +0100
commit90b6561a0525d0888d5d705e343bacaaacd3c021 (patch)
tree807c943c431199006d00447917cbaeb9483e0ee3 /net/ceph
parent418af5b3bfc4f1ef4854e83c5be8a0bdce51e95c (diff)
downloadlinux-90b6561a0525d0888d5d705e343bacaaacd3c021.tar.gz
linux-90b6561a0525d0888d5d705e343bacaaacd3c021.tar.bz2
linux-90b6561a0525d0888d5d705e343bacaaacd3c021.zip
libceph: don't call reset_connection() on version/feature mismatches
A fault due to a version mismatch or a feature set mismatch used to be treated differently from other faults: the connection would get closed without trying to reconnect and there was a ->bad_proto() connection op for notifying about that. This changed a long time ago, see commits 6384bb8b8e88 ("libceph: kill bad_proto ceph connection op") and 0fa6ebc600bc ("libceph: fix protocol feature mismatch failure path"). Nowadays these aren't any different from other faults (i.e. we try to reconnect even though the mismatch won't resolve until the server is replaced). reset_connection() calls there are rather confusing because reset_connection() resets a session together an individual instance of the protocol. This is cleaned up in the next patch. Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'net/ceph')
-rw-r--r--net/ceph/messenger.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c
index f3eb66bab988..a73525de7b70 100644
--- a/net/ceph/messenger.c
+++ b/net/ceph/messenger.c
@@ -2091,7 +2091,6 @@ static int process_connect(struct ceph_connection *con)
ceph_pr_addr(&con->peer_addr),
sup_feat, server_feat, server_feat & ~sup_feat);
con->error_msg = "missing required protocol features";
- reset_connection(con);
return -1;
case CEPH_MSGR_TAG_BADPROTOVER:
@@ -2102,7 +2101,6 @@ static int process_connect(struct ceph_connection *con)
le32_to_cpu(con->out_connect.protocol_version),
le32_to_cpu(con->in_reply.protocol_version));
con->error_msg = "protocol version mismatch";
- reset_connection(con);
return -1;
case CEPH_MSGR_TAG_BADAUTHORIZER:
@@ -2192,7 +2190,6 @@ static int process_connect(struct ceph_connection *con)
ceph_pr_addr(&con->peer_addr),
req_feat, server_feat, req_feat & ~server_feat);
con->error_msg = "missing required protocol features";
- reset_connection(con);
return -1;
}