From 046f635d1c85b84eafe4abfa6887b5145705927e Mon Sep 17 00:00:00 2001 From: =?utf8?q?Piotr=20Da=C5=82ek?= Date: Tue, 20 Feb 2018 15:02:18 +0100 Subject: [PATCH] common/TrackedOp: get duration just once when inserting MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit No need to do this twice. Signed-off-by: Piotr Dałek --- src/common/TrackedOp.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/common/TrackedOp.cc b/src/common/TrackedOp.cc index 7653673948d9b..baf081d0dd4dc 100644 --- a/src/common/TrackedOp.cc +++ b/src/common/TrackedOp.cc @@ -36,9 +36,10 @@ void OpHistory::insert(utime_t now, TrackedOpRef op) Mutex::Locker history_lock(ops_history_lock); if (shutdown) return; - duration.insert(make_pair(op->get_duration(), op)); + double opduration = op->get_duration(); + duration.insert(make_pair(opduration, op)); arrived.insert(make_pair(op->get_initiated(), op)); - if (op->get_duration() >= history_slow_op_threshold) + if (opduration >= history_slow_op_threshold) slow_op.insert(make_pair(op->get_initiated(), op)); cleanup(now); } -- 2.39.5