From 1d92e81fa47925dab1fa3779b8e8e2e35d3c14ce Mon Sep 17 00:00:00 2001 From: Warner Losh Date: Sun, 5 Dec 2021 22:57:50 -0700 Subject: [PATCH] cam/iosched: fix off by one error Set the bucket size to be SBT_1US/50000 + 1 to be the first number > 20us. I had this uncommitted in my three when I pushed 2283206935b8 since kern.cam.iosched.bucket_base_us was reporting 19us. Fixes: 2283206935b8 Sponsored by: Netflix --- sys/cam/cam_iosched.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/cam/cam_iosched.c b/sys/cam/cam_iosched.c index 10caefdb02e..4d584dc160a 100644 --- a/sys/cam/cam_iosched.c +++ b/sys/cam/cam_iosched.c @@ -112,7 +112,7 @@ SYSCTL_INT(_kern_cam_iosched, OID_AUTO, alpha_bits, CTLFLAG_RW | CTLFLAG_TUN, * of 5.2s which is safeily larger than 1s to help diagnose extreme outliers better. */ #ifndef BUCKET_BASE -#define BUCKET_BASE (SBT_1S / 50000) /* 20us */ +#define BUCKET_BASE ((SBT_1S / 50000) + 1) /* 20us */ #endif static sbintime_t bucket_base = BUCKET_BASE; SYSCTL_SBINTIME_USEC(_kern_cam_iosched, OID_AUTO, bucket_base_us, CTLFLAG_RD,