Commit Graph

14047 Commits

Author SHA1 Message Date
Karel Zak 02f2919e95 irqtop: minor cleanup
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 14:11:30 +01:00
Karel Zak 94e7e258a3 irqtop: move independent code to irq-common.c
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 14:04:33 +01:00
Karel Zak c6ccf2ec96 irqtop: add struct irq_output
* add irq_output to keep top-like stufff independent on data
  management

* remove horrible "const"

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 13:38:02 +01:00
Karel Zak 212ca752c1 irqtop: keep table in functions only
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:56:56 +01:00
Karel Zak a6fa6aad4b irqtop: remove dead code
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:46:45 +01:00
Karel Zak ec95b4365f irqtop: separate normal and ncurses way
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:44:16 +01:00
Karel Zak d5280921be irqtop: remove unnecessary code
* remove unused code smp_num_cpus
* care about hostname and timer only real top-like output

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:36:41 +01:00
Karel Zak 60794bd2c6 irqtop: keep WINDOW pointer in functions only
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:32:39 +01:00
Karel Zak f675b00803 irqtop: separate screen and scols code
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:25:40 +01:00
Karel Zak bd83424b80 irqtop: reorder function
Move functions to make next changes more readable.

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:12:12 +01:00
Karel Zak 487c746638 irqtop: cleanup command line options
* use -o for output columns (like we have in another tools)
* support -o +COLUMN

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-06 12:06:22 +01:00
Sami Kerola b5886f298e
irqtop: trim white spaces from end of name field
Excess white spaces were easiest to see in json output.

    $ irqtop --once --json
    {
       "interrupts": [
          {"irq":"LOC", "total":7425148, "name":"Local timer interrupts "},
          {"irq":"51", "total":1848384, "name":"IR-PCI-MSI 32768-edge i915 "},
          {"irq":"RES", "total":1176665, "name":"Rescheduling interrupts "},
          ...

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-03-04 21:40:24 +00:00
Karel Zak f5377ca454 irqtop: use -J for JSON
* add -J to the man page
* add note about default output
* be compatible with another util-linux tools and use -J

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-04 14:58:18 +01:00
Karel Zak 0d8ead9327 irqtop: don't print header for --once
We don't use any extra header for lsblk, lsmem, ... so don't make it
different.

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-04 14:51:12 +01:00
Karel Zak 642f6cb0cb irqtop: simplify terminal resizing
* don't care about term size and attrs when runs only once
* explicitly define signals
* resize immediately after SIGWINCH (to keep it nice for users)

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-04 14:47:20 +01:00
Karel Zak 6f99d9d613 iqrtop: cleanup header
* add '|' to make it more readable
* use time string without 'T'

Signed-off-by: Karel Zak <kzak@redhat.com>
2020-03-04 14:17:27 +01:00
Sami Kerola ebdb0c350a
docs: remove irqtop TODO item
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-24 18:27:16 +00:00
Sami Kerola d7f9cdf535
irqtop: tidy coding style and update usage() text
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-24 18:26:38 +00:00
Sami Kerola d77e9e13dd
irqtop: do not use fixed size /proc/interrupts line buffer
Allocating and freeing all the time is a little bit heavier, but it is also
more robust.  It is impossible to know if field lenghts stay fixed.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-23 17:36:33 +00:00
Sami Kerola 68afc56bcd
irqtop: add total and delta as own columns
This commit add total and delta as separate columns rather than a toggle,
making output generally more informative.  When using command I found that
sort order key bindings should match with first letter of column headers so
that they are as easy as possible to remember, and therefore enhancing user
experience.

Requested-by: Karel Zak <kzak@redhat.com>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-23 17:27:12 +00:00
Sami Kerola 28ddd62bed
irqtop: use name instead of desc as irq name field referal
This matches with sort order n for name, and allows later usage of d for
delta without needing to go around d for description.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-22 21:43:54 +00:00
Sami Kerola daee272009
irqtop: improve header
Change total to display delta or count depending on run mode.  Add space in
between header and irqtop body.  Print only number of active CPU in header.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-22 21:43:54 +00:00
Sami Kerola bf611f4034
irqtop: hide cursor when in interactive mode
Visible cursor is distraction.  Secondly with visible cursor key commands
can be visible adding unwanted characters to end of output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-22 21:43:54 +00:00
Sami Kerola 11b20fb29d
irqtop: display number of new interupts in-between updates
The rirst output is total count followed by number of interupts since
previous screen update.  This is how the irqtop worked before libsmartcols
was added to the command.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-22 21:43:54 +00:00
Sami Kerola e11f242769
irqtop: add bash-completion
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-22 21:43:54 +00:00
Sami Kerola 050dc9f923
irqtop: add manual page
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-22 21:43:54 +00:00
Sami Kerola 8d8cef8045
irqtop: use libsmartcols
This makes output to be automatically be nicely adjusted to optimal width.
In this commit also removes excess spaces from description strings.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-22 21:43:40 +00:00
Sami Kerola a163d853c6
irqtop: include hostname and timestamp to output header
Having a when and from where information is useful in situations when output
is examined later and possibly elsewhere.  Typical example would be a
screenshot or output copy-paste of a terminal to a ticketing system.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:15 +00:00
Sami Kerola fa8c5cd503
irqtop: use epoll event loop
The event loop takes care of the standard screen updates, signals, and user
inputs.  Two nice things using even loop like this are; 1) no need for any
global variables and 2) effect of user commands is immediate.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:15 +00:00
Sami Kerola 29135b2320
irqtop: move screen update to a separate function
The main() was too long.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:15 +00:00
Sami Kerola bc90c16a78
irqtop: fix open file descriptor leak
With very short delay one can see irqfile file descriptors pile up rather
quickly causing program to stop with too many open files error.  This commit
fixes that, and removes couple other memory leaks.  With these the command
should hopefully run fine for days if not even months.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:14 +00:00
Sami Kerola 4e55ffbdde
irqtop: change the update delay to use struct timeval
The struct timeval is more expressive, and it allows use of fractional and
subsecond intervals.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:14 +00:00
Sami Kerola a937ed0fd4
irqtop: use runtime control structure
This is pretty standard util-linux way to avoid global variables, and pass
runtime configuration around.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:14 +00:00
Sami Kerola 465e279a26
irqtop: use lib/monotonic.c to determine uptime
This should be functionally the same, and has the advantage of avoiding
duplicated code.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:14 +00:00
Sami Kerola 9aea73b3ba
irqtop: use memory allocation that check errors
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:14 +00:00
Sami Kerola a11660d155
irqtop: avoid function like pre-processor definitions
And simplify how terminal size determination.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:14 +00:00
Sami Kerola 57e3dc061b
irqtop: use util-linux libcommon facilities
Including NLS support.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:14 +00:00
Sami Kerola 4f6610f5ca
irqtop: make util-linux build-system to build the command
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:13 +00:00
Sami Kerola 952690cfd2
irctop: move source code to sys-utils/ directory
And remove files that util-linux does not need from the earlier repository.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-21 21:09:13 +00:00
zhenwei pi 2cd409cf71
irqtop: fix all warnings
use -Wall for compiling flags, and fix all warnings.

Signed-off-by: zhenwei pi <pizhenwei@bytedance.com>
2020-02-21 21:09:13 +00:00
zhenwei pi 6ce4486272
irqtop: init README
add basic infomation in README

Signed-off-by: zhenwei pi <pizhenwei@bytedance.com>
2020-02-21 21:09:13 +00:00
zhenwei pi ae0dfe1412
irqtop: implement a new utility to display kernel interrupt
currently, there are usually 40/48/64/96 CPUs on a single server,
and a lot of interrupts are enabled by multi-queues of a NIC.
"/proc/interrupts" is not human readable any more.
'watch -d -n 1 "cat /proc/interrupts"' also can not work well.

so implement irqtop to show the interrupts information, we can
sort the interrupts by count(default), name and /proc/interrupts.

irqtop - IRQ : 49, TOTAL : 2361705032, CPU : 8, ACTIVE CPU : 8
 IRQ        COUNT   DESC
 CAL        21196   Function call interrupts
 LOC        13733   Local timer interrupts
 154         1430   IR-PCI-MSI 32768-edge      i915
 127         1322   IR-PCI-MSI 327680-edge      xhci_hcd
 RES         1224   Rescheduling interrupts
 146          336   IR-PCI-MSI 520192-edge      enp0s31f6
 IWI          135   IRQ work interrupts
 147           48   IR-PCI-MSI 31981569-edge      nvme0q2
 151           42   IR-PCI-MSI 31981573-edge      nvme0q6
 TLB            8   TLB shootdowns
 150            7   IR-PCI-MSI 31981572-edge      nvme0q5
 152            5   IR-PCI-MSI 31981574-edge      nvme0q7
 156            4   IR-PCI-MSI 1572864-edge      iwlwifi
 148            3   IR-PCI-MSI 31981570-edge      nvme0q3
 153            2   IR-PCI-MSI 31981575-edge      nvme0q8
 NMI            2   Non-maskable interrupts
 PMI            2   Performance monitoring interrupts
   0            0   IR-IO-APIC    2-edge      timer
   1            0   IR-IO-APIC    1-edge      i8042
   8            0   IR-IO-APIC    8-edge      rtc0
   9            0   IR-IO-APIC    9-fasteoi   acpi
  12            0   IR-IO-APIC   12-edge      i8042

test on 4.14 & 4.19, work fine. test on bare metal & kvm virtual
machine, work fine. hot-plug/hot-unplug virtual NIC during running
irqtop, work fine.

Signed-off-by: zhenwei pi <pizhenwei@bytedance.com>
2020-02-21 21:09:13 +00:00
Karel Zak e14bd7d317 Merge branch 'master' of https://github.com/benfrankel/util-linux
* 'master' of https://github.com/benfrankel/util-linux:
  docs: Improve grammar
2020-02-21 09:29:48 +01:00
Ben Frankel 1ad0b2feda
docs: Improve grammar 2020-02-20 16:46:47 -08:00
Sami Kerola 3c36438f0b
fdisk: update expected test outputs with command outputs
Reference; https://github.com/karelzak/util-linux/pull/959
Addresses: 5d271cefad
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-20 21:19:37 +00:00
Sami Kerola f6b6beaf6a
various: fix more lgtm scan warnings
The logger and rtwake time function changes continue the same fixes as
previous commit - use thread safe functions.  The libsmartcols condition
removal is possible because width must be greater than tb->termwidth that is
size_t and cannot be smaller than zero.  And remove couple FIXME's that are
old and unlikely ever to get fixed.

Reference: 3160589d86
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
2020-02-20 20:18:46 +00:00
Karel Zak 5d271cefad fdisk: improve list-types readability
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-02-20 15:27:05 +01:00
Karel Zak 4d755765b4 sfdisk: (man) add note about type and shortcuts collision
Signed-off-by: Karel Zak <kzak@redhat.com>
2020-02-20 15:26:28 +01:00
Karel Zak aa506f7cc5 Merge branch 'master' of https://github.com/pali/util-linux
* 'master' of https://github.com/pali/util-linux:
  exfat: Fix parsing exfat label
2020-02-18 14:06:09 +01:00
Karel Zak 88dbec9833 Merge branch 'various-fixes' of https://github.com/kerolasa/util-linux
* 'various-fixes' of https://github.com/kerolasa/util-linux:
  kill: include sys/types.h before checking SYS_pidfd_send_signal
  libfdisk: fix pointer wraparound warning
  hwclock: update yacc file
  script: fix minor warning
  getopt: use examples installation directory in man page
  fstrim: randomize timer start time across 100 minutes
  various: use threadsafe versions of time functions [lgtm scan]
  write: fix potential string overflow
2020-02-18 14:02:41 +01:00