Looks good to me. For the series:
Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> On 06/11/2017 03:52 PM, Boyan Ding wrote:
Some performance counters in nouveau uses non-integer type in AMD_perfmon, but they are currently returning int values. One reason behind this is that gallium hud, which is one of the users of the counters, only supports integers. This series tries to fix problem in both parts -- making nouveau return values of appropriate types and teaching hud to be aware of types. Although hud is still not clever enough to handle floating points, it is becoming no worse. Note that this series is highly RFC. I'm posting this to solicit ideas, both on whether this approach is appropriate and details in handling -- the current code looks somewhat ugly. There are also further problems that I look to solve, namely some performance counters, such as ipc, which should be floats instead of ints. But I want to get basics correct first. Boyan Ding (2): hud: Handle query values according to their type nvc0: Return value of appropriate type instead of u64 src/gallium/auxiliary/hud/hud_driver_query.c | 22 ++++++- .../drivers/nouveau/nvc0/nvc0_query_hw_metric.c | 70 +++++++++++++--------- 2 files changed, 61 insertions(+), 31 deletions(-)
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev