libceph: ceph_connection_operations::reencode_message() method
authorIlya Dryomov <idryomov@gmail.com>
Thu, 15 Jun 2017 14:30:54 +0000 (16:30 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Fri, 7 Jul 2017 15:25:15 +0000 (17:25 +0200)
Give upper layers a chance to reencode the message after the connection
is negotiated and ->peer_features is set.  OSD client will use this to
support both luminous and pre-luminous OSDs (in a single cluster): the
former need MOSDOp v8; the latter will continue to be sent MOSDOp v4.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
include/linux/ceph/messenger.h
net/ceph/messenger.c

index c5c4c713e00f52640366014be16d32cb79bc2181..fbd94d9fa5dd4fdcc760a677faf85276cefa6cca 100644 (file)
@@ -44,6 +44,8 @@ struct ceph_connection_operations {
                                        struct ceph_msg_header *hdr,
                                        int *skip);
 
+       void (*reencode_message) (struct ceph_msg *msg);
+
        int (*sign_message) (struct ceph_msg *msg);
        int (*check_message_signature) (struct ceph_msg *msg);
 };
index 9daed25406390ac491d8fd2017f5675b3242a824..0c31035bbfee8cd682ccf63b3e00728b6252f545 100644 (file)
@@ -1288,13 +1288,16 @@ static void prepare_write_message(struct ceph_connection *con)
                m->hdr.seq = cpu_to_le64(++con->out_seq);
                m->needs_out_seq = false;
        }
-       WARN_ON(m->data_length != le32_to_cpu(m->hdr.data_len));
+
+       if (con->ops->reencode_message)
+               con->ops->reencode_message(m);
 
        dout("prepare_write_message %p seq %lld type %d len %d+%d+%zd\n",
             m, con->out_seq, le16_to_cpu(m->hdr.type),
             le32_to_cpu(m->hdr.front_len), le32_to_cpu(m->hdr.middle_len),
             m->data_length);
-       BUG_ON(le32_to_cpu(m->hdr.front_len) != m->front.iov_len);
+       WARN_ON(m->front.iov_len != le32_to_cpu(m->hdr.front_len));
+       WARN_ON(m->data_length != le32_to_cpu(m->hdr.data_len));
 
        /* tag + hdr + front + middle */
        con_out_kvec_add(con, sizeof (tag_msg), &tag_msg);