From: Adam Kupczyk Date: Thu, 5 Jun 2025 19:34:33 +0000 (+0000) Subject: os/bluestore/bluefs: Removed unused field from bluefs_super_t X-Git-Tag: testing/wip-vshankar-testing-20250623.033253-tentacle-debug~16^2~1 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ed2cef2fe6ccce7e21255ee73c8e856fb8d91d0f;p=ceph-ci.git os/bluestore/bluefs: Removed unused field from bluefs_super_t New WAL handling mode added field `wal_version` to bluefs_super_t. At some point of refactor become unused. Removing it while we still can. Signed-off-by: Adam Kupczyk (cherry picked from commit eaed7c053426c339ca743ced25864e63cbcaa287) --- diff --git a/src/os/bluestore/bluefs_types.cc b/src/os/bluestore/bluefs_types.cc index 80e948a72f2..8aaed45a742 100644 --- a/src/os/bluestore/bluefs_types.cc +++ b/src/os/bluestore/bluefs_types.cc @@ -190,9 +190,6 @@ void bluefs_super_t::encode(bufferlist& bl) const encode(block_size, bl); encode(log_fnode, bl); encode(memorized_layout, bl); - if (_version >= 3) { - encode(wal_version, bl); - } ENCODE_FINISH(bl); } @@ -208,9 +205,6 @@ void bluefs_super_t::decode(bufferlist::const_iterator& p) if (struct_v >= 2) { decode(memorized_layout, p); } - if (struct_v >= 3) { - decode(wal_version, p); - } DECODE_FINISH(p); } diff --git a/src/os/bluestore/bluefs_types.h b/src/os/bluestore/bluefs_types.h index 767d0cf1968..1dbf5c4e507 100644 --- a/src/os/bluestore/bluefs_types.h +++ b/src/os/bluestore/bluefs_types.h @@ -328,8 +328,6 @@ struct bluefs_super_t { std::optional memorized_layout; - uint8_t wal_version; - bluefs_super_t(); uint64_t block_mask() const {