Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 1 Aug 2022 14:19:15 GMT
From:      Mark Johnston <markj@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 211c9ad8b532 - stable/13 - sched_ule: Use the correct atomic_load variant for tdq_lowpri
Message-ID:  <202208011419.271EJFcV025264@gitrepo.freebsd.org>

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

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

commit 211c9ad8b53235d92214e2c08c84b34b32c7e73a
Author:     Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2022-07-14 19:34:02 +0000
Commit:     Mark Johnston <markj@FreeBSD.org>
CommitDate: 2022-08-01 14:13:09 +0000

    sched_ule: Use the correct atomic_load variant for tdq_lowpri
    
    Reported by:    tuexen
    Fixes:  11484ad8a2b0 ("sched_ule: Use explicit atomic accesses for tdq fields")
    
    (cherry picked from commit 6cbc4ceb7aa0042e969be7c782436391c579cc3a)
---
 sys/kern/sched_ule.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/kern/sched_ule.c b/sys/kern/sched_ule.c
index 177838aa3885..519b8275cf7d 100644
--- a/sys/kern/sched_ule.c
+++ b/sys/kern/sched_ule.c
@@ -1360,7 +1360,7 @@ sched_pickcpu(struct thread *td, int flags)
 	 * expired and it is idle, run it there.
 	 */
 	if (THREAD_CAN_SCHED(td, ts->ts_cpu) &&
-	    atomic_load_int(&tdq->tdq_lowpri) >= PRI_MIN_IDLE &&
+	    atomic_load_char(&tdq->tdq_lowpri) >= PRI_MIN_IDLE &&
 	    SCHED_AFFINITY(ts, CG_SHARE_L2)) {
 		if (cg->cg_flags & CG_FLAG_THREAD) {
 			/* Check all SMT threads for being idle. */



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