On 28/03/18 19:29, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin<tvrtko.ursu...@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.

Signed-off-by: Tvrtko Ursulin<tvrtko.ursu...@intel.com>
Cc: Chris Wilson<ch...@chris-wilson.co.uk>
Cc: Lionel Landwerlin<lionel.g.landwer...@intel.com>
Cc: Petri Latvala<petri.latv...@intel.com>

Looks good to me too. Sorry, this isn't a detailed review, but since it's a lot safer :

Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to