Removed erroneous 'if' statement in get_delay() 81/269581/4
authorHubert Kowalski <h.kowalski@samsung.com>
Tue, 18 Jan 2022 12:41:25 +0000 (13:41 +0100)
committerAdrian Szyndela <adrian.s@samsung.com>
Tue, 18 Jan 2022 13:08:43 +0000 (13:08 +0000)
That 'if' statement will break average val calculation after 1000 hits.

Change-Id: I77689b6dd1e8fb12925eae47c3cee2cfeacdd05c

tools/dbus-latency.c

index 608552d2cc2b2b7d0fef1ff0f6658c071ccd6c18..f196aa772525e1b553c9030fca36c6702c159ead 100644 (file)
@@ -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);