Merge pull request ceph#56250 from myoungwon/wip-laod-metadata-during-gc

cyx1231st · web-flow · commit e76d3a3f397a · 2024-03-19T09:29:26.000+08:00
crimson/os/seastore: cache metadata during trimming to prevent from disk read

Reviewed-by: Yingxin Cheng &lt;yingxin.cheng@intel.com&gt;
Reviewed-by: Chunmei Liu &lt;chunmei.liu@intel.com&gt;
diff --git a/src/crimson/os/seastore/cache.h b/src/crimson/os/seastore/cache.h
@@ -1295,8 +1295,11 @@ class Cache {
       CachedExtent &ext,
       const Transaction::src_t* p_src=nullptr)
   {
-    if (p_src && is_background_transaction(*p_src))
+    if (p_src &&
+	is_background_transaction(*p_src) &&
+	is_logical_type(ext.get_type())) {
       return;
+    }
     if (ext.is_stable_clean() && !ext.is_placeholder()) {
       lru.move_to_top(ext);
     }

Original file line number	Diff line number	Diff line change
`@@ -1295,8 +1295,11 @@ class Cache {`
`1295`	`1295`	`CachedExtent &ext,`
`1296`	`1296`	`const Transaction::src_t* p_src=nullptr)`
`1297`	`1297`	`{`
`1298`		`- if (p_src && is_background_transaction(*p_src))`
	`1298`	`+ if (p_src &&`
	`1299`	`+ is_background_transaction(*p_src) &&`
	`1300`	`+ is_logical_type(ext.get_type())) {`
`1299`	`1301`	`return;`
	`1302`	`+ }`
`1300`	`1303`	`if (ext.is_stable_clean() && !ext.is_placeholder()) {`
`1301`	`1304`	`lru.move_to_top(ext);`
`1302`	`1305`	`}`