pstree: Don't stop compact with pgids
authorCraig Small <csmall@dropbear.xyz>
Mon, 21 Jun 2021 12:19:44 +0000 (22:19 +1000)
committerCraig Small <csmall@dropbear.xyz>
Mon, 21 Jun 2021 12:19:44 +0000 (22:19 +1000)
Showing pgids used to disable compaction. But, despite the vague
NPTL documentation, its not possible to have different PGID in the
same thread group, so lack of compaction is not required.

With setpgid(), the last thread in the process that calls it
wins. There might be a debate about should this be the right way
but that's a kernel thing.

References:
 #34
 Test program https://gitlab.com/-/snippets/2094161
  https://man7.org/linux/man-pages/man7/nptl.7.html

ChangeLog
doc/pstree.1
src/pstree.c

index 601a918f09f0934f91747a4b0f0986116cae0365..543507c095444634142baaf14d5abcc8d9604289 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,7 @@
 Changes in NEXT
 ===============
         * pstree: Check for process with show_parents #38
+       * pstree: Don't disable compaction with show pgids #34
 
 Changes in 23.4
 ===============
index 0711954a683b534975c45a8480bac969c724fbfd..8209cf5a43d2149d37223d3383c434b3739907da 100644 (file)
@@ -6,7 +6,7 @@
 .\" the Free Software Foundation; either version 2 of the License, or
 .\" (at your option) any later version.
 .\"
-.TH PSTREE 1 "2021-01-05" "psmisc" "User Commands"
+.TH PSTREE 1 "2021-06-21" "psmisc" "User Commands"
 .SH NAME
 pstree \- display a tree of processes
 .SH SYNOPSIS
@@ -105,9 +105,7 @@ newer than an hour yellow and the remaining red.
 .IP \fB\-g\fP
 Show PGIDs.  Process Group IDs are shown as decimal numbers in
 parentheses after each process name.
-.B \-g
-implicitly disables compaction.  If both PIDs and PGIDs are displayed
-then PIDs are shown first.
+If both PIDs and PGIDs are displayed then PIDs are shown first.
 .IP \fB\-G\fP
 Use VT100 line drawing characters.
 .IP \fB\-h\fP
index 5f687657b34dda09310d95697c1ada8cbd18dd40..64ea40fbfc0ce1e19ad187137a6536cd329007a8 100644 (file)
@@ -1413,7 +1413,6 @@ int main(int argc, char **argv)
             break;
         case 'g':
             pgids = 1;
-            compact = 0;
             break;
         case 's':
             show_parents = 1;