I believe I've addressed the various feedback -- I've reduced the pipe-side types to a more manageable quantity, renamed the debug info struct to debug callback, and added a few comments. An earlier version of the st/clover patch was tested too.
Ilia Mirkin (6): gallium: expose a debug message callback settable by context owner st/mesa: set debug callback for debug contexts st/clover: provide a path for drivers to call through to pfn_notify nouveau: add support for sending debug messages via KHR_debug nv50,nvc0: provide debug messages with shader compilation stats nouveau: send back a debug message when waiting for a fence to complete src/gallium/auxiliary/util/u_debug.c | 14 ++++++ src/gallium/auxiliary/util/u_debug.h | 20 ++++++++ src/gallium/docs/source/context.rst | 3 ++ .../drivers/nouveau/codegen/nv50_ir_driver.h | 1 + .../drivers/nouveau/codegen/nv50_ir_target.cpp | 2 + src/gallium/drivers/nouveau/nouveau_buffer.c | 13 ++--- src/gallium/drivers/nouveau/nouveau_context.h | 5 ++ src/gallium/drivers/nouveau/nouveau_fence.c | 14 +++++- src/gallium/drivers/nouveau/nouveau_fence.h | 4 +- src/gallium/drivers/nouveau/nouveau_screen.c | 21 +++++++- src/gallium/drivers/nouveau/nv30/nv30_context.c | 1 + src/gallium/drivers/nouveau/nv30/nv30_screen.c | 2 +- src/gallium/drivers/nouveau/nv50/nv50_context.c | 1 + src/gallium/drivers/nouveau/nv50/nv50_program.c | 8 +++- src/gallium/drivers/nouveau/nv50/nv50_program.h | 3 +- src/gallium/drivers/nouveau/nv50/nv50_screen.c | 2 +- .../drivers/nouveau/nv50/nv50_shader_state.c | 2 +- src/gallium/drivers/nouveau/nv50/nv50_state.c | 3 +- src/gallium/drivers/nouveau/nv50/nv50_vbo.c | 2 +- src/gallium/drivers/nouveau/nvc0/nvc0_compute.c | 2 +- src/gallium/drivers/nouveau/nvc0/nvc0_context.c | 1 + src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 3 +- src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 8 +++- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +- .../drivers/nouveau/nvc0/nvc0_shader_state.c | 2 +- src/gallium/drivers/nouveau/nvc0/nvc0_state.c | 3 +- src/gallium/drivers/nouveau/nvc0/nvc0_transfer.c | 4 +- src/gallium/include/pipe/p_context.h | 8 ++++ src/gallium/include/pipe/p_defines.h | 12 +++++ src/gallium/include/pipe/p_state.h | 25 ++++++++++ src/gallium/state_trackers/clover/api/context.cpp | 7 ++- src/gallium/state_trackers/clover/core/context.cpp | 5 +- src/gallium/state_trackers/clover/core/context.hpp | 7 ++- src/gallium/state_trackers/clover/core/queue.cpp | 21 ++++++++ src/mesa/state_tracker/st_manager.c | 56 ++++++++++++++++++++++ 35 files changed, 258 insertions(+), 29 deletions(-) -- 2.4.10 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev