From: Yingxin Cheng <yingxin.cheng@intel.com>
Date: Mon, 18 Jul 2022 03:29:46 +0000 (+0800)
Subject: crimson/os/seastore: do not assign gen/hint when split a clean extent
X-Git-Tag: v18.0.0~472^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=c8b70d96b7964269d80368ace46e35f4744b4fa9;p=ceph.git

crimson/os/seastore: do not assign gen/hint when split a clean extent

Gen/hint is only required when need to write the extent to disk.

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
---

diff --git a/src/crimson/os/seastore/transaction_manager.h b/src/crimson/os/seastore/transaction_manager.h
index 295ba8f35cafa..1062adfd9370f 100644
--- a/src/crimson/os/seastore/transaction_manager.h
+++ b/src/crimson/os/seastore/transaction_manager.h
@@ -381,9 +381,7 @@ public:
     Transaction &t,
     laddr_t laddr_hint,
     paddr_t existing_paddr,
-    extent_len_t length,
-    placement_hint_t placement_hint = placement_hint_t::HOT,
-    reclaim_gen_t gen = DIRTY_GENERATION) {
+    extent_len_t length) {
     LOG_PREFIX(TransactionManager::map_existing_extent);
     ceph_assert(existing_paddr.is_absolute());
     assert(t.is_retired(existing_paddr, length));
@@ -397,8 +395,8 @@ public:
 
     ext->init(CachedExtent::extent_state_t::EXIST_CLEAN,
 	      existing_paddr,
-	      placement_hint,
-	      gen);
+	      PLACEMENT_HINT_NULL,
+	      NULL_GENERATION);
 
     t.add_fresh_extent(ext);