From: Xuehan Xu Date: Wed, 25 May 2022 10:45:02 +0000 (+0800) Subject: crimson/os/seastore/backref: tolerate removing non-existing backrefs X-Git-Tag: v18.0.0~763^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F46396%2Fhead;p=ceph.git crimson/os/seastore/backref: tolerate removing non-existing backrefs Signed-off-by: Xuehan Xu --- diff --git a/src/crimson/os/seastore/backref/btree_backref_manager.cc b/src/crimson/os/seastore/backref/btree_backref_manager.cc index 9c0f2ba52412..5371351ef69b 100644 --- a/src/crimson/os/seastore/backref/btree_backref_manager.cc +++ b/src/crimson/os/seastore/backref/btree_backref_manager.cc @@ -376,8 +376,9 @@ BtreeBackrefManager::remove_mapping( -> remove_mapping_ret { if (iter.is_end() || iter.get_key() != addr) { LOG_PREFIX(BtreeBackrefManager::remove_mapping); - ERRORT("paddr={} doesn't exist", c.trans, addr); - return crimson::ct_error::enoent::make(); + DEBUGT("paddr={} doesn't exist", c.trans, addr); + return remove_mapping_iertr::make_ready_future< + remove_mapping_result_t>(remove_mapping_result_t()); } auto ret = remove_mapping_result_t{ diff --git a/src/crimson/os/seastore/backref_manager.h b/src/crimson/os/seastore/backref_manager.h index f3f661086d59..6e98f11b4f37 100644 --- a/src/crimson/os/seastore/backref_manager.h +++ b/src/crimson/os/seastore/backref_manager.h @@ -128,9 +128,9 @@ public: const uint64_t max) = 0; struct remove_mapping_result_t { - paddr_t offset; - extent_len_t len; - laddr_t laddr; + paddr_t offset = P_ADDR_NULL; + extent_len_t len = 0; + laddr_t laddr = L_ADDR_NULL; }; /**