vkCmdResetQueryPool called in TracyVkCollect despite using VK_EXT_host_query_reset

[maxime-modulopi]

vkCmdResetQueryPool is called instead of vkResetQueryPool in TracyVkCollect even when the context was created with TracyVkContextHostCalibrated:

tracy/public/tracy/TracyVulkan.hpp

Line 328 in f14ff2b

VK_FUNCTION_WRAPPER( vkCmdResetQueryPool( cmdbuf, m_query, wrappedTail, cnt ) );

I believe forcing the "Cmd" variant is not needed here?

Also, this would allow removing the cmdbuf parameter of TracyVkCollect when host query reset are used, making it more flexible in when it is called.

[wolfpld]

vkCmdResetQueryPool is Vulkan 1.0, vkResetQueryPool is 1.2. Before you ask, Android.

[maxime-modulopi]

Yes but Tracy already supports creating a TracyVkCtx with host resets via TracyVkContextHostCalibrated (which already calls vkResetQueryPool in its constructor).

Similarly to this, I would have expected to be able to use vkResetQueryPool inside TracyVkCollect too (only if the context was created via TracyVkContextHostCalibrated obviously).