From ca2879dae5431a3cb600c21113a5f03d4bdc44f8 Mon Sep 17 00:00:00 2001 From: Xuehan Xu Date: Sun, 29 Jun 2025 17:38:37 +0800 Subject: [PATCH] crimson/os/seastore/btree: only compile debug-purposed methods in debug builds Signed-off-by: Xuehan Xu --- src/crimson/os/seastore/btree/fixed_kv_btree.h | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/crimson/os/seastore/btree/fixed_kv_btree.h b/src/crimson/os/seastore/btree/fixed_kv_btree.h index 513fbbd2e65..d60db362bbd 100644 --- a/src/crimson/os/seastore/btree/fixed_kv_btree.h +++ b/src/crimson/os/seastore/btree/fixed_kv_btree.h @@ -101,7 +101,9 @@ public: op_context_t c, mapped_space_visitor_t *visitor=nullptr) const { +#ifndef NDEBUG assert_valid(); +#endif assert(!is_end()); auto ret = *this; @@ -126,7 +128,9 @@ public: iterator_fut prev(op_context_t c) const { +#ifndef NDEBUG assert_valid(); +#endif assert(!is_begin()); auto ret = *this; @@ -167,6 +171,7 @@ public: }); } +#ifndef NDEBUG void assert_valid() const { assert(leaf.node); assert(leaf.pos <= leaf.node->get_size()); @@ -183,6 +188,7 @@ public: } } } +#endif depth_t get_depth() const { return internal.size() + 1; @@ -555,9 +561,11 @@ public: "ret.leaf.pos {}", c.trans, ret.leaf.pos); +#ifndef NDEBUG if (min_depth == 1) { ret.assert_valid(); } +#endif return std::move(ret); }); } -- 2.39.5