[Mesa-dev] [PATCH 0/4] radeonsi: experimental support for GPUPerfStudio

eocallaghan at alterapraxis.com eocallaghan at alterapraxis.com
Wed Feb 3 21:58:20 UTC 2016


Can't see any serious issues here and a non-verified `working' instance 
of
GPUPerfStudio is sure better than a crashing one!

This series is,

Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>


On 2016-02-04 00:52, Nicolai Hähnle wrote:
> Hi,
> 
> this bunch of patches meets GPUPerfStudio half-way in supporting the 
> timing
> features on CI+ hardware. The latest version of GPUPerfStudio is 
> required.
> 
> With these patches, GPUPerfStudio should recognize our driver as 
> supported
> and offer its frame profiling features without crashing. It should also
> report reasonable numbers in the profile. However, I haven't fully
> validated the reported numbers, so while I'd like to get this merged 
> now,
> it should still be considered as somewhat experimental. Please review.
> 
> Thanks,
> Nicolai
> --
>  .../drivers/radeon/r600_perfcounter.c        |  38 +++---
>  src/gallium/drivers/radeon/r600_query.c      |  80 ++++++++++-
>  src/gallium/drivers/radeon/r600_query.h      |  32 ++---
>  .../drivers/radeonsi/si_perfcounter.c        | 121 +++++++++++++----
>  4 files changed, 201 insertions(+), 70 deletions(-)
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev



More information about the mesa-dev mailing list