Re: [PATCH trivial v3] plugins: add missing docstrings to qemu-plugin.h

On 3/11/26 3:25 AM, Florian Hofhammer wrote:
> This patch adds docstrings for typedefs and function declarations in
> include/plugins/qemu-plugin.h that were previously missing. This
> resolves inconsistencies in the docs, e.g., the description for
> qemu_plugin_read_register() referring to qemu_plugin_register_flush_cb()
> but code cache flush callbacks not being documented themselves.
> 
> Signed-off-by: Florian Hofhammer <[email protected]>
> ---
> Hi,
> 
> While working on a QEMU plugin and browsing the online docs at
> https://www.qemu.org/docs/master/devel/tcg-plugins.html, I noticed that
> some of the API functions were not actually documented.
> 
> I went through the include/plugins/qemu-plugin.h header file and added
> docstrings where they were missing, hoping of course that I documented
> the correct functionality based on my understanding of the plugin
> internals.
> 
> I hope that's useful and I didn't miss anything!
> 
> Best regards,
> Florian
> 
> Changes in v3:
> - Fix documentation build issue due to incorrect parameter name
> - Link to v2: https://lore.kernel.org/qemu-devel/[email protected]
> Changes in v2:
> - Added semantics clarification to the qemu_plugin_register_flush_cb()
>    docstring (suggested by Pierrick)
> - Link to v1: https://lore.kernel.org/qemu-devel/[email protected]
> ---
>   include/plugins/qemu-plugin.h | 106 +++++++++++++++++++++++++++++++++++++-----
>   1 file changed, 94 insertions(+), 12 deletions(-)
> 

Thanks for updating it!
Reviewed-by: Pierrick Bouvier <[email protected]>