diff options
author | John Baldwin <jhb@FreeBSD.org> | 2011-07-18 21:15:47 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2011-07-18 21:15:47 +0000 |
commit | fcc3d62fa50d0433048a71d54e385ef75ec06011 (patch) | |
tree | 51cdc1a8e25efa9aaf2fc32d7b692e155c57606c /usr.bin/top | |
parent | 834377fe2ded06485f80a727846e990622f56f71 (diff) | |
download | src-fcc3d62fa50d0433048a71d54e385ef75ec06011.tar.gz src-fcc3d62fa50d0433048a71d54e385ef75ec06011.zip |
Rework the dynamic per-CPU stats code a bit. Always set 'statics->ncpus'
to the maximum number of CPUs to ensure that lcpustates[] array is always
allocated to the maximum size. Previously, if top was started without
per-CPU stats it would allocate a smaller lcpustates[] array. When
per-CPU stats were then enabled, it would overflow the array and trash
the cpustates_columns[] array causing the CPU stats to be printed in the
wrong locations.
Approved by: re (kib)
MFC after: 1 week
Notes
Notes:
svn path=/head/; revision=224205
Diffstat (limited to 'usr.bin/top')
-rw-r--r-- | usr.bin/top/machine.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/usr.bin/top/machine.c b/usr.bin/top/machine.c index 5e53d5b36ed8..b815ffc67e70 100644 --- a/usr.bin/top/machine.c +++ b/usr.bin/top/machine.c @@ -241,7 +241,7 @@ static void getsysctl(const char *name, void *ptr, size_t len); static int swapmode(int *retavail, int *retfree); void -toggle_pcpustats(struct statics *statics) +toggle_pcpustats(void) { if (ncpus == 1) @@ -256,7 +256,6 @@ toggle_pcpustats(struct statics *statics) y_header += ncpus - 1; /* 6 */ y_procs += ncpus - 1; /* 7 */ Header_lines += ncpus - 1; /* 7 */ - statics->ncpus = ncpus; } else { y_mem = 3; y_swap = 4; @@ -265,7 +264,6 @@ toggle_pcpustats(struct statics *statics) y_header = 6; y_procs = 7; Header_lines = 7; - statics->ncpus = 1; } } @@ -356,10 +354,10 @@ machine_init(struct statics *statics, char do_unames) pcpu_cp_old = calloc(1, size); pcpu_cp_diff = calloc(1, size); pcpu_cpu_states = calloc(1, size); - statics->ncpus = 1; + statics->ncpus = ncpus; if (pcpu_stats) - toggle_pcpustats(statics); + toggle_pcpustats(); /* all done! */ return (0); |