diff options
author | John Baldwin <jhb@FreeBSD.org> | 2013-03-11 16:33:05 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2013-03-11 16:33:05 +0000 |
commit | ee66c8b94e9c132b7bab30375d392e2b1904b065 (patch) | |
tree | 72c3fdf77c3a0ee619fc37a127ed90d9ad3c7dd6 /usr.bin | |
parent | 2a75563d4adb7926ee4dc7df324b2801c20d4b97 (diff) | |
download | src-ee66c8b94e9c132b7bab30375d392e2b1904b065.tar.gz src-ee66c8b94e9c132b7bab30375d392e2b1904b065.zip |
Fix the 'C' field for a running thread to match the behavior described
in the manpage by having it display the current CPU (ki_oncpu) rather
than the previously used CPU (ki_lastcpu). ki_lastcpu is still used for
all other thread states.
Reported by: Chris Ross <cross+freebsd@distal.com>
MFC after: 1 week
Notes
Notes:
svn path=/head/; revision=248167
Diffstat (limited to 'usr.bin')
-rw-r--r-- | usr.bin/top/machine.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/usr.bin/top/machine.c b/usr.bin/top/machine.c index edfa43a5fbe7..e01143d77540 100644 --- a/usr.bin/top/machine.c +++ b/usr.bin/top/machine.c @@ -797,7 +797,7 @@ format_next_process(caddr_t handle, char *(*get_userid)(int), int flags) double pct; struct handle *hp; char status[16]; - int state; + int cpu, state; struct rusage ru, *rup; long p_tot, s_tot; char *proc_fmt, thr_buf[6], jid_buf[6]; @@ -997,6 +997,13 @@ format_next_process(caddr_t handle, char *(*get_userid)(int), int flags) } /* format this entry */ + if (smpmode) { + if (state == SRUN && pp->ki_oncpu != 0xff) + cpu = pp->ki_oncpu; + else + cpu = pp->ki_lastcpu; + } else + cpu = 0; proc_fmt = smpmode ? smp_Proc_format : up_Proc_format; if (ps.thread != 0) thr_buf[0] = '\0'; @@ -1014,7 +1021,7 @@ format_next_process(caddr_t handle, char *(*get_userid)(int), int flags) format_k2(PROCSIZE(pp)), format_k2(pagetok(pp->ki_rssize)), status, - smpmode ? pp->ki_lastcpu : 0, + cpu, format_time(cputime), ps.wcpu ? 100.0 * weighted_cpu(pct, pp) : 100.0 * pct, screen_width > cmdlengthdelta ? screen_width - cmdlengthdelta : 0, |