From: Ronen Friedman Date: Sun, 14 Jun 2026 09:19:20 +0000 (+0000) Subject: crimson,crimson/test: clean-up 'unused' warnings X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=e5c20fa6a8c8ec030253428c77a8142e22d915ec;p=ceph.git crimson,crimson/test: clean-up 'unused' warnings Signed-off-by: Ronen Friedman --- diff --git a/src/crimson/net/io_handler.cc b/src/crimson/net/io_handler.cc index c08bdc87422..f9e9198c748 100644 --- a/src/crimson/net/io_handler.cc +++ b/src/crimson/net/io_handler.cc @@ -1266,6 +1266,7 @@ IOHandler::close_io( } else { return shard_states->close( ).then([this] { + std::ignore = this; // as we are 'assert'ing, not ceph_assert'ing assert(shard_states->assert_closed_and_exit()); }); } diff --git a/src/crimson/os/seastore/object_data_handler.cc b/src/crimson/os/seastore/object_data_handler.cc index a586a606d68..65e6b9cf47c 100644 --- a/src/crimson/os/seastore/object_data_handler.cc +++ b/src/crimson/os/seastore/object_data_handler.cc @@ -1453,6 +1453,7 @@ ObjectDataHandler::clear_ret ObjectDataHandler::trim_data_reservation( ctx.t, unaligned_begin.get_aligned_laddr(ctx.tm.get_block_size()) ).si_then([ctx, data_base, size, this, unaligned_begin, &object_data](auto mapping) { + std::ignore = unaligned_begin; assert(mapping.get_key() <= unaligned_begin && mapping.get_key() + mapping.get_length() > unaligned_begin); auto data_len = object_data.get_reserved_data_len(); diff --git a/src/crimson/os/seastore/transaction_manager.cc b/src/crimson/os/seastore/transaction_manager.cc index f0e586220b9..62d24945517 100644 --- a/src/crimson/os/seastore/transaction_manager.cc +++ b/src/crimson/os/seastore/transaction_manager.cc @@ -945,7 +945,7 @@ TransactionManager::move_region( ).handle_error_interruptible( move_region_iertr::pass_further(), crimson::ct_error::assert_all("invalid error")); - auto off = 0; + [[maybe_unused]] auto off = 0; auto bl = maybe_indirect_extent.get_range( src.get_intermediate_offset(), src.get_length()); diff --git a/src/crimson/osd/ec_backend.cc b/src/crimson/osd/ec_backend.cc index 368d29c7147..4ce37c77920 100644 --- a/src/crimson/osd/ec_backend.cc +++ b/src/crimson/osd/ec_backend.cc @@ -457,6 +457,7 @@ ECBackend::handle_rep_write_op( return handle_sub_write( m->op.from, std::move(m->op), pg ).si_then([&pg] { + std::ignore = pg; assert(!pg.pgb_is_primary()); return write_iertr::now(); }, crimson::ct_error::assert_all("unexpected error")); diff --git a/src/test/crimson/seastore/test_cbjournal.cc b/src/test/crimson/seastore/test_cbjournal.cc index ffd9c96c938..c97b771cf3b 100644 --- a/src/test/crimson/seastore/test_cbjournal.cc +++ b/src/test/crimson/seastore/test_cbjournal.cc @@ -249,7 +249,7 @@ struct cbjournal_test_t : public seastar_test_suite_t, JournalTrimmer auto &dirty_seq, auto &alloc_seq, auto last_modified) { - bool found = false; + [[maybe_unused]] bool found = false; for (auto &i : entries) { paddr_t base = offsets.write_result.start_seq.offset; rbm_abs_addr addr = convert_paddr_to_abs_addr(base); @@ -583,7 +583,7 @@ TEST_F(cbjournal_test_t, multiple_submit_at_end) } }); }).get(); - auto old_written_to = get_written_to(); + [[maybe_unused]] auto old_written_to = get_written_to(); cbj->close().unsafe_get(); cbj->replay( [](const auto &offsets,