From fd53bff9595eef42dab47b3869860f775ea8f8e0 Mon Sep 17 00:00:00 2001 From: Zygo Blaxell Date: Fri, 14 Feb 2025 17:17:40 -0500 Subject: [PATCH] extent scan: drop out-of-date comment The comment describes an earlier version which submitted each extent ref as a separate Task, but now all extent refs are handled by the same Task to minimize the amount of time between processing the first and last reference to an extent. Signed-off-by: Zygo Blaxell --- src/bees-roots.cc | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/bees-roots.cc b/src/bees-roots.cc index 930ac02..a593600 100644 --- a/src/bees-roots.cc +++ b/src/bees-roots.cc @@ -780,10 +780,6 @@ BeesScanModeExtent::SizeTier::create_extent_map(const uint64_t bytenr, const Pro BtrfsExtentDataFetcher bedf(m_ctx->root_fd()); - // Collect extent ref tasks as a series of stand-alone events - // chained after the first task created, then run the first one. - // This prevents other threads from starting to process an - // extent until we have all of its refs in the queue. const auto refs_list = make_shared>(); for (const auto &i : log_ino.m_iors) { catch_all([&](){