Hi Samuel, I thought I should let you and others know of some related changes I've been playing around with to try and improve our support for the INTEL_performance_query extension in the i965 dri driver, since it's quite related to your work.
Some of my work-in-progress changes can currently be seen on github here: https://github.com/rib/mesa/commits/wip/rib/i915_oa_perf I've only recently scrubbed up these mesa changes enough for reviewing to make much sense, so sorry I haven't sent out an RFC to the list before now about this. What I was finding is that it's not very helpful in practice to support the AMD_performance_monitor + INTEL_performance query extensions via the same core infrastructure and so since there weren't previously any other drivers implementing AMD_performance_monitor, my series currently just drops the combined infrastructure in favour of core support for just INTEL_performance_query. The main thing here that might be interesting to think about with respect to your work, is whether you might like to do anything differently with the main/performance_monitor.c support if you were the only user and if it strictly only dealt with the AMD_performance_monitor extension? I don't guess it would be controversial for us to look at removing the current i965 backend for both AMD_performance_monitor and INTEL_performance_query for an interim if that might be convenient for you to evolve the backend interface. (Currently the i965 backend doesn't report many usable metrics unfortunately - mainly just the pipeline statistics that are also accessible via query objects.) Ken and Petri who did much of the work here are already aware that I've been taking this approach as I ran it by them some time ago and their work still carries forward in the series I have but instead focused only on INTEL_performance_query for now. Anyway, I just thought I should throw that out there as a possibility to consider in case it might be helpful. Kind regards, - Robert On Wed, Mar 18, 2015 at 4:00 PM, Samuel Pitoiset <samuel.pitoi...@gmail.com> wrote: > Bump, is someone want to make a review of this patch set ? Especially the > "core" stuff... > > > On 03/09/2015 10:09 PM, Samuel Pitoiset wrote: >> >> Hello, >> >> A series I have waited too long to re-submit, but I recently refactored >> the >> code and fixed some minor issues. >> >> This patchset enables GL_AMD_performance_monitor for svga, freedreno, >> r600, >> radeonsi and nvc0 drivers. >> >> This code has been tested with Nouveau (NVD9 and NVE7) but it should also >> work >> with other drivers. All piglit tests, including API and measurement tests >> are >> okay. >> >> Feel free to make a review. >> >> Christoph Bumiller (1): >> st/mesa: implement GL_AMD_performance_monitor >> >> Samuel Pitoiset (14): >> gallium: add pipe_screen::get_driver_query_group_info >> gallium: add new fields to pipe_driver_query_info >> gallium: add new numeric types to pipe_query_result >> gallium: replace pipe_driver_query_info::max_value by a union >> gallium: make pipe_context::begin_query return a boolean >> gallium: add util_get_driver_query_group_info >> svga: implement pipe_screen::get_driver_query_group_info >> freedreno: implement pipe_screen::get_driver_query_group_info >> radeon: implement pipe_screen::get_driver_query_group_info >> nvc0: implement pipe_screen::get_driver_query_group_info >> docs: mark GL_AMD_performance_monitor for the 10.6.0 release >> nvc0: expose more driver-specific query groups >> nvc0: make begin_query return false when all MP counters are used >> nvc0: all queries use an unsigned 64-bits integer by default >> >> docs/relnotes/10.6.0.html | 1 + >> src/gallium/auxiliary/Makefile.sources | 1 + >> src/gallium/auxiliary/hud/hud_driver_query.c | 2 +- >> src/gallium/auxiliary/util/u_query.c | 50 +++ >> src/gallium/auxiliary/util/u_query.h | 45 ++ >> src/gallium/docs/source/screen.rst | 10 + >> src/gallium/drivers/freedreno/freedreno_query.c | 25 +- >> src/gallium/drivers/freedreno/freedreno_query.h | 3 +- >> src/gallium/drivers/freedreno/freedreno_query_hw.c | 3 +- >> src/gallium/drivers/freedreno/freedreno_query_sw.c | 3 +- >> src/gallium/drivers/galahad/glhd_context.c | 6 +- >> src/gallium/drivers/i915/i915_query.c | 3 +- >> src/gallium/drivers/ilo/ilo_query.c | 3 +- >> src/gallium/drivers/llvmpipe/lp_query.c | 3 +- >> src/gallium/drivers/noop/noop_pipe.c | 3 +- >> src/gallium/drivers/nouveau/nv30/nv30_query.c | 5 +- >> src/gallium/drivers/nouveau/nv50/nv50_query.c | 3 +- >> src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 98 ++++- >> src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 1 + >> src/gallium/drivers/nouveau/nvc0/nvc0_screen.h | 14 + >> src/gallium/drivers/r300/r300_query.c | 9 +- >> src/gallium/drivers/radeon/r600_pipe_common.c | 25 +- >> src/gallium/drivers/radeon/r600_pipe_common.h | 1 + >> src/gallium/drivers/radeon/r600_query.c | 16 +- >> src/gallium/drivers/rbug/rbug_context.c | 8 +- >> src/gallium/drivers/softpipe/sp_query.c | 3 +- >> src/gallium/drivers/svga/svga_context.h | 1 + >> src/gallium/drivers/svga/svga_pipe_query.c | 3 +- >> src/gallium/drivers/svga/svga_screen.c | 17 +- >> src/gallium/drivers/trace/tr_context.c | 6 +- >> src/gallium/include/pipe/p_context.h | 2 +- >> src/gallium/include/pipe/p_defines.h | 34 +- >> src/gallium/include/pipe/p_screen.h | 11 + >> src/mesa/Makefile.sources | 2 + >> src/mesa/state_tracker/st_cb_perfmon.c | 455 >> +++++++++++++++++++++ >> src/mesa/state_tracker/st_cb_perfmon.h | 70 ++++ >> src/mesa/state_tracker/st_context.c | 4 + >> src/mesa/state_tracker/st_extensions.c | 3 + >> 38 files changed, 885 insertions(+), 67 deletions(-) >> create mode 100644 src/gallium/auxiliary/util/u_query.c >> create mode 100644 src/gallium/auxiliary/util/u_query.h >> create mode 100644 src/mesa/state_tracker/st_cb_perfmon.c >> create mode 100644 src/mesa/state_tracker/st_cb_perfmon.h >> > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
