commit 5592e38253bc74c65f4d105f837d8bc048ac1859
author Masayuki Ishikawa <masayuki.ishikawa@gmail.com> Mon Dec 12 14:30:40 2022 +0900
committer Xiang Xiao <xiaoxiang781216@gmail.com> Mon Dec 12 21:45:52 2022 +0800
tree 2711be21f97f9cea525857acd2a16f2cb382fb10
parent 636a155a218e11b9fc2b406ab5b3c554a18abff9

netutils: iperf: Fix the transfer bytes and the bandwidth overflow

Summary:
- I noticed that the iperf shows incorrect transfer bytes
  in each period.
- Also, the bandwidth overflows sometimes.
- This commit fixes these issues.

Impact:
- None

Testing:
- Tested with qemu-armv8a:netnsh_smp on QEMU-7.1

Signed-off-by: Masayuki Ishikawa <Masayuki.Ishikawa@jp.sony.com>

netutils/iperf/iperf.c

diff --git a/netutils/iperf/iperf.c b/netutils/iperf/iperf.c
index aea1b02..ccddcc2 100644
--- a/netutils/iperf/iperf.c
+++ b/netutils/iperf/iperf.c
@@ -267,9 +267,10 @@
       printf("%7.2lf-%7.2lf sec %10ju Bytes %7.2f Mbits/sec\n",
              ts_diff(&last, &start),
              ts_diff(&now, &start),
-             now_len,
-             (((double)(now_len - last_len) * 8) /
-             ts_diff(&now, &last) / 1e6));
+             now_len -last_len,
+             (double)((now_len - last_len) * 8 / 1000000) /
+             (double)ts_diff(&now, &last)
+             );
       if (time != 0 && ts_diff(&now, &start) >= time)
         {
           break;
@@ -282,8 +283,9 @@
              ts_diff(&start, &start),
              ts_diff(&now, &start),
              now_len,
-             (((double)now_len * 8) /
-             ts_diff(&now, &start) / 1e6));
+             (double)(now_len * 8 / 1000000) /
+             (double)ts_diff(&now, &start)
+             );
     }
 
   ctrl->finish = true;