Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 22 Aug 2021 23:04:27 GMT
From:      Alan Somers <asomers@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 5f468877862c - stable/12 - iostat: fix rounding errors in iostat -x
Message-ID:  <202108222304.17MN4Rta044112@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by asomers:

URL: https://cgit.FreeBSD.org/src/commit/?id=5f468877862c4ee4e2d94146b924fa0463f926cb

commit 5f468877862c4ee4e2d94146b924fa0463f926cb
Author:     Alan Somers <asomers@FreeBSD.org>
AuthorDate: 2021-07-08 16:16:32 +0000
Commit:     Alan Somers <asomers@FreeBSD.org>
CommitDate: 2021-08-22 23:04:09 +0000

    iostat: fix rounding errors in iostat -x
    
    Better to round numbers instead of flooring them.
    
    Sponsored by:   Axcient
    
    (cherry picked from commit 61631b24a1347a23cafe0657fba894622b1606e2)
---
 usr.sbin/iostat/iostat.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/usr.sbin/iostat/iostat.c b/usr.sbin/iostat/iostat.c
index e5f746c15284..d1c35063c72f 100644
--- a/usr.sbin/iostat/iostat.c
+++ b/usr.sbin/iostat/iostat.c
@@ -888,17 +888,17 @@ devstats(int perf_select, long double etime, int havelast)
 			    mb_per_second_write > ((long double).0005)/1024 ||
 			    busy_pct > 0.5) {
 				if (Iflag == 0)
-					printf("%-8.8s %7d %7d %8.1Lf "
-					    "%8.1Lf %5d %5d %5d %5d "
-					    "%4" PRIu64 " %3.0Lf ",
+					printf("%-8.8s %7.0Lf %7.0Lf %8.1Lf "
+					    "%8.1Lf %5.0Lf %5.0Lf %5.0Lf %5.0Lf"
+					    " %4" PRIu64 " %3.0Lf ",
 					    devicename,
-					    (int)transfers_per_second_read,
-					    (int)transfers_per_second_write,
+					    transfers_per_second_read,
+					    transfers_per_second_write,
 					    mb_per_second_read * 1024,
 					    mb_per_second_write * 1024,
-					    (int)ms_per_read, (int)ms_per_write,
-					    (int)ms_per_other,
-					    (int)ms_per_transaction,
+					    ms_per_read, ms_per_write,
+					    ms_per_other,
+					    ms_per_transaction,
 					    queue_len, busy_pct);
 				else
 					printf("%-8.8s %11.1Lf %11.1Lf "



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202108222304.17MN4Rta044112>