d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Update td_runtime of running thread on each statclock().

Browse Source 
Normally td_runtime is updated on context switch, but there are some kernel
threads that due to high absolute priority may run for many seconds without
context switches (yes, that is bad, but that is true), which means their
td_runtime was not updated all that time, that made them invisible for top
other then as some general CPU usage.

MFC after:	1 week
Sponsored by:	iXsystems, Inc.
This commit is contained in:
Alexander Motin 2019-06-14 01:09:10 +00:00
parent 1682a3ab4b
commit c80038a0a7
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=349029
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
sys/kern/kern_clock.c
View File

@ -644,6 +644,7 @@ statclock(int cnt, int usermode)
struct proc *p;
long rss;
long *cp_time;
uint64_t runtime, new_switchtime;
td = curthread;
p = td->td_proc;
@ -699,6 +700,17 @@ statclock(int cnt, int usermode)
"prio:%d", td->td_priority, "stathz:%d", (stathz)?stathz:hz);
SDT_PROBE2(sched, , , tick, td, td->td_proc);
thread_lock_flags(td, MTX_QUIET);
/*
* Compute the amount of time during which the current
* thread was running, and add that to its total so far.
*/
new_switchtime = cpu_ticks();
runtime = new_switchtime - PCPU_GET(switchtime);
td->td_runtime += runtime;
td->td_incruntime += runtime;
PCPU_SET(switchtime, new_switchtime);
for ( ; cnt > 0; cnt--)
sched_clock(td);
thread_unlock(td);