Merge pull request ceph#57474 from xxhdx1985126/wip-seastore-rewrite-bug-fix

crimson/os/seastore/transaction_manager: correct the offset of the data copied from the original extents

Reviewed-by: Yingxin Cheng <[email protected]>

Author: cyx1231st

src/crimson/os/seastore/btree/fixed_kv_btree.h

@@ -1040,6 +1040,7 @@ class FixedKVBtree {
         n_fixed_kv_extent->get_bptr().c_str());
       n_fixed_kv_extent->set_modify_time(fixed_kv_extent.get_modify_time());
       n_fixed_kv_extent->range = n_fixed_kv_extent->get_node_meta();
+      n_fixed_kv_extent->set_last_committed_crc(fixed_kv_extent.get_last_committed_crc());
 
       if (fixed_kv_extent.get_type() == internal_node_t::TYPE ||
           leaf_node_t::do_has_children) {

src/crimson/os/seastore/transaction_manager.cc

@@ -542,12 +542,12 @@ TransactionManager::rewrite_logical_extent(
         ceph_assert(left >= nextent->get_length());
         auto nlextent = nextent->template cast<LogicalCachedExtent>();
         lextent->get_bptr().copy_out(
-          0,
+          off,
           nlextent->get_length(),
           nlextent->get_bptr().c_str());
         nlextent->set_laddr(lextent->get_laddr() + off);
         nlextent->set_modify_time(lextent->get_modify_time());
-        nlextent->set_last_committed_crc(nlextent->calc_crc32c());
+        nlextent->set_last_committed_crc(lextent->get_last_committed_crc());
         DEBUGT("rewriting logical extent -- {} to {}", t, *lextent, *nlextent);
 
         /* This update_mapping is, strictly speaking, unnecessary for delayed_alloc