Bug 218889 - top(1) show CPU states header in batch mode
Summary: top(1) show CPU states header in batch mode
Status: New
Alias: None
Product: Base System
Classification: Unclassified
Component: bin (show other bugs)
Version: 11.0-RELEASE
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-bugs mailing list
Keywords: patch
Depends on:
Reported: 2017-04-26 13:35 UTC by reed
Modified: 2018-05-20 18:39 UTC (History)
1 user (show)

See Also:


Note You need to log in before you can comment on or make changes to this bug.
Description reed 2017-04-26 13:35:36 UTC
(The component field here in bugzilla doesn't have "contrib" so I chose "bin".)

I don't know history of why the dostates toggle is there to not show the CPU: states "the first time".  But this makes it so the batch view (like piped top to a file) excludes the CPU states and just has a blank line there.

Here is a simple fix (but maybe fix comment). Or maybe just get rid of "dostates".

Index: contrib/top/top.c
--- contrib/top/top.c   (revision 317435)
+++ contrib/top/top.c   (working copy)
@@ -637,7 +637,7 @@
        /* display the cpu state percentage breakdown */
-       if (dostates)   /* but not the first time */
+       if ((!interactive) || (dostates))       /* but not the first time */

(I originally saw this problem with pfSense's System Activity page while authoring a book about it.)