Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 08 Dec 2025 16:16:33 +0000
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: dafe50ebd874 - main - thread0: Clear td_rux stats in proc0_post
Message-ID:  <6936f9e1.3b936.43b92bf4@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help

The branch main has been updated by jhb:

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

commit dafe50ebd874ca3ccd5908c441cee9f478c501f0
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2025-12-08 16:15:36 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2025-12-08 16:16:13 +0000

    thread0: Clear td_rux stats in proc0_post
    
    proc0_post aims to reset the CPU usage accounting for all threads and
    processes in the system to zero once the time of day is verified.
    However, not all of the per-thread stats were not being cleared,
    resulting in over-reported time for thread0 post-boot.
    
    Reviewed by:    olce, kib, markj
    Fixes:          bed4c5241663 ("Implement RUSAGE_THREAD. Add td_rux...")
    Differential Revision:  https://reviews.freebsd.org/D54040
---
 sys/kern/init_main.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/sys/kern/init_main.c b/sys/kern/init_main.c
index 6495340578da..8d1ee7b0fd99 100644
--- a/sys/kern/init_main.c
+++ b/sys/kern/init_main.c
@@ -648,7 +648,8 @@ proc0_post(void *dummy __unused)
 
 	/*
 	 * Now we can look at the time, having had a chance to verify the
-	 * time from the filesystem.  Pretend that proc0 started now.
+	 * time from the filesystem.  Pretend that all current threads
+	 * started now.
 	 */
 	sx_slock(&allproc_lock);
 	FOREACH_PROC_IN_SYSTEM(p) {
@@ -661,10 +662,13 @@ proc0_post(void *dummy __unused)
 		PROC_STATLOCK(p);
 		rufetch(p, &ru);	/* Clears thread stats */
 		ruxreset(&p->p_rux);
-		PROC_STATUNLOCK(p);
 		FOREACH_THREAD_IN_PROC(p, td) {
 			td->td_runtime = 0;
+			thread_lock(td);
+			ruxreset(&td->td_rux);
+			thread_unlock(td);
 		}
+		PROC_STATUNLOCK(p);
 		PROC_UNLOCK(p);
 	}
 	sx_sunlock(&allproc_lock);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6936f9e1.3b936.43b92bf4>