os/bluestore: Adapt to new DENC_START_UNCHECKED

aclamk · aclamk · commit 14fc61b99b6c · 2025-03-31T07:33:47.000Z
blustore_onode_t encoding version is directly controlled by options.
We cannot use static checker.

Signed-off-by: Adam Kupczyk &lt;akupczyk@ibm.com&gt;
diff --git a/src/os/bluestore/bluestore_types.h b/src/os/bluestore/bluestore_types.h
@@ -1262,7 +1262,7 @@ struct bluestore_onode_t {
     if ((features & FLAG_DEBUG_FORCE_V2) != 0) {
       struct_v_to_use = 2;
     }
-    DENC_START(struct_v_to_use, 1, p);
+    DENC_START_UNCHECKED(struct_v_to_use, 1, p);
     _denc_friend(*this, p, struct_v_to_use);
     DENC_FINISH(p);
   }
@@ -1271,13 +1271,13 @@ struct bluestore_onode_t {
     if ((features & FLAG_DEBUG_FORCE_V2) != 0) {
       struct_v_to_use = 2;
     }
-    DENC_START(struct_v_to_use, 1, p);
+    DENC_START_UNCHECKED(struct_v_to_use, 1, p);
     DENC_DUMP_PRE(Type);
     _denc_friend(*this, p, struct_v_to_use);
     DENC_FINISH(p);
   }
   void decode(::ceph::buffer::ptr::const_iterator& p, uint64_t features = 0) {
-    DENC_START(3, 1, p);
+    DENC_START_UNCHECKED(3, 1, p);
     _denc_friend(*this, p, struct_v); //decode what is
     if ((features & FLAG_DEBUG_FORCE_V2) != 0) {
       this->segment_size = 0;

Original file line number	Diff line number	Diff line change
`@@ -1262,7 +1262,7 @@ struct bluestore_onode_t {`
`1262`	`1262`	`if ((features & FLAG_DEBUG_FORCE_V2) != 0) {`
`1263`	`1263`	`struct_v_to_use = 2;`
`1264`	`1264`	`}`
`1265`		`- DENC_START(struct_v_to_use, 1, p);`
	`1265`	`+ DENC_START_UNCHECKED(struct_v_to_use, 1, p);`
`1266`	`1266`	`_denc_friend(*this, p, struct_v_to_use);`
`1267`	`1267`	`DENC_FINISH(p);`
`1268`	`1268`	`}`
`@@ -1271,13 +1271,13 @@ struct bluestore_onode_t {`
`1271`	`1271`	`if ((features & FLAG_DEBUG_FORCE_V2) != 0) {`
`1272`	`1272`	`struct_v_to_use = 2;`
`1273`	`1273`	`}`
`1274`		`- DENC_START(struct_v_to_use, 1, p);`
	`1274`	`+ DENC_START_UNCHECKED(struct_v_to_use, 1, p);`
`1275`	`1275`	`DENC_DUMP_PRE(Type);`
`1276`	`1276`	`_denc_friend(*this, p, struct_v_to_use);`
`1277`	`1277`	`DENC_FINISH(p);`
`1278`	`1278`	`}`
`1279`	`1279`	`void decode(::ceph::buffer::ptr::const_iterator& p, uint64_t features = 0) {`
`1280`		`- DENC_START(3, 1, p);`
	`1280`	`+ DENC_START_UNCHECKED(3, 1, p);`
`1281`	`1281`	`_denc_friend(*this, p, struct_v); //decode what is`
`1282`	`1282`	`if ((features & FLAG_DEBUG_FORCE_V2) != 0) {`
`1283`	`1283`	`this->segment_size = 0;`