From cfcdac110bb8f191563430603e0bd4d37827c7ea Mon Sep 17 00:00:00 2001 From: Zygo Blaxell Date: Thu, 22 Dec 2022 00:27:08 -0500 Subject: [PATCH] context: don't count MultiLock waiting time in dedup_ms This was inflating the dedup_ms statistic because it was counting all the resolve time too. Signed-off-by: Zygo Blaxell --- src/bees-context.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/bees-context.cc b/src/bees-context.cc index ae2075a..2f96dcf 100644 --- a/src/bees-context.cc +++ b/src/bees-context.cc @@ -217,11 +217,12 @@ BeesContext::dedup(const BeesRangePair &brp_in) THROW_CHECK1(invalid_argument, brp, brp.first.size() == brp.second.size()); BEESCOUNT(dedup_try); - Timer dedup_timer; BEESNOTE("waiting to dedup " << brp); const auto lock = MultiLocker::get_lock("dedupe"); + Timer dedup_timer; + BEESLOGINFO("dedup: src " << pretty(brp.first.size()) << " [" << to_hex(brp.first.begin()) << ".." << to_hex(brp.first.end()) << "] {" << first_addr << "} " << name_fd(brp.first.fd()) << "\n" << " dst " << pretty(brp.second.size()) << " [" << to_hex(brp.second.begin()) << ".." << to_hex(brp.second.end()) << "] {" << second_addr << "} " << name_fd(brp.second.fd())); BEESNOTE("dedup: src " << pretty(brp.first.size()) << " [" << to_hex(brp.first.begin()) << ".." << to_hex(brp.first.end()) << "] {" << first_addr << "} " << name_fd(brp.first.fd()) << "\n"