sergiotarxz/nuttx-apps
sergiotarxz/nuttx-apps
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

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

Browse Source 
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>
This commit is contained in:
Masayuki Ishikawa 2022-12-12 14:30:40 +09:00 committed by Xiang Xiao
parent 636a155a21
commit 5592e38253
1 changed files with 7 additions and 5 deletions

12
netutils/iperf/iperf.c

@ -267,9 +267,10 @@ static void iperf_report_task(FAR void *arg)
printf("%7.2lf-%7.2lf sec %10ju Bytes %7.2f Mbits/sec\n", printf("%7.2lf-%7.2lf sec %10ju Bytes %7.2f Mbits/sec\n",
ts_diff(&last, &start), ts_diff(&last, &start),
ts_diff(&now, &start), ts_diff(&now, &start),
now_len, now_len -last_len,
(((double)(now_len - last_len) * 8) / (double)((now_len - last_len) * 8 / 1000000) /
ts_diff(&now, &last) / 1e6)); (double)ts_diff(&now, &last)
);
if (time != 0 && ts_diff(&now, &start) >= time) if (time != 0 && ts_diff(&now, &start) >= time)
{ {
break; break;
@ -282,8 +283,9 @@ static void iperf_report_task(FAR void *arg)
ts_diff(&start, &start), ts_diff(&start, &start),
ts_diff(&now, &start), ts_diff(&now, &start),
now_len, now_len,
(((double)now_len * 8) / (double)(now_len * 8 / 1000000) /
ts_diff(&now, &start) / 1e6)); (double)ts_diff(&now, &start)
);
} }
ctrl->finish = true; ctrl->finish = true;