From d9044d3b396bc184500eb1183688044f38ef199c Mon Sep 17 00:00:00 2001 From: Yingxin Cheng Date: Mon, 11 Oct 2021 15:57:29 +0800 Subject: [PATCH] crimson/os/seastore: rename ool_write_stats_t::overhead_bytes To header_bytes to be more explicit. Signed-off-by: Yingxin Cheng --- src/crimson/os/seastore/cache.cc | 4 ++-- src/crimson/os/seastore/extent_placement_manager.cc | 2 +- src/crimson/os/seastore/transaction.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/crimson/os/seastore/cache.cc b/src/crimson/os/seastore/cache.cc index 4f810003b90e5..f7f32b8b04e40 100644 --- a/src/crimson/os/seastore/cache.cc +++ b/src/crimson/os/seastore/cache.cc @@ -676,7 +676,7 @@ void Cache::mark_transaction_conflicted( efforts.fresh_ool_written.extents += ool_stats.extents.num; efforts.fresh_ool_written.bytes += ool_stats.extents.bytes; efforts.num_ool_records += ool_stats.num_records; - efforts.ool_record_overhead_bytes += ool_stats.overhead_bytes; + efforts.ool_record_overhead_bytes += ool_stats.header_bytes; if (t.get_src() == Transaction::src_t::CLEANER) { // CLEANER transaction won't contain any onode tree operations @@ -940,7 +940,7 @@ record_t Cache::prepare_record(Transaction &t) auto& ool_stats = t.get_ool_write_stats(); ceph_assert(ool_stats.extents.num == t.ool_block_list.size()); efforts.num_ool_records += ool_stats.num_records; - efforts.ool_record_overhead_bytes += ool_stats.overhead_bytes; + efforts.ool_record_overhead_bytes += ool_stats.header_bytes; auto record_size = get_encoded_record_length( record, segment_manager.get_block_size()); auto inline_overhead = diff --git a/src/crimson/os/seastore/extent_placement_manager.cc b/src/crimson/os/seastore/extent_placement_manager.cc index 43e6235ddabe9..32ff1e94360b0 100644 --- a/src/crimson/os/seastore/extent_placement_manager.cc +++ b/src/crimson/os/seastore/extent_placement_manager.cc @@ -90,7 +90,7 @@ SegmentedAllocator::Writer::_write( auto extent_bytes = record.get_raw_data_size(); stats.extents.bytes += extent_bytes; assert(bl.length() > extent_bytes); - stats.overhead_bytes += (bl.length() - extent_bytes); + stats.header_bytes += (bl.length() - extent_bytes); stats.num_records += 1; return trans_intr::make_interruptible( diff --git a/src/crimson/os/seastore/transaction.h b/src/crimson/os/seastore/transaction.h index e0a1a9f4b3cd3..f1d2e1a52f650 100644 --- a/src/crimson/os/seastore/transaction.h +++ b/src/crimson/os/seastore/transaction.h @@ -302,7 +302,7 @@ public: struct ool_write_stats_t { io_stat_t extents; - uint64_t overhead_bytes = 0; + uint64_t header_bytes = 0; uint64_t num_records = 0; }; ool_write_stats_t& get_ool_write_stats() { -- 2.39.5