Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 1 May 2022 17:27:38 GMT
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: b65803ba5773 - main - cam iosched: default to no read bias in dynamic ioscheduling
Message-ID:  <202205011727.241HRcUI061979@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by imp:

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

commit b65803ba5773d5fb37fa2403105db199569a5811
Author:     Warner Losh <imp@FreeBSD.org>
AuthorDate: 2022-05-01 17:18:23 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2022-05-01 17:27:34 +0000

    cam iosched: default to no read bias in dynamic ioscheduling
    
    When we're doing dynamic I/O scheduling, don't default to a read bias of
    100. Default it to 0 so turning on dynamic scheduling only does
    scheduling tweaks that are requested. The other limiters are off by
    default, and need no further adjustment.
    
    Sponsored by:           Netflix
---
 sys/cam/cam_iosched.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/sys/cam/cam_iosched.c b/sys/cam/cam_iosched.c
index 7ddfabbe5125..21a948b69c69 100644
--- a/sys/cam/cam_iosched.c
+++ b/sys/cam/cam_iosched.c
@@ -1149,8 +1149,8 @@ cam_iosched_init(struct cam_iosched_softc **iscp, struct cam_periph *periph)
 #ifdef CAM_IOSCHED_DYNAMIC
 	if (do_dynamic_iosched) {
 		bioq_init(&(*iscp)->write_queue);
-		(*iscp)->read_bias = 100;
-		(*iscp)->current_read_bias = 100;
+		(*iscp)->read_bias = 0;
+		(*iscp)->current_read_bias = 0;
 		(*iscp)->quanta = min(hz, 200);
 		cam_iosched_iop_stats_init(*iscp, &(*iscp)->read_stats);
 		cam_iosched_iop_stats_init(*iscp, &(*iscp)->write_stats);
@@ -1234,7 +1234,7 @@ void cam_iosched_sysctl_init(struct cam_iosched_softc *isc,
 
 	SYSCTL_ADD_INT(ctx, n,
 	    OID_AUTO, "read_bias", CTLFLAG_RW,
-	    &isc->read_bias, 100,
+	    &isc->read_bias, 0,
 	    "How biased towards read should we be independent of limits");
 
 	SYSCTL_ADD_PROC(ctx, n,



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