[Intel-gfx] [PATCH i-g-t v2] intel-gpu-top: Rewrite the tool to be safe to use

[Chris Wilson]

Quoting Tvrtko Ursulin (2018-03-29 11:33:34)
> From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> 
> intel-gpu-top is a dangerous tool which can hang machines due unsafe mmio
> register access. This patch rewrites it to use only PMU.
> 
> Only overall command streamer busyness and GPU global data such as power
> and frequencies are included in this new version.
> 
> For access to more GPU functional unit level data, an OA metric based tool
> like gpu-top should be used instead.

intel-gpu-top -  750/ 581 MHz;    0% RC6;   6211mW;    98952 irqs/s

                 ^ easier as actual/request.  Maybe just actual?

    rcs0  97.58% |█████████████████████████████████▏|  0.00% wait,   0.00% sema
    bcs0  92.87% |███████████████████████████████▌  |  0.00% wait,   0.00% sema
    vcs0  92.70% |███████████████████████████████▌  |  0.00% wait,   0.00% sema
   vecs0  92.90% |███████████████████████████████▌  |  0.00% wait,   0.00% sema

2 decimal places seem ok for busyness, but for wait/sema seem overkill.
Drop down to 1dp for them (or all), or even 0dp. Don't fancy overlaying
them on the bar as a different colour block? ;) Hmm, already using ANSI
so you could...
-Chris