From: shiqi <1454927420@qq.com> Date: Wed, 12 Apr 2017 09:24:31 +0000 (+0800) Subject: Bluestore :check if blob map returns success X-Git-Tag: v12.0.3~199^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=354e61013c00e7e05f4ac54b9f0a9cf87a18b677;p=ceph.git Bluestore :check if blob map returns success Signed-off-by: shiqi <1454927420@qq.com> --- diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc index 92afe79c9c08..12269b13757a 100644 --- a/src/os/bluestore/BlueStore.cc +++ b/src/os/bluestore/BlueStore.cc @@ -5997,6 +5997,7 @@ int BlueStore::_do_read( return r; return 0; }); + assert(r == 0); } else { // read the pieces for (auto& reg : p.second) { @@ -6034,8 +6035,7 @@ int BlueStore::_do_read( return r; return 0; }); - if (r < 0) - return r; + assert(r == 0); assert(reg.bl.length() == r_len); } } @@ -9353,12 +9353,13 @@ int BlueStore::_do_alloc_write( << l->length() << std::dec << " write via deferred" << dendl; bluestore_deferred_op_t *op = _get_deferred_op(txc, o); op->op = bluestore_deferred_op_t::OP_WRITE; - b->get_blob().map( + int r = b->get_blob().map( b_off, l->length(), [&](uint64_t offset, uint64_t length) { op->extents.emplace_back(bluestore_pextent_t(offset, length)); return 0; }); + assert(r == 0); op->data = *l; } else { b->get_blob().map_bl(