QEMU-Devel Archive mirror
 help / color / mirror / Atom feed
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
To: qemu-devel@nongnu.org
Cc: "Mahmoud Mandour" <ma.mandourr@gmail.com>,
	"Alexandre Iooss" <erdnaxe@crans.org>,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Pierrick Bouvier" <pierrick.bouvier@linaro.org>,
	"Alex Bennée" <alex.bennee@linaro.org>
Subject: [PATCH v6 0/9] TCG plugins new inline operations
Date: Thu,  2 May 2024 14:15:13 -0700	[thread overview]
Message-ID: <20240502211522.346467-1-pierrick.bouvier@linaro.org> (raw)

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.

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.henderson@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

v6
--

- more elaborated commit messages

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(-)

-- 
2.39.2



             reply	other threads:[~2024-05-02 21:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-02 21:15 Pierrick Bouvier [this message]
2024-05-02 21:15 ` [PATCH v6 1/9] plugins: prepare introduction of new inline ops Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 2/9] plugins: extract generate ptr for qemu_plugin_u64 Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 3/9] plugins: add new inline op STORE_U64 Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 4/9] tests/plugin/inline: add test for STORE_U64 inline op Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 5/9] plugins: conditional callbacks Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 6/9] tests/plugin/inline: add test for conditional callback Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 7/9] plugins: distinct types for callbacks Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 8/9] plugins: extract cpu_index generate Pierrick Bouvier
2024-05-02 21:15 ` [PATCH v6 9/9] plugins: remove op from qemu_plugin_inline_cb Pierrick Bouvier
2024-05-07 16:07 ` [PATCH v6 0/9] TCG plugins new inline operations Alex Bennée
2024-05-07 16:52   ` Pierrick Bouvier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240502211522.346467-1-pierrick.bouvier@linaro.org \
    --to=pierrick.bouvier@linaro.org \
    --cc=alex.bennee@linaro.org \
    --cc=erdnaxe@crans.org \
    --cc=ma.mandourr@gmail.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).