From 5df6ffc79b415def0b7984096413e9562e83399a Mon Sep 17 00:00:00 2001 From: myoungwon oh Date: Fri, 19 Apr 2024 04:57:52 +0000 Subject: [PATCH] crimson/seastore: add a TODO comment regarding is_data_stable() Signed-off-by: Myoungwon Oh --- src/crimson/os/seastore/object_data_handler.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/src/crimson/os/seastore/object_data_handler.cc b/src/crimson/os/seastore/object_data_handler.cc index d741fd5c451..b6d5a86dd6d 100644 --- a/src/crimson/os/seastore/object_data_handler.cc +++ b/src/crimson/os/seastore/object_data_handler.cc @@ -297,6 +297,7 @@ overwrite_ops_t prepare_ops_list( interval_set pre_alloc_addr_removed, pre_alloc_addr_remapped; if (delta_based_overwrite_max_extent_size) { for (auto &r : ops.to_remove) { + // TODO: Introduce LBAMapping::is_data_stable() to include EXIST_CLEAN extents if (r->is_stable() && !r->is_zero_reserved()) { pre_alloc_addr_removed.insert(r->get_key(), r->get_length()); -- 2.39.5