libceph: kill bad_proto ceph connection op
authorAlex Elder <elder@inktank.com>
Wed, 30 May 2012 02:47:38 +0000 (21:47 -0500)
committerAlex Elder <elder@dreamhost.com>
Fri, 1 Jun 2012 13:37:55 +0000 (08:37 -0500)
No code sets a bad_proto method in its ceph connection operations
vector, so just get rid of it.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
include/linux/ceph/messenger.h
net/ceph/messenger.c

index aa506cadea672e1fc4816af8a91b068e47c944f5..74f6c9bd80746818a1a3990779668c87633b22b8 100644 (file)
@@ -31,9 +31,6 @@ struct ceph_connection_operations {
        int (*verify_authorizer_reply) (struct ceph_connection *con, int len);
        int (*invalidate_authorizer)(struct ceph_connection *con);
 
-       /* protocol version mismatch */
-       void (*bad_proto) (struct ceph_connection *con);
-
        /* there was some error on the socket (disconnect, whatever) */
        void (*fault) (struct ceph_connection *con);
 
index 42ca8aab6dcf0cc7a3bbedf7432436b6e7c08ef3..07af9948e3f78226ff2a83251a29623a0bce264b 100644 (file)
@@ -1356,11 +1356,6 @@ static void fail_protocol(struct ceph_connection *con)
 {
        reset_connection(con);
        set_bit(CLOSED, &con->state);  /* in case there's queued work */
-
-       mutex_unlock(&con->mutex);
-       if (con->ops->bad_proto)
-               con->ops->bad_proto(con);
-       mutex_lock(&con->mutex);
 }
 
 static int process_connect(struct ceph_connection *con)