[RFC PATCH] plugins: bump plugin API version

Julian Ganz posted 1 patch 1 month, 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20251227094240.6652-1-neither@nut.email
include/qemu/qemu-plugin.h | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
[RFC PATCH] plugins: bump plugin API version
Posted by Julian Ganz 1 month, 1 week ago
We introduced a new function `qemu_plugin_register_vcpu_discon_cb` as
well as some types since v5.

Signed-off-by: Julian Ganz <neither@nut.email>

--

I should probably have included this in my patch series introducing the
register function in question instead of noticing that the API version
was not bumped with 10.2.0 after the fact.
---
 include/qemu/qemu-plugin.h | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/include/qemu/qemu-plugin.h b/include/qemu/qemu-plugin.h
index 60de4fdd3f..b445c26389 100644
--- a/include/qemu/qemu-plugin.h
+++ b/include/qemu/qemu-plugin.h
@@ -72,11 +72,14 @@ typedef uint64_t qemu_plugin_id_t;
  * - added qemu_plugin_write_memory_hwaddr
  * - added qemu_plugin_write_register
  * - added qemu_plugin_translate_vaddr
+ *
+ * version 6:
+ * - qemu_plugin_register_vcpu_discon_cb
  */
 
 extern QEMU_PLUGIN_EXPORT int qemu_plugin_version;
 
-#define QEMU_PLUGIN_VERSION 5
+#define QEMU_PLUGIN_VERSION 6
 
 /**
  * struct qemu_info_t - system information for plugins
-- 
2.51.2
Re: [RFC PATCH] plugins: bump plugin API version
Posted by Alex Bennée 1 month, 1 week ago
Julian Ganz <neither@nut.email> writes:

> We introduced a new function `qemu_plugin_register_vcpu_discon_cb` as
> well as some types since v5.
>
> Signed-off-by: Julian Ganz <neither@nut.email>

Queued to plugins/next, thanks.

>
> --
>
> I should probably have included this in my patch series introducing the
> register function in question instead of noticing that the API version
> was not bumped with 10.2.0 after the fact.
> ---
>  include/qemu/qemu-plugin.h | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/include/qemu/qemu-plugin.h b/include/qemu/qemu-plugin.h
> index 60de4fdd3f..b445c26389 100644
> --- a/include/qemu/qemu-plugin.h
> +++ b/include/qemu/qemu-plugin.h
> @@ -72,11 +72,14 @@ typedef uint64_t qemu_plugin_id_t;
>   * - added qemu_plugin_write_memory_hwaddr
>   * - added qemu_plugin_write_register
>   * - added qemu_plugin_translate_vaddr
> + *
> + * version 6:
> + * - qemu_plugin_register_vcpu_discon_cb
>   */
>  
>  extern QEMU_PLUGIN_EXPORT int qemu_plugin_version;
>  
> -#define QEMU_PLUGIN_VERSION 5
> +#define QEMU_PLUGIN_VERSION 6
>  
>  /**
>   * struct qemu_info_t - system information for plugins

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro
Re: [RFC PATCH] plugins: bump plugin API version
Posted by Alex Bennée 1 month, 1 week ago
Julian Ganz <neither@nut.email> writes:

> We introduced a new function `qemu_plugin_register_vcpu_discon_cb` as
> well as some types since v5.
>
> Signed-off-by: Julian Ganz <neither@nut.email>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Cc: qemu-stable@nongnu.org

>
> --
>
> I should probably have included this in my patch series introducing the
> register function in question instead of noticing that the API version
> was not bumped with 10.2.0 after the fact.
> ---
>  include/qemu/qemu-plugin.h | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/include/qemu/qemu-plugin.h b/include/qemu/qemu-plugin.h
> index 60de4fdd3f..b445c26389 100644
> --- a/include/qemu/qemu-plugin.h
> +++ b/include/qemu/qemu-plugin.h
> @@ -72,11 +72,14 @@ typedef uint64_t qemu_plugin_id_t;
>   * - added qemu_plugin_write_memory_hwaddr
>   * - added qemu_plugin_write_register
>   * - added qemu_plugin_translate_vaddr
> + *
> + * version 6:
> + * - qemu_plugin_register_vcpu_discon_cb
>   */
>  
>  extern QEMU_PLUGIN_EXPORT int qemu_plugin_version;
>  
> -#define QEMU_PLUGIN_VERSION 5
> +#define QEMU_PLUGIN_VERSION 6
>  
>  /**
>   * struct qemu_info_t - system information for plugins

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro