Contrary to what the cover letter says, all patches have been reviewed
in the series since v4.
On 5/2/24 11:08, Pierrick Bouvier wrote:
This series implement two new operations for plugins:
- Store inline allows to write a specific value to a scoreboard.
- Conditional callback executes a callback only when a given condition is true.
The condition is evaluated inline.
It's possible to mix various inline operations (add, store) with conditional
callbacks, allowing efficient "trap" based counters.
It builds on top of new scoreboard API, introduced in the previous series.
NOTE: Two patches still need review
v2
--
- fixed issue with udata not being passed to conditional callback
- added specific test for this in tests/plugin/inline.c (udata was NULL before).
v3
--
- rebased on top of "plugins: Rewrite plugin code generation":
20240316015720.3661236-1-richard.hender...@linaro.org
- single pass code generation
- small cleanups for code generation
v4
--
- remove op field from qemu_plugin_inline_cb
- use tcg_constant_i64 to load immediate value to store
v5
--
- rebase on top of master now that Richard's series was merged
Pierrick Bouvier (9):
plugins: prepare introduction of new inline ops
plugins: extract generate ptr for qemu_plugin_u64
plugins: add new inline op STORE_U64
tests/plugin/inline: add test for STORE_U64 inline op
plugins: conditional callbacks
tests/plugin/inline: add test for conditional callback
plugins: distinct types for callbacks
plugins: extract cpu_index generate
plugins: remove op from qemu_plugin_inline_cb
include/qemu/plugin.h | 42 +++++++----
include/qemu/qemu-plugin.h | 80 ++++++++++++++++++++-
plugins/plugin.h | 12 +++-
accel/tcg/plugin-gen.c | 136 +++++++++++++++++++++++++++--------
plugins/api.c | 39 ++++++++++
plugins/core.c | 109 ++++++++++++++++++++--------
tests/plugin/inline.c | 130 +++++++++++++++++++++++++++++++--
plugins/qemu-plugins.symbols | 2 +
8 files changed, 466 insertions(+), 84 deletions(-)
