]> granicus.if.org Git - procps-ng/log
procps-ng
2 years agotestsuite shmid can be hex too
Craig Small [Tue, 3 May 2022 09:37:39 +0000 (19:37 +1000)]
testsuite shmid can be hex too

Previously the match for shmid was \d+ but the variable is printed
as a hex number, updated the regex to suit.

Added some changes for pmap test so if the test_shm process fails
we just skip past it.

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agotop: turn off the 'Ctrl' bottom window after some keys
Jim Warner [Tue, 3 May 2022 05:00:00 +0000 (00:00 -0500)]
top: turn off the 'Ctrl' bottom window after some keys

To complete the implementation of this new feature, we
must address the problem created by any commands which
offer their own full replacement displays. Should that
'Ctrl' bottom window be active, a return from any such
command leaves that lower portion of a terminal blank.

While we could issue a 'TAGGED_TOSS' macro right after
invoking any of those 4 (currently) affected commands,
it is wrong to make them aware of this 'tagged' stuff.

So, I'll opt for a more generalized solution whereby a
notice regarding a corrupted screen is broadcast. Then
the logic already aware of 'tagging' can deal with it.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: allow sticky 'Ctrl' bottom window code by #define
Jim Warner [Mon, 2 May 2022 05:00:00 +0000 (00:00 -0500)]
top: allow sticky 'Ctrl' bottom window code by #define

After making that new bottom window more persistent, I
decided to offer a build choice for somebody who might
prefer to have the rug pulled out from under him while
reviewing window contents should some target task end.

If 'TAGGED_BRIEF' is defined old behavior is restored.

[ it should be noted, however, that when this define ]
[ is active we'll lose any benefit of onetime update ]
[ for the bottom window. just like with the original ]
[ behavior, it is repainted with every update cycle. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agodoc: adapt to the 'Ctrl' bottom window one changed key
Jim Warner [Mon, 2 May 2022 05:00:00 +0000 (00:00 -0500)]
doc: adapt to the 'Ctrl' bottom window one changed key

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: trade just 1 'Ctrl' bottom window key association
Jim Warner [Mon, 2 May 2022 05:00:00 +0000 (00:00 -0500)]
top: trade just 1 'Ctrl' bottom window key association

I grew tired of inadvertently causing a premature exit
from top when all I wanted was to review some process'
environment variables. That 'V' key was just too close
to the 'C' key so the '^V' attempt became one of '^C'.

Well, not any more. Should a user wish to see a task's
environment, it will now require the '^N' combination.

[ it also now places G/U & N/K on a similar diagonal ]

Along the way, lower case Ctrl key combinations on the
help screen were all changed into upper case versions.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agodoc: adapt to the 'Ctrl' bottom window sticky behavior
Jim Warner [Mon, 2 May 2022 05:00:00 +0000 (00:00 -0500)]
doc: adapt to the 'Ctrl' bottom window sticky behavior

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: make the new 'Ctrl' bottom window behavior sticky
Jim Warner [Mon, 2 May 2022 05:00:00 +0000 (00:00 -0500)]
top: make the new 'Ctrl' bottom window behavior sticky

After using this new feature for awhile, I encountered
numerous times when that 'Ctrl' window would disappear
as I was studying the contents. This was caused by the
death of the target task but it made me think that the
window could/should persist until dismissed by a user.

Well, now it will persist until that same Ctrl key has
been struck a 2nd time or a different Ctrl key used to
target another process after positioning it on row #1.

So now, instead of that former 'moving target' type of
approach, this window is more of a 'snapshot' in time.

[ even more significant, perhaps, is the fact that a ]
[ Ctrl window will now be drawn exactly 1 time only, ]
[ instead of being repainted after each delay cycle. ]

[ this will remain true no matter how many intervals ]
[ might elapse while its contents were under review. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: eliminate insidious 'LIBPROC_HIDE_KERNEL' bug
Jim Warner [Sun, 1 May 2022 05:00:00 +0000 (00:00 -0500)]
library: eliminate insidious 'LIBPROC_HIDE_KERNEL' bug

When support for the 'LIBPROC_HIDE_KERNEL' environment
variable was introduced, a deficiency was present that
allowed any rejected proc_t (i.e. a kworker thread) to
preserve the strdup'd 'cmd' value. That residual value
would prevent stat2proc or status2proc from updating a
cmd field with the proper program name for some tasks.

This patch just ensures a proc_t is freshened whenever
it has been rejected due to an active PT->hide_kernel.

[ again thanks to Björn for initiating the extension ]

Reference(s):
. original hide_kernel implementation
commit 2a7ec67ac806d91520f59c384684e2332774368b
. original hide_kernel proposal
https://gitlab.com/procps-ng/procps/-/merge_requests/147

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: add a program name to 'Ctrl' bottom window header
Jim Warner [Sun, 1 May 2022 05:00:00 +0000 (00:00 -0500)]
top: add a program name to 'Ctrl' bottom window header

This minor addition to that new 'Ctrl' feature is more
important than it seems. It's responsible for alerting
yours truly to the 'LIBPROC_HIDE_KERNEL' bug addressed
in the very next commit. All this while, program names
for some tasks would be mysteriously replaced with the
kernel 'kworker' names. However, customary clutter for
the top process display tended to obscure such a fact.

[ and, of course, to notice the problem in the first ]
[ place required one to have the command line toggle ]
[ 'off' or perhaps to toggle its state 'on' & 'off'. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agodoc: add variable width data display without scrolling
Jim Warner [Sat, 30 Apr 2022 05:00:00 +0000 (00:00 -0500)]
doc: add variable width data display without scrolling

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: add variable width data display without scrolling
Jim Warner [Sat, 30 Apr 2022 05:00:00 +0000 (00:00 -0500)]
top: add variable width data display without scrolling

There are times when one might want to see some task's
particular variable width data. However, prior to this
commit, the only way was to first turn on a field then
scroll through it via repeated right arrow keystrokes.

[ this also required that field to be displayed last ]

Needless to say, given the potential length of some of
that variable data this could be extremely cumbersome.

Now with this patch, a Ctrl keystroke combination will
create a separate window at the bottom of the terminal
screen where such variable width data is seen in full.

[ the targeted task is the 1st task displayed, which ]
[ is a convention employed in some existing commands ]

[ the targeted data was determined by these Ctrl key ]
[ combinations: CtrlG = ctrl group; CtrlK = cmdline; ]
[ CtrlU = supplementary groups; plus CtrlV = environ ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: eliminate the 'keys_global' redundant definitions
Jim Warner [Fri, 29 Apr 2022 05:00:00 +0000 (00:00 -0500)]
top: eliminate the 'keys_global' redundant definitions

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotestsuite: Add check for shared memory
Craig Small [Tue, 26 Apr 2022 12:04:05 +0000 (22:04 +1000)]
testsuite: Add check for shared memory

Created a test process test_shm that allocates a shared memory
segment and prints the segment ID. pmap testsuite runs pmap to
check that the segment is found.

The value returned by shmget() is the same value that is printed
in the fifth column /proc/<PID>/maps

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agopmap: Don't reopen stdin for reading file
Craig Small [Tue, 26 Apr 2022 11:18:03 +0000 (21:18 +1000)]
pmap: Don't reopen stdin for reading file

pmap uses freopen to read /proc/self/maps. There doesn't
seem to be a good reason to do this and if pmap has its
stdin previously closed then it fails.

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agoMerge branch 'time-river/procps-newlib' into newlib
Craig Small [Tue, 26 Apr 2022 10:25:00 +0000 (20:25 +1000)]
Merge branch 'time-river/procps-newlib' into newlib

References:
 procps-ng/procps!155
 procps-ng/procps#236

2 years agosignal: fix suspending ps when receving SIGTERM or SIGHUP
liutie [Tue, 26 Apr 2022 04:11:07 +0000 (12:11 +0800)]
signal: fix suspending ps when receving SIGTERM or SIGHUP

Call trace:
  #0 __lll_lock_wait_private () at
     ../nptl/sysdeps/unix/sysv/linux/x86_64/lowlevellock.S:95
  #1 0x00007f95c059f9d7 in _L_lock_638 () from /lib64/libc.so.6
  #2 0x00007f95c059f8b6 in _nl_expand_alias
  #3 0x00007f95c059dad8 in _nl_find_domain
  #4 0x00007f95c059d22e in __dcigettext
  #5 0x00007f95c059c05f in __GI___dcgettext
  #6  0x00000000004032b3 in signal_handler (signo=15) at display.c:54
  #7  <signal handler called>
  #8  __memcpy_sse2 () at ../sysdeps/x86_64/memcpy.S:104
  #9  0x00007f95c05d9934 in __GI__IO_getline_info
  #10 0x00007f95c05d99b8 in __GI__IO_getline
  #11 0x00007f95c05e2a5d in __GI_fgets_unlocked
  #12 0x00007f95c059f478 in read_alias_file
  #13 0x00007f95c059f97a in _nl_expand_alias
  #14 0x00007f95c059dad8 in _nl_find_domain
  #15 0x00007f95c059d22e in __dcigettext
  #16 0x00007f95c059c05f in __GI___dcgettext
  #17 0x0000000000403a8d in reset_global () at global.c:410
  #18 0x0000000000402605 in main at display.c:650

The above call trace happens when the ps process is suspending, and the
signal SIGTERM is sent to the ps process at the same time.
Just cancel the SIGTERM and SIGHUP handler when suspending to prevent
the problem.

Signed-off-by: liutie <liutie4@huawei.com>
Signed-off-by: fu.lin <fulin10@huawei.com>
2 years agodoc: added provision for autogroup nice (AGNI) changes
Jim Warner [Sun, 24 Apr 2022 05:00:00 +0000 (00:00 -0500)]
doc: added provision for autogroup nice (AGNI) changes

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: added provision for autogroup nice (AGNI) changes
Jim Warner [Sun, 24 Apr 2022 05:00:00 +0000 (00:00 -0500)]
top: added provision for autogroup nice (AGNI) changes

When autogroups are active programs such as renice and
nice are basically useless because the nice value will
only affect the target task priority relative to other
processes in the same autogroup. So to accomplish what
we thought of as renice, /proc/<pid>/autogroup must be
changed. Altering a single member in an autogroup will
also affect every other member of that same autogroup.

Since top's renice provision ('r') suffers constraints
like those of the stand alone nice/renice programs, we
will now provide a means to manipulate that nice value
found within some process' /proc/<pid>/autogroup file.

[ to alter this file for other user's tasks requires ]
[ root privileges, as does setting a negative value. ]

[ however, unlike that 'r' command, this new command ]
[ allows raising *and* lowering all positive values. ]

Reference(s):
. Aug 2021, autogroups added to librady
commit 631e5d91f3c34374a095b8351235627545617de7
. Aug 2021, autogroups added to top
commit b789b46f84da590a39afd8d74c76c2f5e5436d40

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: the '%CUC' column must be just a little bit wider
Jim Warner [Wed, 20 Apr 2022 05:00:00 +0000 (00:00 -0500)]
top: the '%CUC' column must be just a little bit wider

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: that '%CUC' column must be just a little bit wider
Jim Warner [Wed, 20 Apr 2022 05:00:00 +0000 (00:00 -0500)]
ps: that '%CUC' column must be just a little bit wider

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: fix some 'origin' header comments, <pids> api
Jim Warner [Wed, 20 Apr 2022 05:00:00 +0000 (00:00 -0500)]
library: fix some 'origin' header comments, <pids> api

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: annotate header with tics/jiffies, <pids> api
Jim Warner [Fri, 15 Apr 2022 05:00:00 +0000 (00:00 -0500)]
library: annotate header with tics/jiffies, <pids> api

To avoid potential user confusion, like that reflected
in the previous commit message, a short narrative will
be included in the header file as programmer comments,

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: improved three elapsed 'jiffies/tics' calculations
Jim Warner [Fri, 15 Apr 2022 05:00:00 +0000 (00:00 -0500)]
ps: improved three elapsed 'jiffies/tics' calculations

With openSUSE's tumbleweed release of procps-ng-4.0.0,
their source rpm contained a questionable patch. It is
ostensibly devoted to increasing the accuracy for %cpu
calculations in the ps program. However, in my opinion
it goes too far and is quite flawed for these reasons:

1. Six separate files were impacted instead of just 1.
2. While ps was the object, libproc-2 was changed too.
3. A header file's alphabetic item order was violated.
4. The library API and ABI were altered unnecessarily.

It should be noted that all the <pids> TIME items were
made 'real' when representing seconds so that they can
be converted into jiffies/tics if multiplied by hertz.
Thus, there was absolutely no justification for adding
a new PIDS_TICS_ELAPSED ull_int item to the interface.

As an example, assuming a hertz value of 100, there is
enough capacity in all of those 'TIME' seconds to hold
nearly 3 million years worth of tics without a loss of
precision whenever they are changed back into jiffies.

[ that's nine quadrillion seven trillion one hundred ]
[ ninety-nine billion two hundred fifty-four million ]
[ seven hundred forty thousand nine hundred ninety + ]
[ one! or more concisely, as: 9,007,199,254,740,991. ]

So, any need for increased accuracy in that ps program
can be realized within the single output.c file alone.

That's what will be accomplished with this new commit.

And for any doubters here's a 'double' capacity proof:

beg ------ 9007199254740991.000000000000000000000 tic|
 d /= 100, 90071992547409.90625000000000000000000 sec
 d /=  60, 1501199875790.165039062500000000000000 min
 d /=  60, 25019997929.83608245849609375000000000 hrs
 d /=  24, 1042499913.743170142173767089843750000 day
 d /=   7, 148928559.1061671674251556396484375000 wks
 d /=  52, 2864010.752041676081717014312744140625 yrs
 d *=  52, 148928559.1061671674251556396484375000 wks
 d *=   7, 1042499913.743170142173767089843750000 day
 d *=  24, 25019997929.83608245849609375000000000 hrs
 d *=  60, 1501199875790.165039062500000000000000 min
 d *=  60, 90071992547409.90625000000000000000000 sec
 d *= 100, 9007199254740991.000000000000000000000 tic
end ------ 9007199254740991.000000000000000000000  " |

[ who knows, maybe we'll even convince openSUSE that ]
[ the original sledgehammer 'dif' should be dropped. ]

Reference(s):
. openSUSE equivalent: 'procps-ng-4.0.0-accuracy.dif'

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: tighten up some TIME calculations, <pids> api
Jim Warner [Fri, 15 Apr 2022 05:00:00 +0000 (00:00 -0500)]
library: tighten up some TIME calculations, <pids> api

This patch trades some recurring per-task calculations
for calculations performed once at get, select or reap
time. It was prompted by the openSUSE dif named below.

[ my next commit will deal more thoroughly with that ]

Reference(s)
. openSUSE patch named: 'procps-ng-4.0.0-accuracy.dif'

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: eliminate redundant assignment, <meminfo> api
Jim Warner [Thu, 14 Apr 2022 05:00:00 +0000 (00:00 -0500)]
library: eliminate redundant assignment, <meminfo> api

With the memset to zero of a 'new' meminfo_data struct
near the beginning of the meminfo_read_failed function
there's never a need to later set anything in it to 0.

[ who knows, our patch might even coax opensuse into ]
[ reevaluating that 4.0.0 'overflow' guy named below ]

Reference(s):
. overflow: 'procps-ng-4.0.0-integer-overflow.patch'

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: that restored aix behavior gets its ultimate tweak
Jim Warner [Wed, 13 Apr 2022 05:00:00 +0000 (00:00 -0500)]
ps: that restored aix behavior gets its ultimate tweak

Now that this 'c' variable initial assignment has been
moved outside the looping code, there is no longer any
need to specifically check for space/comma when 'c' is
not '%'. So, let us eliminate those two lines of code.

[ i promise not to change this algorithm ever again! ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: remove those winflags which restricted 'x' toggle
Jim Warner [Tue, 12 Apr 2022 05:00:00 +0000 (00:00 -0500)]
top: remove those winflags which restricted 'x' toggle

On occasion, even as the top author, I wonder why that
'x' toggle has stopped working. Of course, it actually
was working but a locate request ('L') or other filter
('O') operation was active and thus temporarily turned
if off. Such behavior is documented in top's man page.

Well, with this patch that 'x' suppression is no more.

[ the original justification, however, remains true. ]

[ but there's really only one character which causes ]
[ any potential trouble & i'm gonna' keep it secret. ]

[ besides, if a display is corrupted, there's always ]
[ that '=' key which restores things back to normal. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agop/version.c: osrelease for Cygwin
Craig Small [Mon, 11 Apr 2022 07:30:25 +0000 (17:30 +1000)]
p/version.c: osrelease for Cygwin

On Cygwin systems use a different file and format to get the
OS version.

Thanks to procps Cygwin maintainer Achim!

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agobuild-sys:
Craig Small [Mon, 11 Apr 2022 07:09:53 +0000 (17:09 +1000)]
build-sys:

While the previous commit checked for a GNU environment so Hurd
compilied ok, this tripped up Cygwin. configure now explicitly tests
for the structure field rather than trying to guess through compilier
flags about what the environment tells us about signals.h

References:
 commit d39d9db07974f4f09cd2f31137742a7ea3be65c5

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agolibrary: fix 'smaps' bug preventing a build with clang
Jim Warner [Sun, 10 Apr 2022 05:00:00 +0000 (00:00 -0500)]
library: fix 'smaps' bug preventing a build with clang

As a result of the issue referenced below, we'll trade
our homegrown offset generator for an 'offsetof' macro
found in the stddef.h header file. This pleases clang.

[ and thanks to Daniel Kolesa for the report and fix ]

Reference(s):
. bug report & recommended solution
https://gitlab.com/procps-ng/procps/-/issues/235

. clang error message
proc/readproc.c:673:9: error: initializer element is not a compile-time constant
        mkENT(Rss),
        ^~~~~~~~~~
proc/readproc.c:661:34: note: expanded from macro 'mkENT'
  #define mkENT(F) { #F ":", -1, (int)((void*)&q->smap_ ## F - (void*)&q->fZERO) }
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: raise %CPU maximum if large numbers of processors
Jim Warner [Sat, 9 Apr 2022 05:00:00 +0000 (00:00 -0500)]
top: raise %CPU maximum if large numbers of processors

It would appear that openSUSE was the first to release
procps-ng version 4.0.0 (in the tumbleweed distro). As
a result I checked their source rpm and found a couple
of patches I'm porting to newlib for the next release.

This particulate patch, with some changes, is openSUSE
patch 'procps-ng-3.3.10-large_pcpu.patch'. The changes
were strictly cosmetic. They involved whitespace only.

[ it should be noted that the net effect for raising ]
[ the maximum is to produce a '?' in the %CPU field. ]
[ that's because its width is fixed at five columns. ]
[ but at least the '?' won't distort actual results. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: ignore the SIGHUP signal if running in batch mode
Jim Warner [Sat, 9 Apr 2022 05:00:00 +0000 (00:00 -0500)]
top: ignore the SIGHUP signal if running in batch mode

It would appear that openSUSE was the first to release
procps-ng version 4.0.0 (in the tumbleweed distro). As
a result I checked their source rpm and found a couple
of patches I'm porting to newlib for the next release.

This particulate commit was a refactor of the openSUSE
patch 'procps-ng-3.3.8-bnc634840.patch'. Unfortunately
their original patch did not have the intended effect.

That was because the amended signal handling logic was
performed well before the command line parameters were
parsed. So the global 'Batch' flag was in its 0 state.

. what follows is the original openSUSE commit message
------------------------------------------------------
Do not setup SIGHUP signal handler if we are in the batch mode

Top enables a signal handler for the SIGHUP signal (loss of terminal).  While
this makes sense for top's default interactive mode, it doesn't make any sense
for batch mode. If you run top in nohup just to collect data over time and
disconnect top finishes which is not what one would expect.
------------------------------------------------------

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotestsuite: Add AIX field checks for ps
Craig Small [Sat, 9 Apr 2022 04:50:53 +0000 (14:50 +1000)]
testsuite: Add AIX field checks for ps

AIX fields (generally %char) have had some love now but
are parsed slightly differently to other fields, so they
have some tests.

References:
 commit 4fbf8d22a99445bc7b37c6d36ed9e045821439e3
 https://www.freelists.org/post/procps/some-procpsn4400-fixes,7
Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agosysctl: print dotted keys again
Craig Small [Sat, 9 Apr 2022 04:18:28 +0000 (14:18 +1000)]
sysctl: print dotted keys again

When the globbing update was put into sysctl, you could no longer
simply use the keys because one key could potentially be
multiple paths once the glob expansion occured.  Using the path
instead gave a unique output.

Except certain programs, such as salt, expected the output to use
the dotted path "kernel.hostname" and not "kernel/hostname".

We can no longer use the original key, so now for each path:
  Copy the path
  strip off /proc/
  convert all / to .

The sysctl testsuite was also updated to check for a few different
types of conversion failures.

References:
 commit 6389deca5bf667f5fab5912acde78ba8e0febbc7
 https://www.freelists.org/post/procps/some-procpsn4400-fixes,4
 https://repo.saltproject.io/

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agops: that restored aix behavior is now even more robust
Jim Warner [Mon, 4 Apr 2022 05:00:00 +0000 (00:00 -0500)]
ps: that restored aix behavior is now even more robust

With the commit referenced below a nasty bug affecting
aix parsing was swatted. Beyond the bug, the logic was
enhanced to disallow commas in the format string since
they would otherwise be shown with their field's data.

However, there remained many characters other than ','
that could survive the edits to then be shown adjacent
to the data. Well, with this patch they won't anymore!

[ along the way we will no longer try to forgive the ]
[ use of a double '%%' prefix since that resulted in ]
[ display of one '%' and field code instead of data. ]

Reference(s):
. March, 2022 - restored aix bahavior
commit 8cb646bdfc6e37b221f9616df96c6c116e6def28

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agoNEWS: update with 2 ps related release 4.0.0 bug fixes
Jim Warner [Fri, 1 Apr 2022 05:00:00 +0000 (00:00 -0500)]
NEWS: update with 2 ps related release 4.0.0 bug fixes

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agoNEWS: Mention elogind merge
Craig Small [Fri, 1 Apr 2022 06:28:53 +0000 (17:28 +1100)]
NEWS: Mention elogind merge

Reference:
 procps-ng/procps!151

2 years agoAdd support for elogind
Sven Eden [Tue, 30 May 2017 14:33:28 +0000 (16:33 +0200)]
Add support for elogind

A session manager similar to logind from systemd.
See https://github.com/elogind/elogind

Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>
2 years agops: restore thread display when using a pidlist option
Jim Warner [Wed, 30 Mar 2022 17:00:00 +0000 (12:00 -0500)]
ps: restore thread display when using a pidlist option

This commit will once again display threads when using
the -L option along with any of those pidlist options.

Reference(s):
https://gitlab.com/procps-ng/procps/-/issues/234

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: restore aix behavior while keeping an original fix
Jim Warner [Wed, 30 Mar 2022 17:00:00 +0000 (12:00 -0500)]
ps: restore aix behavior while keeping an original fix

The commit shown below broke the aix behavior that Dr.
Fink recently reported. However, in the proposed patch
the old behavior, showing garbage when '%cpu' was used
with an invalid formatting option, would appear again.

So this patch, based on Werner's patch, goes the extra
distance to prevent that. Along the way we'll disallow
commas in the aix format str to prevent their display.

Reference(s):
https://www.freelists.org/post/procps/Procpsng-400-released-with-newlib,2
. Mar, 2022 - where aix bug was introduced
commit 81df85a1b528d4edb9ab98b37fb6c6244430b6c4

Prototyped-by: Dr. Werner Fink <werner@suse.de>
Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: update one function prologue after rcfile changes
Jim Warner [Tue, 29 Mar 2022 05:00:00 +0000 (00:00 -0500)]
top: update one function prologue after rcfile changes

When top's rcfile was reorganized, trading that 'char'
based field representation for the 'int' scheme, there
was a function prologue that should have been changed.

Thus, this patch will correct that original oversight.

Reference(s):
. Mar, 2022 - rcfile reorganized
commit 46aa96e438435628810b33d20cb1c14dc33dd84f

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agonls: man-po fixes
Craig Small [Wed, 23 Mar 2022 06:56:20 +0000 (17:56 +1100)]
nls: man-po fixes

Add update-po target, to update the manpage po files run
make -C man-po update-po

Updated the po files.

References:
 procps-ng/procps#230

2 years agomisc: Add v4.0.0 to NEWS
Craig Small [Wed, 23 Mar 2022 06:56:11 +0000 (17:56 +1100)]
misc: Add v4.0.0 to NEWS

2 years agonls: Update translations v4.0.0
Craig Small [Tue, 22 Mar 2022 10:40:10 +0000 (21:40 +1100)]
nls: Update translations

2 years agotestsuite: ps out tests more lenient
Craig Small [Tue, 22 Mar 2022 09:20:53 +0000 (20:20 +1100)]
testsuite: ps out tests more lenient

Sometimes due to race conditions or the way dejagnu gates the
output, or even there is another interesting process, the ps tests
sometimes fail. These changes make it a little more lenient without
losing the purpose of the test.

Replaces Debian patch ps_tests

References:
 https://salsa.debian.org/debian/procps/-/blob/debian/2%253.3.17-7/debian/patches/ps_checks

2 years agotestsuite: Return error if tty is TTY
Craig Small [Tue, 22 Mar 2022 08:53:28 +0000 (19:53 +1100)]
testsuite: Return error if tty is TTY

In some build systems, such as the Debian pbuilders, the
environment is strange. The tty is called "TTY" which causes
some of the ps tests to fail.

This commit checks for that specific result and returns ""
so the tests can be bypassed.

Replaces Debian patch fix_checks.

References:
 https://salsa.debian.org/debian/procps/-/blob/debian/2%253.3.17-7/debian/patches/fix_checks

2 years agolibrary: fixed some header inconsistencies, <pids> api
Jim Warner [Tue, 22 Mar 2022 05:00:00 +0000 (00:00 -0500)]
library: fixed some header inconsistencies, <pids> api

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: why not maximize usage of some manifest constants
Jim Warner [Mon, 21 Mar 2022 05:00:00 +0000 (00:00 -0500)]
top: why not maximize usage of some manifest constants

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agonls: Remove incorrect Romanian file
Craig Small [Mon, 21 Mar 2022 11:01:53 +0000 (22:01 +1100)]
nls: Remove incorrect Romanian file

This is for psmisc

2 years agonls: Add Romanian (ro) translation
Craig Small [Mon, 21 Mar 2022 11:01:29 +0000 (22:01 +1100)]
nls: Add Romanian (ro) translation

2 years agonls: Update translation files
Craig Small [Mon, 21 Mar 2022 10:55:01 +0000 (21:55 +1100)]
nls: Update translation files

2 years agobuild-sys: check depends on programs
Craig Small [Mon, 21 Mar 2022 10:37:17 +0000 (21:37 +1100)]
build-sys: check depends on programs

For some unknown reason, make check doesn't depend on either
the check programs or the actual programs under test. This means
if you try to run make check right off the bat from a clean git
repositorty, it will fail as it won't be able to run the unbuilt
binaries.

2 years agobuild-sys: Repeat man pages into man-po Makefile
Craig Small [Mon, 21 Mar 2022 10:33:11 +0000 (21:33 +1100)]
build-sys: Repeat man pages into man-po Makefile

I'm not sure why this is needed, but if the dist_man_MANS
variable is not copied from the top-level Makefile down to the
man-po one then make dist from a clean archive will not work.

I'm sure someone with better automake magic than me can make
this more elegant but for now it works if a little ugly.

References:
  https://www.freelists.org/post/procps/important-newlib-patches,8

2 years agotop: fix initial cursor position for fields management
Jim Warner [Sat, 19 Mar 2022 05:00:00 +0000 (00:00 -0500)]
top: fix initial cursor position for fields management

When we transitioned from 'char' based fields to 'int'
proper initial cursor positioning (highlight) was lost
in the 'fields_utility' function. The highlight should
appear on the current sort field. Instead, however, it
was often positioned on the first field listed or some
completely different field other than that sort field.

So this patch will restore the master branch behavior.

[ and on the assumption that a sort field is visible ]
[ we will search first for a field with the 'on' bit ]

Reference(s):
. Mar, 2022 - trade 'char' for 'int'
commit 46aa96e438435628810b33d20cb1c14dc33dd84f

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: add more robustness to logic for reading a rcfile
Jim Warner [Fri, 18 Mar 2022 05:00:00 +0000 (00:00 -0500)]
top: add more robustness to logic for reading a rcfile

At this point I don't know if the Rc.id ever carried a
value of 'b', 'c', 'd' or 'e' in a public release. But
I do know that those values ought not to be recognized
in any configuration file. Now they won't be for sure.

[ the net result is that now a user with such a file ]
[ will see the 'incompatible rcfile' message instead ]
[ of that misleading 'window entry corrupt' message. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: do not add an extra line when rewritting a rcfile
Jim Warner [Thu, 17 Mar 2022 05:00:00 +0000 (00:00 -0500)]
top: do not add an extra line when rewritting a rcfile

When the configuration file is first written, there is
just the normal single newline which ends that rcfile.

However, when any existing rcfile was rewritten, there
was one extra newline '\n' character added at the end.
This will happen just once. The file does NOT continue
to grow with the extra blank lines always being added.

Anyway, this patch will put an end to such a practice.

[ along the way i fixed an unrelated whitespace oops ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: man page now has 'UTILIZATION_C' as newest column
Jim Warner [Wed, 16 Mar 2022 05:00:00 +0000 (00:00 -0500)]
top: man page now has 'UTILIZATION_C' as newest column

When %CUC was added in the commit shown below I failed
to update the man document. This fixes that oversight.

Reference(s):
commit cfa553883259a3741abd37287b6f6fc737f2a505

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: swat a bug with 3.2.8 support for field additions
Jim Warner [Mon, 14 Mar 2022 05:00:00 +0000 (00:00 -0500)]
top: swat a bug with 3.2.8 support for field additions

This new integer based version of 'fieldscur' deviated
a bit too much from that character based logic when it
comes to ancient 3.2.8 rcfile support. This results in
an inappropriate extra field shown as the last column.

If we're truly dealing with 20 year old 3.2.8 rcfiles,
it's important the 3.3.9 extension (RCF_PLUS_H) not be
concatenated to what was read. That's because a search
for the special SUSE characters will always find a '\'
in the 26th position (after normal 3.2.8 'fieldscur').

[ for symmetry, we'll also avoid RCF_PLUS_J (3.3.17) ]

All other 'old' rcfiles are transformed without error.

Reference(s):
. Mar, 2022 - 3.2.8 support restores

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: honor old rcfiles after change to field additions
Jim Warner [Sun, 13 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: honor old rcfiles after change to field additions

This patch will restore the previously abandoned older
rcfile support. To be honest, I wasn't sure this would
be possible given the vast difference between formats.

But after some very intense hacking our newlib top can
again fully support seamless migration from some older
'char' based file to this new 'int' based file format.

[ top even supports a 20+ year old 3.2.8 config file ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: bump current upper max to 100 for field additions
Jim Warner [Sun, 13 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: bump current upper max to 100 for field additions

As I speculated in a prior commit, this patch will now
demonstrate how simple it is to extend the upper limit
of supportable fields. With these changes we'll now go
from facing the old limit of 86 to a new limit of 100.

No other changes are needed nor is any logic impacted.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agodoc: tweak top man page for changes to field additions
Jim Warner [Sun, 13 Mar 2022 06:00:00 +0000 (00:00 -0600)]
doc: tweak top man page for changes to field additions

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: provide an unlimited capacity for field additions
Jim Warner [Sun, 13 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: provide an unlimited capacity for field additions

With each of those references shown below, the maximum
number of fields was increased. However, with a 'char'
based implementation we're nearing the upper limits of
total displayable fields. We currently use 76 of a max
of 86 fields. With extra effort, 94 might be possible.

But 94 is the absolute upper limit of possible fields!

Moreover, the current implementation yields characters
that were unprintable in the rcfile. This could become
an issue with that 'inspect' feature when/if an rcfile
is edited to add entries (as opposed to using 'echo').

So, with this commit the internals of field management
has been completely reimagined. It is now based on the
integer type, not a character. And whereas that former
design used the high order bit to show the 'on' state,
thus yielding an unprintable character, the new design
uses the low order bit for the state. As such, numbers
will be kept small and an even number will be an 'off'
field whereas an odd number will become an 'on' state.

The bottom line is that this new design will afford an
unlimited number of new fields while keeping an rcfile
completely free of that potential unprintable garbage.

And it is embarrassingly easy to extend the maximum of
supportable fields from the currently implemented 100.
Who knows, maybe a future patch will prove this point.

[ unless a subsequent commit proves otherwise, given ]
[ the dramatic differences in rcfile contents, i had ]
[ to abandon the practice of supporting old rcfiles. ]

Reference(s):
. Nov, 2013 - RCF_PLUS_H introduced
commit af4e6533ba5ee04f414bc8774df31ddef38e8880
. Jul, 2016 - RCF_PLUS_J introduced
commit d5c5051fb395e8bebaadb42add79ca99a16afe85

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: warn of man page impact if 'USE_X_COLHDR' defined
Jim Warner [Sat, 12 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: warn of man page impact if 'USE_X_COLHDR' defined

This warning was added after discovering openSUSE uses
the option with (at least) their 'tumbleweed' version.

The man document sections impacted are: 4e, 5d and 5e.
That is where it was asserted sort column highlighting
would be temporarily turned off if search ('L') and/or
filtering ('O') is active. This option makes it false.

[ shame on suse for not also correcting the man page ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: plus for symmetry 'UTILIZATION_C' is newest format
Jim Warner [Sun, 6 Mar 2022 06:00:00 +0000 (00:00 -0600)]
ps: plus for symmetry 'UTILIZATION_C' is newest format

This commit just compliments a change referenced below
by providing the value which includes reaped children.

[ as an aside, it looks like ps only includes reaped ]
[ children values under an obscure BSD or GNU option ]

[ so, this change is possibly of even more relevance ]

Reference(s):
. Mar, 2022 - added %CUU
commit 2ac72e2e801e015532b555ae0f2272c8ecef2c8b

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: and for symmetry 'UTILIZATION_C' is newest column
Jim Warner [Sun, 6 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: and for symmetry 'UTILIZATION_C' is newest column

This commit just compliments a change referenced below
by providing the value which includes reaped children.

Reference(s):
. Feb, 2022 - added %CUU field
commit 7647e96b0a35d473fa9bc644ea6107487b3b0527

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: introduced a 'UTILIZATION_C' item, <pids> api
Jim Warner [Sun, 6 Mar 2022 06:00:00 +0000 (00:00 -0600)]
library: introduced a 'UTILIZATION_C' item, <pids> api

Some items in the <pids> API carry a '_C' suffix which
implies such values include reaped children. So, we'll
now extend this practice to that new UTILIZATION item.

Reference(s):
. Mar, 2022 - tweak PIDS_UTILIZATION
commit 9c0e8e9429d232a85baef3d6fa8269963fd80741
. Feb, 2022 - added PIDS_UTILIZATION
commit c69104b2b835176f9801dfd1a1d893579879a5b6

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: enabled sort with 'utilization' related specifiers
Jim Warner [Sun, 6 Mar 2022 06:00:00 +0000 (00:00 -0600)]
ps: enabled sort with 'utilization' related specifiers

When %CUU was added, in the commit referenced below, I
stated that 4 similar specifiers already existed. Well
I misspoke since there is actually 5 including 'util'.

Each of those fields had no sort capability. Since the
values are dynamically calculated, they were forced to
use 'PIDS_extra' as the format_array 'sr' designation.

Now each will use 'PIDS_UTILIZATION' and be sort-able.

[ yes, sometimes the calculated values could contain ]
[ reaped children while the sort field does not. but ]
[ such disparity depends on obscure bsd/gnu options. ]

Reference(s):
. Mar, 2022 - added %CUU
commit 2ac72e2e801e015532b555ae0f2272c8ecef2c8b

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: enhanced 'scale_tics' function for Ctrl-E scaling
Jim Warner [Sun, 6 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: enhanced 'scale_tics' function for Ctrl-E scaling

When scale_tics was refactored and then Ctrl-E support
added to top, the complete range of scaling values was
not visible. Namely, a single 'd' (days) & 'w' (weeks)
was never seen with ^E. With this commit they will be.

Reference(s):
. Mar, 2022 - introduced ^E tics scaling
commit 402bf1903b25ac78b0c4cbf63d457feee78a391c
. Mar, 2022 - refactored scale_tics
commit 71eb90c1b27ef039ccc90f6060406099df45eade

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: swat insidious bug with the %cpu' format specifier
Jim Warner [Fri, 4 Mar 2022 06:00:00 +0000 (00:00 -0600)]
ps: swat insidious bug with the %cpu' format specifier

Whoa, my head really hurts but this commit should help
with a speedy recovery hopefully, after it is applied.

If the '%cpu' field is used as a format specifier with
that 'o' option, you will encounter a SIGSEGV if there
is also an invalid argument on that same command line.

For example, try 'ps/pscommand -o %cpu,x' with newlib.
With any format specifier other than the '%cpu', there
is an error message, as would happen with '-o pcpu,x'.

For a 3.3.17 version of ps, there's no abend. Instead,
the program will just display a bunch of gobbledygook.
This boo-boo was found to exist as far back as v3.3.0.

[ ok, i am starting to feel very much better already ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: exploit the library addition for 'cpu utilization'
Jim Warner [Fri, 4 Mar 2022 06:00:00 +0000 (00:00 -0600)]
ps: exploit the library addition for 'cpu utilization'

This just parallels the top program by adding that new
library PIDS_UTILIZATION item to the ps repertoire. It
should be noted, however, that the new %CUU field is a
little redundant. I mean, ps already has 4 such fields
implemented identified as: '%cpu', 'c', 'cp' & 'pcpu'.

Oh well, at least the newest one offers a little value
added in the form of extra precision. We'll follow the
top lead and display results in the form of: '##.###'.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: guard against 'inf/nan' for 'cpu utilization'
Jim Warner [Thu, 3 Mar 2022 06:00:00 +0000 (00:00 -0600)]
library: guard against 'inf/nan' for 'cpu utilization'

When preparing the ps program for a %CUU field, it was
revealed the PIDS_UTILIZATION item may sometimes yield
a result of 'nan' or 'inf' for that ps program itself.

Apparently, it was caused by the short lived nature of
such a one-shot program. And, while this anomaly could
be handled in ps, the solution belongs in the library.

So, I reworked an item's algorithm to avoid this oops.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agodoc: document 'scale_tics' function for Ctrl-E scaling
Jim Warner [Thu, 3 Mar 2022 06:00:00 +0000 (00:00 -0600)]
doc: document 'scale_tics' function for Ctrl-E scaling

[ i also tweaked that 'STARTED' narrative just a bit ]
[ since its original wording implied the value could ]
[ change, whereas it's fixed when a task is started. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: extended 'scale_tics' function for Ctrl-E scaling
Jim Warner [Thu, 3 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: extended 'scale_tics' function for Ctrl-E scaling

That normalization of the 'scale_tics' function in the
prior commit convinced me that I won't please everyone
with my arbitrary choices for the scaling transitions.

So, this patch will provide the users with a means for
setting their own scaling transition points with a new
toggle. Ctrl-E was chosen since the 'e/E' toggles were
already present as a means of scaling (albeit memory).

[ this toggle will also serve an educational purpose ]
[ by allowing one to see all the scaling conventions ]

The scaling a user establishes is saved in the rcfile.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: refactor 'scale_tics' function for better scaling
Jim Warner [Thu, 3 Mar 2022 06:00:00 +0000 (00:00 -0600)]
top: refactor 'scale_tics' function for better scaling

This patch refactors the 'scale_tics' function to more
closely parallel uptime shown on the first line of the
summary area. The old logic has been preserved through
the header file's new  #define SCALE_FORMER provision.

However, the former logic was actually a big disaster.
These are some potential problems with that old logic:

1. With respect to our time fields top no longer deals
solely with cpu time. So, the old limits of '68 weeks'
could possibly be insufficient to reflect those times.

2. Given the widths of top's new time fields, the code
never got beyond scaling to hours. For example, with a
ridiculously large span of 19 years, the scaled result
would then be shown as '167832h'. We never reached the
days ('6993d') or even the weeks ('999w') equivalents.

3. Similarly, with that 'TIME+' field and a large tics
value, results would then appear as 'MMMMMM:SS' rather
than the more meaningful 'HH:MM:SS' or days and hours.

So henceforth we will adopt these scaling conventions:

  MMM:SS.hh ... minutes:seconds.hundredths
  MMM:SS ...... minutes:seconds
  HH,MM ....... hours,minutes
  D+H ......... days+hours (with 'd' & 'h' suffixes)
  D ........... days (with 'd' suffix)
  W+D ......... weeks+days (with 'w' & 'd' suffixes)
  W ........... weeks (with 'w' suffix)

Note that, unlike our former scaling logic, that 'MMM'
portion won't be allowed to grow unconditionally. It's
limited (arbitrarily?) to 360 total minutes (6 hours).
Additionally, the 'HH' guy will be limited to 96 hours
(4 days) while that 'D' limit was set at 14 (2 weeks).

Whenever a limit is hit, scaling will advance a level.

Reference(s):
. Feb, 2022 - added 'ELAPSED'
commit 9348d3b0085bd389749cc5619fdde4ec3fae5f11
. Feb, 2022 - added 'STARTED'
commit 7647e96b0a35d473fa9bc644ea6107487b3b0527

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: added some elapsed running time 'ELAPSED' support
Jim Warner [Sun, 27 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: added some elapsed running time 'ELAPSED' support

When the 'STARTED' field was added, in the message for
the commit referenced below, I explained why 'ELAPSED'
shouldn't be implemented though it might be preferred.

Well, after climbing out of my box to do a little more
thinking, I came up with the way to add that 'ELAPSED'
field while avoiding the possible performance penalty.

Just do not show what would change with every refresh!

If we do not show the seconds portion of a scaled tics
amount then the problem goes away. And this comes with
an additional benefit. The HH,MM (hours,minutes) style
then is readily compared with that system uptime shown
as HH:MM. The only difference is just the comma/colon.

[ assuming the top uptime/load average toggle was on ]

Reference(s):
. introduced 'start time' field
commit 7647e96b0a35d473fa9bc644ea6107487b3b0527

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: tweaked the recently introduced 'STARTED' support
Jim Warner [Sun, 27 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: tweaked the recently introduced 'STARTED' support

When the 'STARTED' field was added, the width was made
the same as the 'TIME+' field. Thus, a full time could
be shown (which then included hundredths of a second).

That kind of granularity is totally unnecessary. After
all, this column is potentially confusing enough since
it is so counterintuitive. So, this commit will reduce
the width of the field with some help from scale_tics.

Henceforth it will not include those ol' centiseconds.

[ along the way let's expand the man document with a ]
[ a remainder about content representation & scaling ]

Reference(s):
. introduced 'start time' field
commit 7647e96b0a35d473fa9bc644ea6107487b3b0527

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: trade 'float' type for a 'double', <pids> api
Jim Warner [Sun, 27 Feb 2022 06:00:00 +0000 (00:00 -0600)]
library: trade 'float' type for a 'double', <pids> api

While we really do not need the extra precision, we'll
trade that recently introduced float type for a double
for efficiency on a 64 bit platform. Additionally, the
UTILIZATION algorithm was tweaked producing less code.

The net result is four fewer machine instructions with
a reduction of 14 total bytes in that function's size.

Reference(s):
. introduced 'real' type & 'utilization' item
commit c69104b2b835176f9801dfd1a1d893579879a5b6

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agow: responded to those library changes for 'TIME' stuff
Jim Warner [Sat, 26 Feb 2022 06:00:00 +0000 (00:00 -0600)]
w: responded to those library changes for 'TIME' stuff

The library refactored 'TIME' items for consistency so
we must adapt to some new data types and calculations.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agops: response to those library changes for 'TIME' stuff
Jim Warner [Sat, 26 Feb 2022 06:00:00 +0000 (00:00 -0600)]
ps: response to those library changes for 'TIME' stuff

The library refactored 'TIME' items for consistency so
we must adapt to some new data types and calculations.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agopgrep: adapted to the library changes for 'TIME' stuff
Jim Warner [Sat, 26 Feb 2022 06:00:00 +0000 (00:00 -0600)]
pgrep: adapted to the library changes for 'TIME' stuff

The library refactored 'TIME' items for consistency so
we must adapt to some new data types and calculations.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agodoc: updated with new 'start time' & 'cpu utilization'
Jim Warner [Fri, 25 Feb 2022 06:00:00 +0000 (00:00 -0600)]
doc: updated with new 'start time' & 'cpu utilization'

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: added fields for 'start time' & 'cpu utilization'
Jim Warner [Fri, 25 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: added fields for 'start time' & 'cpu utilization'

This patch will exploit some new library capabilities.

[ one will raise eyebrows, the other likely will not ]

A new 'STARTED' field was added which shows the time a
process started after system boot. As such the largest
interval represents the most recently started process.

This is the field that will likely be questioned since
it's somewhat counterintuitive. But were we to instead
use TIME_ELAPSED, the value will change with every top
refresh. This will defeat any PUFF macro optimization.

The new '%CUU' field will probably be better received.
It represents the cpu usage over the life of the task.
When a process was showing high %CPU usage, this field
can be used to determine if it's an anomaly or normal.

[ and as with %CPU, %CUU shows a '?' when running in ]
[ a namespace when /proc was mounted with subset=pid ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agolibrary: added important new functionality, <pids> api
Jim Warner [Fri, 25 Feb 2022 06:00:00 +0000 (00:00 -0600)]
library: added important new functionality, <pids> api

This commit introduces some new capabilities available
in libproc-2 under the <PIDS> interface. Along the way
errors impacting some item values have been corrected.

The following summarizes the major changes being made.

1. The PIDS_TIME_START item was represented as seconds
since system boot but really held tics since boot. And
some programs properly divided it by Hertz to actually
yield seconds while others acted as if it already was.

So, now we have a new PIDS_TICS_BEGAN field and all of
the 'TIME' fields properly reflect seconds. With those
'TIME' fields, the type was changed to 'float/real' so
one could convert it back to tics without loss of some
centiseconds reflected in the Hertz guy (usually 100).

2. The boot_seconds was established in procps_pids_new
meaning it was fixed/unchanging. As a result, one item
(PIDS_TIME_ELAPSED) was rendered useless. So now, each
of the three retrieval functions establishes a current
boot_seconds well before the set functions are called.

3. Added a PIDS_UTILIZATION item that will provide the
CPU usage over the life of a process, as a percentage.

4. Added PIDS_TIME_ALL_C for symmetry with the similar
item called PIDS_TICS_ALL_C (which reflects raw tics).

5. That 'derived from' notation has been added to some
additional header items to reflect their true origins.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agodoc: minor tweaks affecting three newlib man documents
Jim Warner [Thu, 24 Feb 2022 06:00:00 +0000 (00:00 -0600)]
doc: minor tweaks affecting three newlib man documents

. add the proper function names to the procps_pids man
page when discussing the 'LIBPROC_HIDE_KERNEL' feature
under recently added 'ENVIRONMENT VARIABLE(S)' section

. ensure the 'SEE ALSO' references are comma delimited

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: just trade an assignement for a manifest constant
Jim Warner [Wed, 23 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: just trade an assignement for a manifest constant

The assignment was originally created to reduce length
of one line of code (i.e. for readability). But we can
achieve this objective without adding executable code.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: trade that 'strncpy' for more efficient 'memccpy'
Jim Warner [Mon, 14 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: trade that 'strncpy' for more efficient 'memccpy'

This patch was prompted by the merge request for pgrep
referenced below. In top's case, any performance gains
will be minimal since the now defunct strncpy was only
employed for termcap rebuilds after interacting with a
user (+ 1 other non-termcap related user interaction).

[ golly, strncpy always calls at least two functions ]
[ but usually calls a total of 3. on the other hand, ]
[ memccpy will only call a maximum of two functions. ]

And thanks to Baruch Siach for these strncpy insights.

Reference(s):
https://gitlab.com/procps-ng/procps/-/merge_requests/148

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: tweak a couple task_show() macros for consistency
Jim Warner [Sun, 13 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: tweak a couple task_show() macros for consistency

There was a potential problem with these macros should
that '#define SCROLLVAR_NO' be active and they were to
appear in an 'if' statement (like is necessary for the
master branch version of top under that EU_CMD label).

[ now they're always usable without requiring an ';' ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: let's eliminate some additional compiler warnings
Jim Warner [Sat, 12 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: let's eliminate some additional compiler warnings

This patch eliminates those warnings referenced below.
They are sometimes associated with an obscure #define.

We'll also corrrect one header file function prototype
so it will aggree with the actual function definition.

Reference(s):
top.c: In function 'adj_geometry':
top.c:1874:20: warning: comparison of integer expressions of different signedness: 'int' and 'long unsigned int' [-Wsign-compare]
 1874 |    if (Screen_cols < DOUBLE_limit) Curwin->rc.double_up = 0;
      |                    ^
top.c: In function 'zap_fieldstab':
top.c:2359:26: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare]
 2359 |    if (wtab[EU_CPN].wmin < digits) {
      |                          ^
top.c:2365:26: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare]
 2365 |    if (wtab[EU_NMA].wmin < digits) {
      |                          ^
top.c: In function 'keys_summary':
top.c:5128:45: warning: comparison of integer expressions of different signedness: 'int' and 'long unsigned int' [-Wsign-compare]
 5128 |          if (w->rc.double_up && Screen_cols < DOUBLE_limit) {
      |                                             ^
top.c: In function 'sum_tics':
top.c:5605:22: warning: unused variable 'num_syst' [-Wunused-variable]
 5605 |    int ix, num_user, num_syst;
      |                      ^~~~~~~~
top.c:5605:12: warning: unused variable 'num_user' [-Wunused-variable]
 5605 |    int ix, num_user, num_syst;
      |            ^~~~~~~~

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: attempt to clarify the 'iokey' parameter meanings
Jim Warner [Fri, 11 Feb 2022 06:00:00 +0000 (00:00 -0600)]
top: attempt to clarify the 'iokey' parameter meanings

The 'iokey' function's parameter 'action' was utilized
with literal numbers in the calling functions. So this
change will replace those literal numbers with #define
constants which, one hopes, will clarify their impact.

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agofree: Use wstr width and not length
Craig Small [Tue, 1 Feb 2022 05:46:09 +0000 (16:46 +1100)]
free: Use wstr width and not length

The previous commit used the value from mbstowcs() to work out
the spacing required, as printf() got it completely wrong.

However, for alignment of text you don't use the string length
but the string width.

As the referenced website says:
 Use wcslen when allocating memory for wide characters, and use wcswidth to
 align text.

Which is what free does now. Chinese is still off by one but I cannot
see why this is so. It's close enough for now. If someone can work
it out, I'd love to know what the fix is.

As a side effect, #213 is fixed because we are putting the correct
number of spaces in.

French is still an issue (see #24 ) but this is because the string is
too long!

References:
 procps-ng/procps#24
 procps-ng/procps#213
 procps-ng/procps#229
 commit 9f4db0fb5606e4872829bd44b29443d5707b1505
 https://www.linux.com/news/programming-wide-characters/

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agofree: Fix first column justification
Craig Small [Mon, 31 Jan 2022 07:14:12 +0000 (18:14 +1100)]
free: Fix first column justification

free has for many years had a problem with translated header columns
or the first column. This is because printf("-9s", str) doesn't use
the wide length of the string but the char length meaning they are
mis-aligned.

Using the mbstowcs() function to get the wide length and then
a precision parameter to append the right number of spaces after the
number means we get what we need.

References:
 procps-ng/procps#229
 procps-ng/procps#204
 procps-ng/procps#206
 https://bugs.debian.org/1001689

Signed-off-by: Craig Small <csmall@dropbear.xyz>
2 years agoNEWS: updated to acknowledge 'LIBPROC_HIDE_KERNEL' var
Jim Warner [Thu, 27 Jan 2022 06:00:00 +0000 (00:00 -0600)]
NEWS: updated to acknowledge 'LIBPROC_HIDE_KERNEL' var

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: tweak the heck out of some command line help text
Jim Warner [Wed, 26 Jan 2022 06:00:00 +0000 (00:00 -0600)]
top: tweak the heck out of some command line help text

This patch began as just an attempt to make any option
which also included an argument a little more readable
by adding one space before the '=ARGUMENT' convention.

[ by the way, i don't agree with most of those other ]
[ procps-ng programs that use an '<arg>' convention. ]
[ it's too easily misread as an 'optional' argument. ]

[ top uses a convention like that found in coreutils ]
[ albeit now with one extra space before the equals. ]

In adjusting those arguments it was apparent that many
explanations already lined up nicely at the right hand
margin. So, this commit will force right-justification
with all explanations (as we do with commit messages).

[ and as a final challenge, for those options taking ]
[ an argument, that argument was repeated within the ]
[ explanation and made the rightmost item on a line. ]

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: eliminate the potential warnings '-Wsign-compare'
Jim Warner [Tue, 25 Jan 2022 06:00:00 +0000 (00:00 -0600)]
top: eliminate the potential warnings '-Wsign-compare'

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: avoid a potential warning '-Wmaybe-uninitialized'
Jim Warner [Tue, 25 Jan 2022 06:00:00 +0000 (00:00 -0600)]
top: avoid a potential warning '-Wmaybe-uninitialized'

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: avoid a possible warning '-Wimplicit-fallthrough'
Jim Warner [Tue, 25 Jan 2022 06:00:00 +0000 (00:00 -0600)]
top: avoid a possible warning '-Wimplicit-fallthrough'

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: add visual clue when focus toggle ('F') is active
Jim Warner [Fri, 14 Jan 2022 20:14:14 +0000 (14:14 -0600)]
top: add visual clue when focus toggle ('F') is active

When in forest view mode, that focus toggle ('F') is a
useful tool occasionally. But, if a focused parent has
enough cloned siblings to exceed screen rows, it could
be hard to remember that such a toggle remains active.

So, this patch will provide a subtle visual clue added
to the leftmost position in the COMMAND column. Now if
the focus toggle was active, regardless of total tasks
affected, the users will always know when it's active.

Reference(s):
. -7/24/21, introduced new focus toggle
commit 3e922e671daf09ae60ee211b5c0ef94996ee573d
. 09/23/21, ensure focused tasks stay focused
commit 69978e365043f27305e487709474947bb377084d

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: correct one source comment plus one man page oops
Jim Warner [Thu, 13 Jan 2022 06:00:00 +0000 (00:00 -0600)]
top: correct one source comment plus one man page oops

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agotop: banish one more warning for '-Wformat-truncation'
Jim Warner [Thu, 13 Jan 2022 19:13:13 +0000 (13:13 -0600)]
top: banish one more warning for '-Wformat-truncation'

In the commit referenced below, I fixed what I thought
was all the top 'truncation' warnings. For that commit
I had been using CFLAGS='-ggdb -Wall'. However, if one
uses just a vanilla './configure', then a hidden extra
warning will surface. This patch will finally kill it.

[ thanks a bunch gcc - we love this kind of behavior ]

Reference(s):
. 01/08/22, original warning fix
commit 44ca06f1a05b31880ca0ec4e59923a3255f5816d

Signed-off-by: Jim Warner <james.warner@comcast.net>
2 years agosysctl: eliminate a warning '-Wmisleading-indentation'
Jim Warner [Sat, 8 Jan 2022 06:00:00 +0000 (00:00 -0600)]
sysctl: eliminate a warning '-Wmisleading-indentation'

[ and in the affected function, we'll also eliminate ]
[ all those f**king tab characters making a hot mess ]
[ of any attempt at a properly formatted C function! ]

[ plus i adjusted a little non-tab misalignment too! ]

Signed-off-by: Jim Warner <james.warner@comcast.net>