os/bluestore/bluefs: Removed unused field from bluefs_super_t

aclamk · aclamk · commit eaed7c053426 · 2025-06-05T19:34:33.000Z
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 &lt;akupczyk@ibm.com&gt;
diff --git 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
@@ -328,8 +328,6 @@ struct bluefs_super_t {
 
   std::optional<bluefs_layout_t> memorized_layout;
 
-  uint8_t wal_version;
-
   bluefs_super_t();
 
   uint64_t block_mask() const {

Original file line number	Diff line number	Diff line change
`@@ -190,9 +190,6 @@ void bluefs_super_t::encode(bufferlist& bl) const`
`190`	`190`	`encode(block_size, bl);`
`191`	`191`	`encode(log_fnode, bl);`
`192`	`192`	`encode(memorized_layout, bl);`
`193`		`- if (_version >= 3) {`
`194`		`- encode(wal_version, bl);`
`195`		`- }`
`196`	`193`	`ENCODE_FINISH(bl);`
`197`	`194`	`}`
`198`	`195`
`@@ -208,9 +205,6 @@ void bluefs_super_t::decode(bufferlist::const_iterator& p)`
`208`	`205`	`if (struct_v >= 2) {`
`209`	`206`	`decode(memorized_layout, p);`
`210`	`207`	`}`
`211`		`- if (struct_v >= 3) {`
`212`		`- decode(wal_version, p);`
`213`		`- }`
`214`	`208`	`DECODE_FINISH(p);`
`215`	`209`	`}`
`216`	`210`