[v1] plugin: fix clearing of plugin_mem_cbs on TB exit

[PATCH 0/2] plugin: fix clearing of plugin_mem_cbs on TB exit

Richard Henderson posted 2 patches 1 year, 2 months ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230301024737.1210851-1-richard.henderson@linaro.org

Maintainers: Richard Henderson <richard.henderson@linaro.org>, Paolo Bonzini <pbonzini@redhat.com>

There is a newer version of this series

include/qemu/plugin.h       | 4 ----
accel/tcg/cpu-exec-common.c | 4 ++++
accel/tcg/cpu-exec.c        | 5 +++--
3 files changed, 7 insertions(+), 6 deletions(-)

Expand all Fold all

[PATCH 0/2] plugin: fix clearing of plugin_mem_cbs on TB exit

Posted by Richard Henderson 1 year, 2 months ago

Supercedes: <20230222043204.941336-1-cota@braap.org>
("[PATCH] plugin: fix clearing of plugin_mem_cbs before TB exit")

Let's handle this as we do can_do_io, and reset the value in C.


r~


Richard Henderson (2):
  tcg: Clear plugin_mem_cbs on TB exit
  include/qemu/plugin: Remove QEMU_PLUGIN_ASSERT

 include/qemu/plugin.h       | 4 ----
 accel/tcg/cpu-exec-common.c | 4 ++++
 accel/tcg/cpu-exec.c        | 5 +++--
 3 files changed, 7 insertions(+), 6 deletions(-)

-- 
2.34.1

Re: [PATCH 0/2] plugin: fix clearing of plugin_mem_cbs on TB exit

Posted by Alex Bennée 1 year, 2 months ago

Richard Henderson <richard.henderson@linaro.org> writes:

> Supercedes: <20230222043204.941336-1-cota@braap.org>
> ("[PATCH] plugin: fix clearing of plugin_mem_cbs before TB exit")
>
> Let's handle this as we do can_do_io, and reset the value in C.

Queued to plugins/next, thanks.

I've manually fixed up the #ifdefs and called
plugin_gen_disable_mem_helpers() directly as Emilio recommended.

>
>
> r~
>
>
> Richard Henderson (2):
>   tcg: Clear plugin_mem_cbs on TB exit
>   include/qemu/plugin: Remove QEMU_PLUGIN_ASSERT
>
>  include/qemu/plugin.h       | 4 ----
>  accel/tcg/cpu-exec-common.c | 4 ++++
>  accel/tcg/cpu-exec.c        | 5 +++--
>  3 files changed, 7 insertions(+), 6 deletions(-)


-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro