This service will be undergoing maintenance at 00:00 UTC, 2017-10-23 It is expected to last about 30 minutes
Bug 1160811 - pmatop display corruption
pmatop display corruption
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: pcp (Show other bugs)
22
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Nathan Scott
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-05 11:56 EST by Frank Ch. Eigler
Modified: 2015-07-29 22:23 EDT (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-07-29 22:23:59 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
pmatop random screenshot (51.98 KB, image/png)
2014-11-05 11:56 EST, Frank Ch. Eigler
no flags Details
concurrent iostat screenshot (15.30 KB, image/png)
2014-11-05 11:56 EST, Frank Ch. Eigler
no flags Details
proposed pmatop patch (3.42 KB, text/plain)
2014-11-25 13:37 EST, Stan Cox
no flags Details

  None (edit)
Description Frank Ch. Eigler 2014-11-05 11:56:15 EST
Created attachment 954104 [details]
pmatop random screenshot

When running pmatop on a moderately busy workstation for an
extended period of a few minutes, I'm seeing several kinds of 
display corruption.

1) Additional rows appear & disappear near the CPU row temporarily
   duplicating "cpu" information briefly; when the rows disappear,
   the last rows (disk info lines) are left on the screen (now
   duplicated).

2) Disk I/O is not attributed correctly.  While doing a backup, 
   comparing iostat -m and pmatop values, there is little 
   correspondence.  (Attaching screen shots.)  Plus even with
   a fresh ^L screen-refresh, the number of rows with MBr/s 
   stats is fewer than the number of rows with busy/etc. stats.
Comment 1 Frank Ch. Eigler 2014-11-05 11:56:42 EST
Created attachment 954105 [details]
concurrent iostat screenshot
Comment 2 Stan Cox 2014-11-25 13:37:39 EST
Created attachment 961338 [details]
proposed pmatop patch
Comment 3 Stan Cox 2014-11-25 13:38:59 EST
I haven't yet duplicated the exact situation but I noticed this:

Enforce screen width for pmatop.
* pmatop.py (StandardOutput):  Remember the window width...
  (addstr): ...and check we don't write beyond it.
  (timeout): Sleep for piped stdout
  (_Options): Override -L N

with that change I ran this:
atop 5 8  |& /usr/bin/grep 'DSK\|LVM' >| /tmp/,atop &
pmatop.py 5 8 |& /usr/bin/grep 'DSK\|LVM' >| /tmp/,pmatop &

and the results were quite close:

atop
LVM | yong-lv_work | busy      1% | read       0 | write      3 | avio 21.3 ms |
DSK |          sda | busy      1% | read       0 | write      3 | avio 21.3 ms |
LVM | yong-lv_home | busy      2% | read       0 | write     13 | avio 8.46 ms |
LVM | yong-lv_work | busy      0% | read       0 | write      1 | avio 0.00 ms |
DSK |          sda | busy      2% | read       0 | write      8 | avio 13.8 ms |
pmatop
LVM | yong-lv_work |              | read       0 | write      3 |
DSK | sda          | busy      1% | read       0 | write      3 | avio   21 ms |
LVM | yong-lv_work |              | read       0 | write      1 |
LVM | yong-lv_home |              | read       0 | write     13 |
DSK | sda          | busy      2% | read       0 | write      8 | avio   14 ms |
Comment 4 Frank Ch. Eigler 2014-12-15 15:49:20 EST
The new code does not correct the original repot of duplicated lines.
Here's a possible way to reproduce it.
Start pmatop on a nice big xterm, say 132x44, on a fairly idle SMP box.
Once pmatop has started drawing the text, run a couple of cpu-sucking
tasks in the background.  You will notice more "cpu | ..." lines coming
onto the pmatop screen, with different clock frequencies, idle%, etc,
pushing the disks down.  Stop the cpu hogs.  Notice the disk bits left
abandoned at the bottom.

Maybe it's just a matter of clear-screen()ing when dynamic-row type data
changes cardinality.
Comment 5 Jaroslav Reznik 2015-03-03 12:04:58 EST
This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle.
Changing version to '22'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora22
Comment 6 Nathan Scott 2015-07-29 22:23:59 EDT
Resolved in pcp-3.10.5.

Note You need to log in before you can comment on or make changes to this bug.