From 057ab4a6c76f893b19dfd2526c08e5904a700322 Mon Sep 17 00:00:00 2001 From: Hubert Kowalski Date: Tue, 18 Jan 2022 13:41:25 +0100 Subject: [PATCH] Removed erroneous 'if' statement in get_delay() That 'if' statement will break average val calculation after 1000 hits. Change-Id: I77689b6dd1e8fb12925eae47c3cee2cfeacdd05c --- tools/dbus-latency.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/tools/dbus-latency.c b/tools/dbus-latency.c index 608552d2..f196aa77 100644 --- a/tools/dbus-latency.c +++ b/tools/dbus-latency.c @@ -172,14 +172,11 @@ static int get_delay(int serial, struct data_t *receiver_data) { else if (stats.max < delta) { stats.max = delta; } - u32 avg = stats.avg; - if (stats.count > 1000) { - avg *= 1000; - } - else { - avg *= stats.count; - stats.count++; - } + u64 avg = stats.avg; + + avg *= stats.count; + stats.count++; + avg += delta; stats.avg = (avg / stats.count); msg_latency.update(receiver_data, &stats); -- 2.34.1