8 years agops: ignore SIGCONT
Craig Small [Tue, 14 Jan 2014 11:23:58 +0000 (22:23 +1100)]
ps: ignore SIGCONT

SIGCONT is a continue signal.  It seems that some zsh setups can send
this signal, causing ps to abort.  This is not what "continue" means.
This change just uses the default handler which will continue a stopped


Signed-off-by: Craig Small <>
8 years agosysctl: Fixing memory leaks in PreloadSystem()
Jaromir Capik [Thu, 2 Jan 2014 18:49:36 +0000 (19:49 +0100)]
sysctl: Fixing memory leaks in PreloadSystem()

8 years agopgrep: Fixing memory leak in do_regcomp()
Jaromir Capik [Thu, 2 Jan 2014 17:39:27 +0000 (18:39 +0100)]
pgrep: Fixing memory leak in do_regcomp()

8 years agolibrary: fixing uninitialized variable 'pos' in whattime.c
Jaromir Capik [Thu, 2 Jan 2014 17:21:31 +0000 (18:21 +0100)]
library: fixing uninitialized variable 'pos' in whattime.c

8 years agoSplit help lines to help translators
Craig Small [Fri, 27 Dec 2013 22:25:39 +0000 (09:25 +1100)]
Split help lines to help translators

To assist translators, the help lines are split so that each translation
chunk has one option. This gives bonus of if we add or change an option,
only that option remains untranslated rather than the entire help block.


Signed-off-by: Craig Small <>
8 years agosysctl --system loads default config file
Craig Small [Fri, 27 Dec 2013 12:08:14 +0000 (23:08 +1100)]
sysctl --system loads default config file

Commit cdca71e94506fbb921ab2c626be3ad05c4287498 fixed
the loading the sysctl.conf file, but had the logic
for checking the file exists reversed incorrectly.

8 years agotop: do not forget the fscanf %s terminating null byte
Jim Warner [Tue, 3 Dec 2013 21:00:00 +0000 (15:00 -0600)]
top: do not forget the fscanf %s terminating null byte


Signed-off-by: Jim Warner <>
8 years agoUpdate the library version to 3:0:0 v3.3.9
Craig Small [Tue, 3 Dec 2013 11:16:18 +0000 (22:16 +1100)]
Update the library version to 3:0:0

We have had some API changes which means the library version needs to
be incremented.

8 years agoFixed check for vmstat with stolen time
Craig Small [Tue, 3 Dec 2013 11:08:30 +0000 (22:08 +1100)]
Fixed check for vmstat with stolen time

Commit a8a4a4f added stolen time to vmstat, but broke the checks as
we have another column. This commit fixes the checks

8 years agotop: follow usual name conventions for global variable
Jim Warner [Sat, 30 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: follow usual name conventions for global variable

A recent change involving a one cycle stderr redirect,
to handle a libnuma potential transgression, failed to
follow normal global variable naming conventions. This
patch will capitalize the 1st letter of 'Stderr_save'.

commit 35dc6dcc49cc9cf8cff4300cb03a38dbe44c05db

Signed-off-by: Jim Warner <>
8 years agotop: do not lie about purported alphabetical orderings
Jim Warner [Fri, 29 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: do not lie about purported alphabetical orderings

Excluding those special X_XON/X_XOF enums, which might
not even be present, restore strict collating order of
all the case labels in the task_show switch statement.

Also, adjust a few sort callbacks for the same reason.

Signed-off-by: Jim Warner <>
8 years agopmap: Including -p in the man page
Jaromir Capik [Tue, 26 Nov 2013 12:45:22 +0000 (13:45 +0100)]
pmap: Including -p in the man page

Previously the pmap man page was missing the -p switch.
This commit fixes that.

8 years agopmap: Including -c -C -n -N in the man page
Jaromir Capik [Tue, 26 Nov 2013 12:31:44 +0000 (13:31 +0100)]
pmap: Including -c -C -n -N in the man page

Previously the pmap man page was missing the -c -C -n -N switches.
This commit fixes that.

8 years agovmstat: support for time stolen from virtual machines
Jaromir Capik [Mon, 25 Nov 2013 16:39:14 +0000 (17:39 +0100)]
vmstat: support for time stolen from virtual machines

Surprisingly the code for the 'st' column was added in the past,
but wasn't enabled. The vmstat manual already contains the 'st'
column and this commit finally enables the feature.

8 years agovmstat: -w switch for wider output
Jaromir Capik [Mon, 25 Nov 2013 15:56:10 +0000 (16:56 +0100)]
vmstat: -w switch for wider output

This is a rework of the merge request #5 that unconditionally
forced the output to cross the 80 chars border.
With this commit users can switch to the wide output mode
with the -w option.

8 years agoNEWS: updated with the next procps-ng release of 3.3.9
Jim Warner [Fri, 15 Nov 2013 06:00:00 +0000 (00:00 -0600)]
NEWS: updated with the next procps-ng release of 3.3.9

Signed-off-by: Jim Warner <>
8 years agotop: increase the maximum number of displayable fields
Jim Warner [Wed, 13 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: increase the maximum number of displayable fields

The recent addition of namespaces, combined with those
potential suse out-of-memory fields, means that we are
close to the maximum number of fields poor ol' top can
display. Imagine, the really old top was limited to 26
fields (28 with the suse hack) and this top had neared
the version 'g' rcfile limits which were a healthy 55.

This patch adds another 15 fields to the maximum while
making it even easier to increase in the future. Also,
top still silently accommodates older config files all
the way back to the original pre-ng version top-3.2.8!

Signed-off-by: Jim Warner <>
8 years agotop: expand the man page to include namespaces support
Jim Warner [Mon, 11 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: expand the man page to include namespaces support

Signed-off-by: Jim Warner <>
8 years agotop: expand this program to include namespaces support
Jim Warner [Mon, 11 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: expand this program to include namespaces support

Signed-off-by: Jim Warner <>
8 years agolibrary: normalize recently added namespaces interface
Jim Warner [Sun, 10 Nov 2013 06:00:00 +0000 (00:00 -0600)]
library: normalize recently added namespaces interface

While 'invisible' thread subdirectories are accessible
under /proc/ with stat/opendir calls, they have always
been treated as non-existent, as is true with readdir.

This patch trades the /proc/#/ns access convention for
the more proper /proc/#/task/#/ns approach when thread
access is desired. In addition some namespace code has
been simplified and made slightly more efficient given
the calloc nature of proc_t acquisition and its reuse.

commit a01ee3c0b32d4c39aa83066ed61103343469527e

Signed-off-by: Jim Warner <>
8 years agotop: eliminate yet more gcc subscript resolution bloat
Jim Warner [Sat, 9 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: eliminate yet more gcc subscript resolution bloat

This patch is inspired by the 'minimize numa overhead'
patch. It trades the use of subscripts for pointers to
avoid gcc repeated subscript offset calculation bloat.

Now, throughout the cpus_refresh function, a subscript
will be resolved just once & this will (dramatically?)
reduce the path-length taken for each and every frame!

For example, a non-optimized compilation could produce
400+ fewer machine instructions through pointer usage.

[ ok, optimized compiles only save 18+ instructions! ]

Lastly, any residual 'symmetry only' crap is now gone!

Signed-off-by: Jim Warner <>
8 years agotop: minimize the statistics overhead for numa support
Jim Warner [Fri, 8 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: minimize the statistics overhead for numa support

A recent libnuma potential corruption problem solution
has caused me to reevaluate some associated numa logic
for efficiency. Here is a summary of the problems that
exist with current libnuma/user possible interactions:

. Whenever the numa library was present extra overhead
would always be incurred in maintaining the node stats
even when the '2' or '3' commands were not being used.

. As part of such overhead a separate loop was used to
reinitialize each cpu/node structure with each display
cycle so that prior accumulated totals were preserved.
Again, it didn't matter if numa data was really shown.

This commit attempts to refocus on the 'critical path'
costs in a running top by optimizing for the occasions
when numa node data is not being displayed. Under such
conditions, no extra overhead will be incurred whether
or not a distribution has the libnuma library present.

To achieve this goal, some additional overhead will be
incurred, but only when actually displaying numa data.
And all such new costs have been minimized in spite of
the gcc inclination to duplicate subscript resolution.

commit 24bd950cb2e1722d459461f0f9c0c30a4b9ffdaa

Signed-off-by: Jim Warner <>
8 years agotop: add some flexibility to dlopen() for numa support
Jim Warner [Thu, 7 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: add some flexibility to dlopen() for numa support

A recent libnuma potential corruption problem solution
suggests that libnuma could change in the future so as
to not spew to stderr. This then raises a question how
top could exploit any such library change since we are
currently locked into version #1 of the library by way
of our dlopen("", RTLD_LAZY) runtime call.

While not an ultimate solution, this commit will first
try for the most recent version of that library during
top's startup before trying the original
We do this via the unqualified library soname symlink.

For this new dlopen() call to succeed, technically the
numa 'devel' package would usually have been required,
but that's not always true with every distro. And when
the symlink isn't present, it can always be
manually added should a newer & better behaved library
arrive & users tire of the stderr warning at top exit.

commit 24bd950cb2e1722d459461f0f9c0c30a4b9ffdaa

Signed-off-by: Jim Warner <>
8 years agotop: expand on column highlight quirks in man document
Jim Warner [Wed, 6 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: expand on column highlight quirks in man document

Confession is supposed to be good for the sole, right?

After a senior moment regarding the 'x' toggle quirks,
and thinking top had somehow regressed, I concluded an
additional explanatory note might well be appropriate.

Those quirks were already documented under the 5d & 5e
topics. But there was no such caution documented under
the 'x' command explanation itself, found in topic 4c.

Signed-off-by: Jim Warner <>
8 years agotop: fix miscellaneous spelling errors in man document
Jim Warner [Tue, 5 Nov 2013 06:00:00 +0000 (00:00 -0600)]
top: fix miscellaneous spelling errors in man document

Signed-off-by: Jim Warner <>
8 years agopidof: fixing compiller warnings
Jaromir Capik [Mon, 4 Nov 2013 15:26:19 +0000 (16:26 +0100)]
pidof: fixing compiller warnings

Removing unused variable and adding explicit pid_t retype in printf.

8 years agotop: address some potential libnuma display corruption
Jim Warner [Sun, 3 Nov 2013 05:00:00 +0000 (00:00 -0500)]
top: address some potential libnuma display corruption

There is a chance that the libnuma library may corrupt
top's display with some stderr warning messages in the
event something under /sys/devices/system/node/ cannot
be accessed. And, while 2 overridable 'weak' functions
are provided to alter such behavior, we can't use them
since top dynamically links to the library via dlopen.

This commit will redirect stderr to '/dev/null' during
just the first screen display cycle. Thus we can avoid
the corruption which would have remained visible until
the underlining screen row's data had finally changed.

Lastly, this patch should allow such a library warning
to actually appear when one finally exits our program.

[ i think the libnuma folks should consider changing ]
[ the error/warning interfaces to accommodate dlopen ]
[ rather than forcing something like the ugly kludge ]
[ we have employed or libnuma dependency on everyone ]


Signed-off-by: Jim Warner <>
8 years agobuild-sys: treate new pidof the same as other products
Jim Warner [Sat, 2 Nov 2013 06:00:00 +0000 (00:00 -0600)]
build-sys: treate new pidof the same as other products

Signed-off-by: Jim Warner <>
8 years agops: possibility to display slice unit for a process
Lukas Nykryn [Wed, 7 Aug 2013 09:57:24 +0000 (11:57 +0200)]
ps: possibility to display slice unit for a process

Library systemd-login offers possibility to display
name of a systemd slice unit for specific pid.

This patch adds output option "slice" which will
show name of systemd slice unit.

To maintain compatibility with non-systemd systems,
procps must be configured with --with-systemd option
to enable this option.

8 years agopidof: support for omitted %PPID and additional separators
Jaromir Capik [Mon, 14 Oct 2013 13:38:33 +0000 (15:38 +0200)]
pidof: support for omitted %PPID and additional separators

This commit introduces support for special %PPID value that
can be passed to the -o option as a substitution for parent
PID. It also allows users to use two additional separators
for omitted PIDs - colon and semicolon.

8 years agoMerge branch 'master' of
Craig Small [Thu, 10 Oct 2013 23:09:07 +0000 (10:09 +1100)]
Merge branch 'master' of

8 years agoUpdate options to single strings
Craig Small [Thu, 10 Oct 2013 23:07:10 +0000 (10:07 +1100)]
Update options to single strings

To assist the translators, each option is a separate string.
This means if we add/change/delete an option the remaining ones
will just keep working and only the impacted option needs some
translation work on it.

8 years agopidof: minor fixes
Jaromir Capik [Mon, 7 Oct 2013 16:17:19 +0000 (18:17 +0200)]
pidof: minor fixes

Adding forgotten --check-root switch. Removing uneeded headers.

8 years agopidof: reimplemented from scratch (replacing sysvinit pidof)
Jaromir Capik [Fri, 4 Oct 2013 18:35:01 +0000 (20:35 +0200)]
pidof: reimplemented from scratch (replacing sysvinit pidof)

As the sysvinit becomes obsolete, some of the bundled tools
need to find a new home. The procps-ng project seems to be
the most suitable project for adopting the pidof tool.
This commit introduces a redesigned version of pidof
that satisfies the LSB requirements.
In corner cases the behaviour might differ from the former
one as the new version doesn't use any stat(2) calls.

8 years agoUpdate free text to help translators
Benno Schulenberg [Wed, 9 Oct 2013 22:56:44 +0000 (09:56 +1100)]
Update free text to help translators

Split up the free options so that each option has its own
gettext field, for ease of translating.

Signed-off-by: Craig Small <>
8 years agoSkip some tests if kill cannot be found
Craig Small [Wed, 9 Oct 2013 08:18:55 +0000 (19:18 +1100)]
Skip some tests if kill cannot be found

Some Debian pbuilders error out on some of the tests because
they cannot find kill to kill the test processes. Now if we
cannot find kill we skip those lot of tests.

Still need to work out why the S390 doesn't like test_sched


8 years agotop: swat bug impacting 'idle' mode & 'user' filtering
Jim Warner [Fri, 27 Sep 2013 17:12:12 +0000 (12:12 -0500)]
top: swat bug impacting 'idle' mode & 'user' filtering

When Other filtering was introduced the nature of what
constituted a displayed row changed. No longer would a
task_show() call guarantee that another line is shown.
Rather, a non-empty string must have also been tested.

Unfortunately, when any task window was being filtered
for 'idle' mode or a particular 'user', the proc index
was incremented twice due to the perils of copy/paste.
Combining such an index increment with the new test of
task_show results works fine if filtering is inactive.

This was a particularly insidious bug which meant that
an adjacent task would be skipped whenever the current
task met 'idle' and/or 'user' filter criteria, and was
not otherwise excluded due to 'Other' filter criteria.

And, since it was the very next task that was ignored,
the bug was very susceptible to a window's sort order.
This could be illustrated when filtering on some user,
while sorting on PID. Then, toggling Forest View could
make otherwise unseen tasks appear and then disappear.

User workarounds are possible via interactive commands
trading the 'i' and 'u'/'U' provisions for the 'o'/'O'
other filtering capability thus avoiding an extra i++.
But that is certainly less than ideal and doesn't help
the 3.3.7 and 3.3.8 distorted command line provisions.

( this little buggie may end up costing me my pocket )
( protector, my coding badge & maybe even my cubicle )

. bug originated with 'Other' filtering
commit 5edc6fb3174f1fd02bbfca61ec6d8a3a2e12f71c

Signed-off-by: Jim Warner <>
8 years agosysctl --system ignores missing /etc/sysctl.conf
Craig Small [Fri, 20 Sep 2013 12:34:32 +0000 (22:34 +1000)]
sysctl --system ignores missing /etc/sysctl.conf

sysctl --system would not correctly return the RC for files in
subdirectories and would insist on having /etc/sysctl.conf

This update makes two changes when using sysctl --system:
  - The RC status is ORed for each config file, meaning an error in
    any file is propated to the RC
  - If /etc/sysctl.conf doesn't exist we just don't load it


8 years agotop: restore the lost final newline when in Batch mode
Jim Warner [Tue, 17 Sep 2013 05:00:00 +0000 (00:00 -0500)]
top: restore the lost final newline when in Batch mode

This patch adds the final newline when exiting 'Batch'
mode. Interestingly, it has been missing since release
3.3.5 but undetected until the Redhat bugzilla report.


Signed-off-by: Jim Warner <>
8 years agomisc: correct additional errors from merge request #13
Jim Warner [Thu, 12 Sep 2013 17:12:12 +0000 (12:12 -0500)]
misc: correct additional errors from merge request #13

Additional errors resulting from merge request #13 are
being addressed in this commit. They involve two cases
of trailing whitespace and one xwarnx printf type arg.

. earlier build-sys fix
commit e2242cb943c620599913db3a68ccba06fa07ac36
. original merge
commit dd6f24dbed12e95235b3df49e550b5039e74e6d8

Signed-off-by: Jim Warner <>
8 years agobuild-sys: fix the fatal 'make dist' error for nsutils
Jim Warner [Wed, 11 Sep 2013 14:11:13 +0000 (09:11 -0500)]
build-sys: fix the fatal 'make dist' error for nsutils

While a 'make dist' appeared to work fine without this
patch, the nsutils.h file was missing from the include
subdirectory. Thus the tarball could not support make.

commit dd6f24dbed12e95235b3df49e550b5039e74e6d8

Signed-off-by: Jim Warner <>
8 years agoFix integer overflow in getstat()
Craig Small [Wed, 11 Sep 2013 11:57:10 +0000 (21:57 +1000)]
Fix integer overflow in getstat()

Merge request 16

8 years agoFix off-by-one by pmap
Craig Small [Wed, 11 Sep 2013 11:52:49 +0000 (21:52 +1000)]
Fix off-by-one by pmap

Merge request 15

8 years agoMerge commit 'refs/merge-requests/13' of git:// into merge...
Craig Small [Wed, 11 Sep 2013 11:34:05 +0000 (21:34 +1000)]
Merge commit 'refs/merge-requests/13' of git:// into merge-requests/13


8 years agoMerge commit 'refs/merge-requests/2' of git:// into merge...
Craig Small [Wed, 11 Sep 2013 10:50:48 +0000 (20:50 +1000)]
Merge commit 'refs/merge-requests/2' of git:// into merge-requests/2


8 years agofree: reusing 'shared' for Shmem
Adrian Brzezinski [Tue, 23 Jul 2013 18:33:00 +0000 (20:33 +0200)]
free: reusing 'shared' for Shmem

Previously the shared memory column was always zero
for 2.6 series kernels (and later) due to the fact,
that the value was taken from the MemShared entry
that disappeared with 2.6 series kernels.
Later a new Shmem entry appeared in the /proc/meminfo
file and the 'shared' column now displays either
the MemShared or the Shmem value (depending on their
presence - the presence is mutually exclusive).
If none of the two entries is exported by the kernel,
then the column is zero.

8 years agofree: clarification of credits for the Shmem support
Jaromir Capik [Mon, 9 Sep 2013 14:15:54 +0000 (16:15 +0200)]
free: clarification of credits for the Shmem support

An unpleasant thing happened when I comitted the shmem support
for the 'free' tool. We already had a merge request from
Adrian Brzezinski in the queue, doing exactly the same.
As Adrian deserves credits, I'm reverting the change
and re-applying with the next commit in order to make
him a part of the project history.

8 years agotop: swat bug affecting batch mode and width provision
Jim Warner [Thu, 29 Aug 2013 05:00:00 +0000 (00:00 -0500)]
top: swat bug affecting batch mode and width provision

Normally, the internally tracked 'Screen_cols' can not
exceed the lessor of 512 or actual screen width. There
was one case, however, where that 512 byte upper limit
was no longer properly imposed as it should have been.

When operating in 'Batch' mode the actual screen width
was allowed to be exceeded when the optional -w switch
was also used. But, it should never have exceeded 512.

This patch ensures the upper limit is always observed.


Signed-off-by: Jim Warner <>
8 years agotop: correct, improve and otherwise tweak configs_read
Jim Warner [Tue, 20 Aug 2013 05:00:00 +0000 (00:00 -0500)]
top: correct, improve and otherwise tweak configs_read

This patch addresses the four '-Wunused-result' errors
generated whenever an optimized compile is invoked. It
also made the configs_read() guy a little more robust.

In the process, some logic was rearranged slightly and
some comments were re-indented simply for consistency.

warning: ignoring return value of 'fgets', declared with attribute warn_unused_result [-Wunused-result]
warning: ignoring return value of 'fscanf', declared with attribute warn_unused_result [-Wunused-result]

Signed-off-by: Jim Warner <>
8 years agotop: modest efficiency change to message line handling
Jim Warner [Thu, 15 Aug 2013 05:00:00 +0000 (00:00 -0500)]
top: modest efficiency change to message line handling

When the final solution for cursor positioning for all
^Z or ^C cases was introduced the revised placement of
message line management introduced with the window mgr
'screen' refactor was retained. Those two commits mean
that a former tgoto was no longer needed when clearing
that msg line or displaying the scroll coordinate msg.

This patch eliminates the tgoto employed by frame_make
while assimilating a now defunct show_scroll function.

. final cursor positioning for ^Z or ^C
commit 46a1356219cfab67240704af9cd73b58a20d4232
. 'screen' window manager refactor
commit 0fe393ff270922cd4f6edbcaabba006314e73a37

Signed-off-by: Jim Warner <>
8 years agotop: hint that Summary Area 'cached' is Mem (not Swap)
Jim Warner [Fri, 9 Aug 2013 05:00:00 +0000 (00:00 -0500)]
top: hint that Summary Area 'cached' is Mem (not Swap)

This patch just provides a visual clue to separate the
values reported for cached Memory from other values on
the Swapped line (which is being shared due to space).


Signed-off-by: Jim Warner <>
8 years agoIncrease watch interval.
Craig Small [Sun, 25 Aug 2013 07:43:20 +0000 (17:43 +1000)]
Increase watch interval.

watch would only use an interval of up to 4294 seconds and silently
change to this limit. The 4294 seconds is 2^32/10^6 or how many
microseconds fit into unsigned int.

This change increases the limit to 2^32 seconds which is
approximately 136 years. This should be ok for now. Anything above
the old limit now uses sleep() instead of usleep() which only uses
integers (so 9999.123 seconds will be 9999 seconds)

This bug was first reported in 2006 and included a patch by
Stephen Kratzer. The patch was updated to fit the current source.


Signed-off-by: Craig Small <>
9 years agobuild-sys: add support for silent buils
Karel Zak [Tue, 13 Aug 2013 13:21:33 +0000 (15:21 +0200)]
build-sys: add support for silent buils

The automake AM_SILENT_RULES macro is supported since automake 1.11
(which is required for procps). The silent functionality is enabled by
default, you can change it by:

  ./configure --disable-silent-rules
   make V=1

Note that make still prints compiler errors, etc.

Signed-off-by: Karel Zak <>
9 years agobuild-sys: correct one AC_ARG_ENABLE() cleanup default
Jim Warner [Tue, 13 Aug 2013 14:44:54 +0000 (09:44 -0500)]
build-sys: correct one AC_ARG_ENABLE() cleanup default

Signed-off-by: Jim Warner <>
9 years agobuild-sys: cleanup AC_ARG_ENABLE() usage
Karel Zak [Tue, 13 Aug 2013 13:21:31 +0000 (15:21 +0200)]
build-sys: cleanup AC_ARG_ENABLE() usage

 * don't duplicate default behavior with [enable_foo=$enableval]
 * don't introduce things like disable_* variables
 * use everywhere the same coding style

Signed-off-by: Karel Zak <>
9 years agotop: Fixing duplicate words in the man page
Jaromir Capik [Fri, 9 Aug 2013 16:11:01 +0000 (18:11 +0200)]
top: Fixing duplicate words in the man page

9 years agops: address a potential 'newline' quirk the libselinux
Jim Warner [Wed, 7 Aug 2013 17:58:57 +0000 (12:58 -0500)]
ps: address a potential 'newline' quirk the libselinux

Sometimes with libselinux present but SELinux inactive
the context reported is "unconfined" which contains an
embedded newline. This then causes misalignment of any
subsequent data. So, ps will now protect against that.


Signed-off-by: Jim Warner <>
9 years agotop: correct cursor positioning for all ^Z or ^C cases
Jim Warner [Sat, 6 Jul 2013 05:00:00 +0000 (00:00 -0500)]
top: correct cursor positioning for all ^Z or ^C cases

Some more (very obscure) conditions where a suspension
or program end might embed the shell prompt within top
output have been uncovered beyond the 2 already known.

We had already covered some suspend/end contingencies:
1. the users were using the 'fields management' screen
2. the users were prompted for any line oriented input

However, there remained some situations where ^Z or ^C
could still produce a misplaced cursor + shell prompt:
3. the 'g' command while waiting for the window choice
4. the 'W' command if about to overwrite an old rcfile
5. the '=' command when exploiting the Inspect feature
6. the period during which any error message was shown

But, even when all those bases are covered there still
remains a remote possibility that such interrupts will
occur during a top repaint cycle. So rather than throw
yet more code at these self-inflicted problems perhaps
it is better if we just throw in the proverbial towel.

Thus, I'll take the only sane approach and restore the
results expected ever since top's inception and before
scrollback buffers entered the picture. Namely, with a
^Z or ^C the cursor will be placed on the final screen
row. That usually means it will immediately follow the
last output line but it may follow many blank lines if
the user interrupts top when *not* on the main screen.

. expanded repositioning (for line oriented input)
commit 33104a2bcc321495107d72e4cfee4090b1d90f76
. introduced repositioning (for fields management)
commit 5c974ff44da4fbbb9170dd15bdd81555c62c31a9
. scrollback buffers (the cursor handling changes)
commit dedaf6e1a81738ff08ee8e8523871e12f555ad6d

Signed-off-by: Jim Warner <>
9 years agotop: tweak cursor state code to swat an obscure buglet
Jim Warner [Fri, 5 Jul 2013 05:00:00 +0000 (00:00 -0500)]
top: tweak cursor state code to swat an obscure buglet

An obscure bug has been discovered where a 'W' with an
old rcfile, warning against overwrite, would display a
cursor that should normally be hidden. This followed a
user's reply. So some logic was rearranged just a bit.

Signed-off-by: Jim Warner <>
9 years agotop: correct, improve or otherwise tweak some comments
Jim Warner [Thu, 4 Jul 2013 05:00:00 +0000 (00:00 -0500)]
top: correct, improve or otherwise tweak some comments

Signed-off-by: Jim Warner <>
9 years agofree: reusing 'shared' for Shmem
Jaromir Capik [Wed, 7 Aug 2013 16:21:43 +0000 (18:21 +0200)]
free: reusing 'shared' for Shmem

Previously the shared memory column was always zero
for 2.6 series kernels (and later) due to the fact,
that the value was taken from the MemShared entry
that disappeared with 2.6 series kernels.
Later a new Shmem entry appeared in the /proc/meminfo
file and the 'shared' column now displays either
the MemShared or the Shmem value (depending on their
presence - the presence is mutually exclusive).
If none of the two entries is exported by the kernel,
then the column is zero.

9 years agops: making the libselinux support configurable
Jaromir Capik [Wed, 7 Aug 2013 15:52:38 +0000 (17:52 +0200)]
ps: making the libselinux support configurable

Previously the libselinux support was present
in the sources, but disabled with a preprocessor
condition (#if 0).
From now the libselinux support can be enabled with
the --enable-libselinux switch available
in the configuration script. That way is more
flexible than local patches modifying the condition
value from 0 to 1.

9 years agopwdx: fails when run in a nonexistent locale
Colin Watson [Mon, 5 Aug 2013 10:52:22 +0000 (20:52 +1000)]
pwdx: fails when run in a nonexistent locale

pwdx rather mysteriously fails with "invalid process id" when run in a
nonexistent locale (e.g. "LC_ALL=foo pwdx $$").  This is because it
fails to obey the documented calling sequence for strtol - that is, set
errno to 0 before the call - and thus the errno from the setlocale
failure bleeds over into its check for whether strtol failed.


Signed-off-by: Craig Small <>
9 years agoConditional test vmstat -p
Craig Small [Wed, 31 Jul 2013 11:54:53 +0000 (21:54 +1000)]
Conditional test vmstat -p

9 years agosysctl, w: miscelaneous usage/man fixes
Jaromir Capik [Fri, 19 Jul 2013 12:50:02 +0000 (14:50 +0200)]
sysctl, w: miscelaneous usage/man fixes

This commit is a result of RH man page mass scan, that failed.
All the broken/missing switches should be fixed now.

9 years agotop: cursor repositioning includes line oriented input
Jim Warner [Mon, 1 Jul 2013 05:00:00 +0000 (00:00 -0500)]
top: cursor repositioning includes line oriented input

A recent patch introduced the ability to recognize the
need to reposition the cursor at suspension or program
end. There remained unmet 1 additional potential need.

This commit extends that ability to embrace line input
so that if a user issues ^Z or ^C while being prompted
the resulting shell output will no longer be embedded.

commit 5c974ff44da4fbbb9170dd15bdd81555c62c31a9

Signed-off-by: Jim Warner <>
9 years agotop: refine some miscellaneous signals interrupt stuff
Jim Warner [Sun, 30 Jun 2013 05:00:00 +0000 (00:00 -0500)]
top: refine some miscellaneous signals interrupt stuff

This commit mostly justs renames a few identifiers but
it also will now suppress any end-of-job report if top
wasn't ended via the 'q' key convention (i.e. signal).

Signed-off-by: Jim Warner <>
9 years agotop: enable screen contents preservation at end-of-job
Jim Warner [Fri, 28 Jun 2013 05:00:00 +0000 (00:00 -0500)]
top: enable screen contents preservation at end-of-job

The title of this commit is actually quite misleading.

Were it more accurate, it would at least mention a tty
emulator's scrollback buffer, which was the cumulation
of a long pursuit to reduce the SIGWINCH overhead when
a window manager carelessly floods an application with
that signal *while* a user is still resizing a window!

Disabling and enabling that scrollback buffer resulted
in the final top display replaced with original screen
contents, a phenomenon acknowledged at the time but it
also represented a user interface change which has now
produced the first request for return to old behavior.

After the SIGWINCH dust settled, another problem arose
regarding behaviors under the 'screen' window manager.
In response, top was refactored a bit to avoid display
corruption. That was before discovering 'screen' could
duplicate the scrollback buffer behavior top expected.

As it turns out, the 'screen' refactoring had probably
made scrollback buffer manipulation unnecessary. Still
one could argue that a window should not be allowed to
scroll while a constantly updating program was active.

The solution represented in this commit returns former
behavior at program end (retaining top's last screen).
And if we ever wish to disable scrollback buffers, the
associated logic was retained but made conditional. It
is not reflected in but might be someday.

Lastly, this commit corrects cursor positioning when a
^C is issued under 'Fields Management' at any terminal
that didn't have a scrollback buffer (i.e. a console).

. screen program refactor
commit 0fe393ff270922cd4f6edbcaabba006314e73a37
. scrollback buffer disabled
commit dedaf6e1a81738ff08ee8e8523871e12f555ad6d
. sigwinch management defines
commit adca737758e5afc7be344a736953931894cbc19f
commit 4f33b6b8c56464b4044deb29a3bb0e32622e108f

Signed-off-by: Jim Warner <>
9 years agolibrary: for atexit() support, fix fileutils for EPIPE
Jim Warner [Tue, 25 Jun 2013 05:00:00 +0000 (00:00 -0500)]
library: for atexit() support, fix fileutils for EPIPE

When fileutils with stream error checking was borrowed
from GNU lib, an omission was also propagated where an
errno of EPIPE wouldn't be preserved in close_stream()
making a test for EPIPE in close_stdout() meaningless.

This patch corrects such oversight so that an errno of
EPIPE no longer produces 'write error' at program end.

( gnulib provides for optionally ignoring EPIPE, but )
( if a program chooses to ignore it, then their code )
( appears to suffer from this close_stream oversight )

. original fileutilis addition
commit c7cf98b0e03780f78abe5275c6fb282f71a2369f
. bugzilla report

9 years agotop: trade two groff_mdoc macros for groff equivalents
Jim Warner [Thu, 20 Jun 2013 05:00:00 +0000 (00:00 -0500)]
top: trade two groff_mdoc macros for groff equivalents

The .Bd and .Ed macros, from the mdoc collection, have
caused a few raised eyebrows (lintian, doclifter, etc)
in the past. With this commit, we will trade their use
for the standard existing groff .nf & .fi equivalents.

Signed-off-by: Jim Warner <>
9 years agoFix overflow of "running" value in getstat() 16
Vadim Ushakov [Tue, 11 Jun 2013 04:09:05 +0000 (12:09 +0800)]
Fix overflow of "running" value in  getstat()

It seems in some cases procs_running field of /proc/stat can contain 0 even if vmstat itself is running. At least this can be reproduced on Linux 3.9.3 compiled with BFS scheduler.

Since getstat() decrements value of procs_running by 1, we get overflow:

$ vmstat 1
procs -----------memory---------- ---swap-- -----io---- -system-- ----cpu----
 r  b   swpd   free   buff  cache   si   so    bi    bo   in   cs us sy id wa
 0  0 667732 918996  57376 911260   21   30    36    40   98   45 14 82  4  1
 4294967295  0 667728 916716  57376 911264    8    0     8     0 1958 3733 28  7 65  1
 0  0 667700 915996  57376 911416   24    0   152     0 1735 3600 23  5 71  1
 4294967295  0 667700 915872  57376 911392    0    0     0     0 1528 3165 21  4 76  0

9 years agotop: add the major version to dlopen of libnuma soname
Jim Warner [Thu, 6 Jun 2013 05:00:00 +0000 (00:00 -0500)]
top: add the major version to dlopen of libnuma soname

When the plug-in approach to NUMA support was added, I
carelessly employed the compile-time linker convention
for naming the library. Technically this then required
the 'devel' package for NUMA support to be present for
the unqualified soname symlink to be available. Either
that or one must have manually created such a symlink.

This commit adds the missing major version to dlopen()
of so simply having a more likely package
such as 'numactl' will enable both '2' & '3' commands.

. initial dlopen support
commit edba932a7e9b950dd91bc486e107788e977a5186

Signed-off-by: Jim Warner <>
9 years ago3.3.8 NEWS and API v3.3.8
Craig Small [Sat, 25 May 2013 22:03:01 +0000 (08:03 +1000)]
3.3.8 NEWS and API

NEWS mentions systemd unit support for ps

Library REVISION (internal change only) incremented

Signed-off-by: Craig Small <>
9 years agoprocps-ng testsuite : simplify DEJAGNU workaround
Gilles Espinasse [Thu, 23 May 2013 21:35:09 +0000 (23:35 +0200)]
procps-ng testsuite : simplify DEJAGNU workaround

No need to distribute an empty file
/dev/null is enought to silent global config file warning

Signed-off-by: Gilles Espinasse <>
9 years agoprocps-ng : tests, remove which dependency
Gilles Espinasse [Thu, 23 May 2013 21:22:49 +0000 (23:22 +0200)]
procps-ng : tests, remove which dependency

which is a separate package that may not be available and is not yet build on Linux from scratch build order.
Instead use posix command -v. command -v  is a builtin working with bash-4.2, 3.0.25 or even old bash-2.05 or current debian dash

Signed-off-by: Gilles Espinasse <>
9 years agoskip kill test too if running runtest
Gilles Espinasse [Sat, 25 May 2013 21:25:55 +0000 (07:25 +1000)]
skip kill test too if running runtest

I find more readable instead of make check to run
cd testsuite && make site.exp && DEJAGNU=global-conf.exp runtest

But in that case, kill.exp was still trying to run with
ERROR: tcl error sourcing ./kill.test/kill.exp.
ERROR: couldn't execute "/usr/src/procps-ng-3.3.7/kill": no such file or

Simply return from test if kill is not build

Signed-off-by: Gilles Espinasse <>
9 years agoMerge branch 'master' of
Craig Small [Thu, 23 May 2013 11:54:33 +0000 (21:54 +1000)]
Merge branch 'master' of

9 years agotop: change man document license used from GFDL to GPL
Jim Warner [Wed, 22 May 2013 05:00:00 +0000 (00:00 -0500)]
top: change man document license used from GFDL to GPL

This patch brings the man document into agreement with
Debian's current licensing standards. The 3 urls below
discuss their concerns over the GFDL and also document
quite a few additional packages with similar licenses.

( jeeze, 12+ friggin' years & finally someone grep'd )
( for what was just a couple of 'invariant' sections )

. why gfdl is unsuitable
. draft position statement
. busted by lintian

Signed-off-by: Jim Warner <>
9 years agoFix off-by-one in pmap 15
Cristian Rodríguez [Wed, 22 May 2013 22:22:37 +0000 (18:22 -0400)]
Fix off-by-one in pmap

When procps is built with gcc 4.8 address sanitizer

static int one_proc(proc_t * p)..
char smap_key[20];
(sscanf(mapbuf, "%20[^:]: %llu", smap_key..

rightfully results in an overflow and the program aborts.

9 years agops: remove sd_ prefix from systemd output options
Václav Pavlín [Fri, 10 May 2013 11:39:00 +0000 (13:39 +0200)]
ps: remove sd_ prefix from systemd output options

ps : This patch removes sd_ prefix from recently added systemd output options
to let them look more tied with the system.

Patch does not change behaviour of these options, only modifies their
representation to user.

9 years agotop: inoculated against a window manager like 'screen'
Jim Warner [Sat, 18 May 2013 05:00:00 +0000 (00:00 -0500)]
top: inoculated against a window manager like 'screen'

If top were invoked under the 'screen' window manager,
writing the terminfo string 'exit_ca_mode' at top exit
would not restore the display to the state existing at
the time top was started. That's what occurs normally.
The net result of that failure was a corrupted screen.

However, there is a 'screen' configuration option that
will produce proper 'rmcup' behavior, but it is off by
default. That screencr option is known as 'altscreen'.
I stumbled across this provision by cloning the screen
git repository then searching for references to 'cup'.

If 'altscreen on' had been in either the /etc/screenrc
or the $HOME/.screenrc configuration file, my poor old
top would never have been accused of such corruptions.
Of course, the Programming Gods decree that any simple
solution for our problem must always be revealed last.

So before discovering that rc option, another approach
was taken involving top only. With just a little extra
refactoring of top display logic he was made immune to
any such quirk in the implementation of 'smcup/rmcup'.

I always feel good about any enhancement that actually
reduces the total number of lines of code. Even though
this change involved mostly rearranging some logic, it
yielded one less line (can't judge by diffstat because
of braces & notes). Anyway, rather than requiring some
change to a screenrc file, now we are self-sufficient.

  procps ---------------------------------------------,3
. top : disable tty scrollback buffer to improve SIGWINCH
commit dedaf6e1a81738ff08ee8e8523871e12f555ad6d
  screen ---------------------------------------------
. Improve cursor store/restore on smcup/rmcup.
commit f95352946080be803b794c9f2733d8c809c1a39a
. Fix using alternate screen buffers in some cases.
commit ad56f746c6243d45124485d198d577bdbb78071c

Signed-off-by: Jim Warner <>
9 years agoNEWS: update of some changes destined for next release
Jim Warner [Wed, 15 May 2013 05:00:00 +0000 (00:00 -0500)]
NEWS: update of some changes destined for next release

Signed-off-by: Jim Warner <>
9 years agotop: provide a build option to change memory precision
Jim Warner [Tue, 14 May 2013 05:00:00 +0000 (00:00 -0500)]
top: provide a build option to change memory precision

When summary & task area memory scaling was introduced
in release 3.3.6, the memory field widths were widened
slightly so unscaled KiB values could be provided more
consistently and scaled values (beyond MiB) could show
3 decimal places of precision. However, some users may
prefer the former widths/precisions for memory fields.

This commit will provide a build time configure option
to return top to those former defaults as a compliment
to a new %CPU & %MEM field precision configure option.

commit 21e550bc080eb30f503b2ca6fe4e9cb12b8a1616

Signed-off-by: Jim Warner <>
9 years agotop: revert %CPU and %MEM precision to former defaults
Jim Warner [Tue, 14 May 2013 05:00:00 +0000 (00:00 -0500)]
top: revert %CPU and %MEM precision to former defaults

When summary & task area memory scaling was introduced
in release 3.3.6, the percentage columns were expanded
to provide 3 decimal places of precision. In hindsight
that may have been overkill, making those columns more
of a distraction than useful, with just too much info.

This patch will revert those columns to the former one
decimal place. And as was true, that decimal point may
be sacrificed depending on the number of cpus present.

And, in case anyone might prefer additional precision,
a build option can provide it (--enable-wide-percent).

commit 21e550bc080eb30f503b2ca6fe4e9cb12b8a1616

Signed-off-by: Jim Warner <>
9 years agotop: eliminate troublesome apostrophes in man document
Jim Warner [Mon, 13 May 2013 05:00:00 +0000 (00:00 -0500)]
top: eliminate troublesome apostrophes in man document

Some versions of man have had trouble with the leading
apostrophe employed with interactive commands/toggles.
Instead, a '?' would be displayed in their place. With
this patch all such conventions have now been removed.

Signed-off-by: Jim Warner <>
9 years agotop: reduce function call overhead in the NUMA support
Jim Warner [Mon, 13 May 2013 05:00:00 +0000 (00:00 -0500)]
top: reduce function call overhead in the NUMA support

Signed-off-by: Jim Warner <>
9 years agoFix btime handling reading from /proc/stat
Markus Mayer [Wed, 8 May 2013 23:34:44 +0000 (16:34 -0700)]
Fix btime handling reading from /proc/stat

Function getbtime() currently makes the assumption that btime==0 equals
btime not being present in /proc/stat. This is not quite accurate, as
timestamp 0 is, in fact, also a valid time (Epoch), and /proc/stat may
report it as such.

We introduce a flag to indicate whether btime was found in /proc/stat.
In this way, btime==0 becomes a valid case, provided /proc/stat
actually reports this as the boot time.

procps can still detect the case of btime actually not being reported
by the kernel.

Signed-off-by: Markus Mayer <>
9 years agobuild-sys: allow a build when is truly absent
Jim Warner [Sun, 5 May 2013 05:00:00 +0000 (00:00 -0500)]
build-sys: allow a build when is truly absent

The earlier commit purporting to allow top to be built
in the absence of that dynamic linking library stopped
just a little short of the truth. So this will fix it.

commit 5686877cd4c83a2daf1be6f2f7f93cd2c1451e75

Signed-off-by: Jim Warner <>
9 years agoUpdated
Craig Small [Sun, 5 May 2013 00:07:30 +0000 (10:07 +1000)]

Been a while since we ran a re-scan over the autotools files. This
change modernises the configure file.  Not a great deal of changes
required to bring us up to date, autoscan doesn't understand our
optional things, which is fine.

9 years agotop: introduce a plug-in approach for the NUMA support
Jim Warner [Thu, 25 Apr 2013 05:00:00 +0000 (00:00 -0500)]
top: introduce a plug-in approach for the NUMA support

The NUMA/Nodes support in top has gone through several
evolutions (primarily dealing with build-sys options).
With this commit the library dependency issues are now
moot and the responsibilities for run-time loading and
dynamic linking are assumed by the top program itself.

Henceforth, if top is executed in an environment where is present, top will offer such extensions.
Even more importantly, when a missing libnuma is later
installed, top will offer numa support auto-magically.
All NUMA/Node build-sys dependencies are thus removed.

The former NUMA_ENABLED define has become NUMA_DISABLE
should anyone wish to test user interface implications
in an environment that *does* have libnuma. It is also
represented as the ./configure option: --disable-numa.

Lastly, the 't' (View_STATES) toggle will be forced on
for sanity whenever the '1', '2' or '3' keys are used.

. original idea from: Dr. Fink <>,18
. original numa suppoort
commit 8d989c68c068541a814bf0d2340ac9b0373f24b5

Signed-off-by: Jim Warner <>
9 years agobuild-sys: eliminate dependencies for the NUMA support
Jim Warner [Thu, 25 Apr 2013 05:00:00 +0000 (00:00 -0500)]
build-sys: eliminate dependencies for the NUMA support

Oh that poor ol' build system. With this patch it will
have gone through three separate incarnations in terms
of NUMA/Node support. Those 3 iterations consisted of:

1. A 'porridge too hot' where the top numa support was
enabled if it was built in the presence of libnuma and
the numa.h header. But if the numa library wasn't part
of core packages, that would have broken poor old top.

2. A 'porridge too cold' where numa support was off by
default and must have been explicitly enabled when the
./configure script was run. This could have meant that
distros might not distribute a numa-aware procps, even
though their numa library would have been distributed.

3. And this 'porridge' where the top numa support will
become a 'plug-in' feature activated when the presence
of can be verified at runtime. We'll do our
own loading and symbol resolution (with some help from
dlopen in libdl). Thus maintainers' responsibility for
enabling numa support and then satisfying that library
dependency is now an entirely optional --disable-numa.

As Goldilocks might say about our current
"Ummm, I think this porridge tastes just about right".

. 1) too-hot
commit 87ac6383bb575d964ba9ef6a100b61cdcdc7f15d
. 2) too-cold
commit 53fd7dd1ed120f901cbb31e69453720b038a7ac6
. original idea from: Dr. Fink <>,18

Signed-off-by: Jim Warner <>
9 years agointegrate systemd-login support to ps
Craig Small [Thu, 25 Apr 2013 22:28:18 +0000 (08:28 +1000)]
integrate systemd-login support to ps

Merge commit 'refs/merge-requests/12' of git:// into merge-requests/12

9 years agops: possibility to display seat for a process 12
Lukas Nykryn [Mon, 22 Apr 2013 11:16:33 +0000 (13:16 +0200)]
ps: possibility to display seat for a process

Library systemd-login offers possibility to display
name of seat for a session on multi-seat systems.

This patch adds output option "sd_seat" which will
show name of seat or "-", when name of seat can not
be determined, but "seat0" should always exist.

To maintain compatibility with non-systemd systems,
procps must be configured with --with-systemd option
to enable this option.

9 years agops: possibility to display systemd user unit for a process
Lukas Nykryn [Mon, 22 Apr 2013 10:56:02 +0000 (12:56 +0200)]
ps: possibility to display systemd user unit for a process

Library systemd-login offers possibility to display
name of systemd user unit for specific pid. Note that not all
processes are part of a user unit.

This patch adds output option "sd_uunit" which will
show name of user unit or "-", when process does not belong
to any user unit. This is similar to "sd_unit" but applies
to user units instead of system units.

To maintain compatibility with non-systemd systems,
procps must be configured with --with-systemd option
to enable this option.

9 years agops: possibility to display machine name for a process
Lukas Nykryn [Mon, 22 Apr 2013 10:33:31 +0000 (12:33 +0200)]
ps: possibility to display machine name for a process

Library systemd-login offers possibility to display
the name of the VM or container which process belongs to.

This patch adds output option "sd_machine" which will
show machine name or "-" when the name can not be determined.

To maintain compatibility with non-systemd systems,
procps must be configured with --with-systemd option
to enable this option.

9 years agops: possibility to display uid of session owner for a process
Lukas Nykryn [Wed, 17 Apr 2013 08:32:50 +0000 (10:32 +0200)]
ps: possibility to display uid of session owner for a process

Library systemd-login offers possibility to display the Unix
user identifier of the owner of the session of a process.
This information will also be displayed for user processes which
are shared between multiple login sessions of the same user,
where sd_session will be blank.

This patch adds output option "sd_ouid" which will show
user UID or "-", when there is no owner for a process.

To maintain compatibility with non-systemd systems,
procps must be configured with --with-systemd option
to enable this option.

9 years agops: possibility to display login session for a process
Lukas Nykryn [Tue, 16 Apr 2013 13:58:32 +0000 (15:58 +0200)]
ps: possibility to display login session for a process

Library systemd-login offers possibility to display name
of login session for specific pid.
Note that not all processes are part of a login session
(e.g. system service processes, user processes that are shared
between multiple sessions of the same user, or kernel threads).

This patch adds output option "sd_session" which will
show name of session or "-", when process does not belong
to any session.

To maintain compatibility with non-systemd systems,
procps must be configured with --with-systemd option
to enable this option.

9 years agops: possibility to display systemd unit for a process
Lukas Nykryn [Tue, 16 Apr 2013 12:42:30 +0000 (14:42 +0200)]
ps: possibility to display systemd unit for a process

Library systemd-login offers possibility to display
name of a systemd unit file for specific pid. Note that
not all processes are part of a system unit/service
(e.g. user processes, or kernel threads).

This patch adds output option "sd_unit" which will
show name of systemd unit or "-", when process does not
belong to any unit.

To maintain compatibility with non-systemd systems,
procps must be configured with --with-systemd option
to enable this option.

9 years agolibrary: dynamic buffer management even more efficient
Jim Warner [Sun, 14 Apr 2013 05:00:00 +0000 (00:00 -0500)]
library: dynamic buffer management even more efficient

One recent patch to dynamic buffer management involved
over-allocating the buffer increase to lessen calls to
xrealloc. That was successful, but the actual increase
amount did not attempt to optimize size or alignments.

With this commit, we'll copy an approach recently used
by the top program and round up buffer sizes to 1 KiB.
More importantly, while buffers are quickly reaching a
KiB optimum multiple, no memcpy will ever be employed!

To illustrate just how effective top's algorithm would
be, just change the initial and subsequent allocations
from the current 1024 bytes to just a single byte then
add an fprintf.  Those one byte reallocations while on
the way to optimum buffer size will be a one-time cost
and won't represent any recurring performance penalty.

( gosh, that top program *must be* one fart smeller, )
( or was that a smart feller, i can't remember which )

commit 6d605f521c54a7011b7a255762621febda4f67e8
commit a45dace4b82c9cdcda7020ca5665153b1e81275f

Signed-off-by: Jim Warner <>
9 years agotop: snapshot /proc/stat reads to minimize distortions
Jim Warner [Sun, 14 Apr 2013 05:00:00 +0000 (00:00 -0500)]
top: snapshot /proc/stat reads to minimize distortions

Almost forever, top has been accessing the /proc/stat/
directory one line at a time until either smp_num_cpus
was reached or (more recently) Screen_rows is reached.

When NUMA/Nodes support is enabled screen rows will no
longer serve as a limit because all cpus must be read.

With this commit, the entire /proc/stat/ directory can
be read at once so all statistics will be frozen. Thus
individual cpus will no longer keep "ticing" until top
gets around to accessing them via some separate fgets.

The distortion this commit eliminates was quite easily
seen when comparing old/new tops using: individual cpu
stats vs. cpu summary; a healthy delay interval of 3-5
seconds; manually synchronized update cycles (the hard
part); some system loading (maybe another top at -d0).

Additionally, this patch eliminates some long standing
unnecessary initialization made possible because of an
allocation via calloc. If some parts are never touched
by sscanf due to a kernel version, it's unnecessary to
repeatedly re-initialize those portions to zero again.

. numa extensions added
commit 8d989c68c068541a814bf0d2340ac9b0373f24b5
. useless initialization evolution (old to new)
commit e54c8239b1c46cf7b99dd6120c9090303fa969c8
commit 9278134e49b6bf4d6e05c40f63cc6d6d6936e01b
commit fd62123562c2b71f292d3d3ee1a085709048b11a
commit f348575edc915db9df89acfb3f292920726ffe9a

Signed-off-by: Jim Warner <>