From: Yingxin Cheng <yingxin.cheng@intel.com>
Date: Thu, 1 Aug 2024 08:44:48 +0000 (+0800)
Subject: crimson/os/seastore: drop duplicated calls to touch_extent()
X-Git-Tag: testing/wip-vshankar-testing-20240820.042758-debug~10^2~9
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=dba31242e73c2b5830006c4c4f8bcd7ef1247922;p=ceph-ci.git

crimson/os/seastore: drop duplicated calls to touch_extent()

The extent is already PRESENT, which means it was already touched in
this transaction.

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

diff --git a/src/crimson/os/seastore/cache.h b/src/crimson/os/seastore/cache.h
index 6528854e072..0807e83d371 100644
--- a/src/crimson/os/seastore/cache.h
+++ b/src/crimson/os/seastore/cache.h
@@ -371,7 +371,6 @@ public:
         });
       } else {
 	assert(!ret->is_mutable());
-	touch_extent(*ret);
         SUBDEBUGT(seastore_cache, "{} {}~{} is present on t without been \
           fully loaded, reading ... {}", t, T::TYPE, offset, length, *ret);
         auto bp = alloc_cache_buf(ret->get_length());
@@ -712,7 +711,6 @@ private:
         });
       } else {
 	assert(!ret->is_mutable());
-	touch_extent(*ret);
         SUBDEBUGT(seastore_cache, "{} {}~{} {} is present on t without been \
                   fully loaded, reading ...", t, type, offset, length, laddr);
         auto bp = alloc_cache_buf(ret->get_length());