libceph: don't call reset_connection() on version/feature mismatches
authorIlya Dryomov <idryomov@gmail.com>
Fri, 6 Nov 2020 16:09:24 +0000 (17:09 +0100)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 14 Dec 2020 22:21:48 +0000 (23:21 +0100)
commit90b6561a0525d0888d5d705e343bacaaacd3c021
tree807c943c431199006d00447917cbaeb9483e0ee3
parent418af5b3bfc4f1ef4854e83c5be8a0bdce51e95c
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>
net/ceph/messenger.c