libceph: initialize data fields on last msg put
authorAlex Elder <elder@inktank.com>
Mon, 25 Mar 2013 16:54:30 +0000 (11:54 -0500)
committerSage Weil <sage@inktank.com>
Thu, 2 May 2013 04:17:22 +0000 (21:17 -0700)
When the last reference to a ceph message is dropped,
ceph_msg_last_put() is called to clean things up.

For "normal" messages (allocated via ceph_msg_new() rather than
being allocated from a memory pool) it's sufficient to just release
resources.  But for a mempool-allocated message we actually have to
re-initialize the data fields in the message back to initial state
so they're ready to go in the event the message gets reused.

Some of this was already done; this fleshes it out so it's done
more completely.

This resolves:
    http://tracker.ceph.com/issues/4540

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
net/ceph/messenger.c

index 2aecc4896a03a9be77341506ef68cba8cf7532f5..0a9f6362d4d8bfa245f480193157968b4522429d 100644 (file)
@@ -3331,12 +3331,17 @@ void ceph_msg_last_put(struct kref *kref)
        if (ceph_msg_has_pages(m)) {
                m->p.length = 0;
                m->p.pages = NULL;
+               m->p.type = CEPH_OSD_DATA_TYPE_NONE;
        }
-
        if (ceph_msg_has_pagelist(m)) {
                ceph_pagelist_release(m->l.pagelist);
                kfree(m->l.pagelist);
                m->l.pagelist = NULL;
+               m->l.type = CEPH_OSD_DATA_TYPE_NONE;
+       }
+       if (ceph_msg_has_bio(m)) {
+               m->b.bio = NULL;
+               m->b.type = CEPH_OSD_DATA_TYPE_NONE;
        }
 
        if (m->pool)