All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: qemu-devel@nongnu.org
Cc: qemu-ppc@nongnu.org,
	"Yoshinori Sato" <ysato@users.sourceforge.jp>,
	qemu-arm@nongnu.org, "Liu Zhiwei" <zhiwei_liu@linux.alibaba.com>,
	"Michael Rolnik" <mrolnik@gmail.com>,
	"Yanan Wang" <wangyanan55@huawei.com>,
	"Eduardo Habkost" <eduardo@habkost.net>,
	"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
	"Beraldo Leal" <bleal@redhat.com>,
	"Cleber Rosa" <crosa@redhat.com>,
	"David Hildenbrand" <david@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Song Gao" <gaosong@loongson.cn>,
	"Daniel Henrique Barboza" <danielhb413@gmail.com>,
	"Laurent Vivier" <laurent@vivier.eu>,
	"Warner Losh" <imp@bsdimp.com>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	"Alexandre Iooss" <erdnaxe@crans.org>,
	"Ilya Leoshkevich" <iii@linux.ibm.com>,
	"Cédric Le Goater" <clg@kaod.org>,
	qemu-s390x@nongnu.org, "Kyle Evans" <kevans@freebsd.org>,
	"Brad Smith" <brad@comstyle.com>,
	"Nicholas Piggin" <npiggin@gmail.com>,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
	"Pierrick Bouvier" <pierrick.bouvier@linaro.org>,
	"John Snow" <jsnow@redhat.com>,
	"Alistair Francis" <alistair.francis@wdc.com>,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Brian Cain" <bcain@quicinc.com>,
	"Thomas Huth" <thuth@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>,
	qemu-riscv@nongnu.org, "Bin Meng" <bin.meng@windriver.com>,
	"Daniel Henrique Barboza" <dbarboza@ventanamicro.com>,
	"Weiwei Li" <liwei1518@gmail.com>,
	"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Mahmoud Mandour" <ma.mandourr@gmail.com>
Subject: [PATCH v4 16/29] plugins: add qemu_plugin_num_vcpus function
Date: Tue, 27 Feb 2024 14:43:22 +0000	[thread overview]
Message-ID: <20240227144335.1196131-17-alex.bennee@linaro.org> (raw)
In-Reply-To: <20240227144335.1196131-1-alex.bennee@linaro.org>

From: Pierrick Bouvier <pierrick.bouvier@linaro.org>

We now keep track of how many vcpus were started. This way, a plugin can
easily query number of any vcpus at any point of execution, which
unifies user and system mode workflows.

Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20240213094009.150349-3-pierrick.bouvier@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 include/qemu/qemu-plugin.h   | 3 +++
 plugins/plugin.h             | 4 ++++
 plugins/api.c                | 5 +++++
 plugins/core.c               | 6 ++++++
 plugins/qemu-plugins.symbols | 1 +
 5 files changed, 19 insertions(+)

diff --git a/include/qemu/qemu-plugin.h b/include/qemu/qemu-plugin.h
index e45181c793c..93981f8f89f 100644
--- a/include/qemu/qemu-plugin.h
+++ b/include/qemu/qemu-plugin.h
@@ -645,6 +645,9 @@ QEMU_PLUGIN_API
 void qemu_plugin_register_atexit_cb(qemu_plugin_id_t id,
                                     qemu_plugin_udata_cb_t cb, void *userdata);
 
+/* returns how many vcpus were started at this point */
+int qemu_plugin_num_vcpus(void);
+
 /**
  * qemu_plugin_outs() - output string via QEMU's logging system
  * @string: a string
diff --git a/plugins/plugin.h b/plugins/plugin.h
index 90f3f324ab6..00b3509f708 100644
--- a/plugins/plugin.h
+++ b/plugins/plugin.h
@@ -44,6 +44,8 @@ struct qemu_plugin_state {
      * the code cache is flushed.
      */
     struct qht dyn_cb_arr_ht;
+    /* How many vcpus were started */
+    int num_vcpus;
 };
 
 
@@ -97,4 +99,6 @@ void plugin_register_vcpu_mem_cb(GArray **arr,
 
 void exec_inline_op(struct qemu_plugin_dyn_cb *cb);
 
+int plugin_num_vcpus(void);
+
 #endif /* PLUGIN_H */
diff --git a/plugins/api.c b/plugins/api.c
index 2926b1961a8..116b8bd603c 100644
--- a/plugins/api.c
+++ b/plugins/api.c
@@ -342,6 +342,11 @@ const char *qemu_plugin_hwaddr_device_name(const struct qemu_plugin_hwaddr *h)
 #endif
 }
 
+int qemu_plugin_num_vcpus(void)
+{
+    return plugin_num_vcpus();
+}
+
 /*
  * Plugin output
  */
diff --git a/plugins/core.c b/plugins/core.c
index ee2fa41af9e..caa66311351 100644
--- a/plugins/core.c
+++ b/plugins/core.c
@@ -213,6 +213,7 @@ void qemu_plugin_vcpu_init_hook(CPUState *cpu)
     bool success;
 
     qemu_rec_mutex_lock(&plugin.lock);
+    plugin.num_vcpus = MAX(plugin.num_vcpus, cpu->cpu_index + 1);
     plugin_cpu_update__locked(&cpu->cpu_index, NULL, NULL);
     success = g_hash_table_insert(plugin.cpu_ht, &cpu->cpu_index,
                                   &cpu->cpu_index);
@@ -570,3 +571,8 @@ static void __attribute__((__constructor__)) plugin_init(void)
              QHT_MODE_AUTO_RESIZE);
     atexit(qemu_plugin_atexit_cb);
 }
+
+int plugin_num_vcpus(void)
+{
+    return plugin.num_vcpus;
+}
diff --git a/plugins/qemu-plugins.symbols b/plugins/qemu-plugins.symbols
index ca806000d54..adb67608598 100644
--- a/plugins/qemu-plugins.symbols
+++ b/plugins/qemu-plugins.symbols
@@ -16,6 +16,7 @@
   qemu_plugin_mem_is_sign_extended;
   qemu_plugin_mem_is_store;
   qemu_plugin_mem_size_shift;
+  qemu_plugin_num_vcpus;
   qemu_plugin_outs;
   qemu_plugin_path_to_binary;
   qemu_plugin_register_atexit_cb;
-- 
2.39.2



  parent reply	other threads:[~2024-02-27 14:48 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-27 14:43 [PATCH v4 00/29] maintainer updates for 9.0 pre-PR (tests, plugin register support) Alex Bennée
2024-02-27 14:43 ` [PATCH v4 01/29] tests/tcg: update licenses to GPLv2 as intended Alex Bennée
2024-02-27 14:43 ` [PATCH v4 02/29] tests/tcg: bump TCG test timeout to 120s Alex Bennée
2024-02-27 14:43 ` [PATCH v4 03/29] tests/vm: avoid re-building the VM images all the time Alex Bennée
2024-02-27 17:14   ` Thomas Huth
2024-02-27 14:43 ` [PATCH v4 04/29] tests/vm: update openbsd image to 7.4 Alex Bennée
2024-02-27 17:35   ` Thomas Huth
2024-02-27 14:43 ` [PATCH v4 05/29] target/arm: Use GDBFeature for dynamic XML Alex Bennée
2024-02-27 14:43 ` [PATCH v4 06/29] target/ppc: " Alex Bennée
2024-02-27 14:43 ` [PATCH v4 07/29] target/riscv: " Alex Bennée
2024-02-27 14:43 ` [PATCH v4 08/29] gdbstub: Use GDBFeature for gdb_register_coprocessor Alex Bennée
2024-02-27 14:43 ` [PATCH v4 09/29] gdbstub: Use GDBFeature for GDBRegisterState Alex Bennée
2024-02-27 14:43 ` [PATCH v4 10/29] gdbstub: Change gdb_get_reg_cb and gdb_set_reg_cb Alex Bennée
2024-02-27 14:43 ` [PATCH v4 11/29] gdbstub: Simplify XML lookup Alex Bennée
2024-02-27 14:43 ` [PATCH v4 12/29] gdbstub: Infer number of core registers from XML Alex Bennée
2024-02-27 14:43 ` [PATCH v4 13/29] hw/core/cpu: Remove gdb_get_dynamic_xml member Alex Bennée
2024-02-27 14:43 ` [PATCH v4 14/29] gdbstub: Add members to identify registers to GDBFeature Alex Bennée
2024-02-27 14:43 ` [PATCH v4 15/29] plugins: remove previous n_vcpus functions from API Alex Bennée
2024-02-27 14:43 ` Alex Bennée [this message]
2024-02-27 14:43 ` [PATCH v4 17/29] plugins: fix order of init/idle/resume callback Alex Bennée
2024-02-27 14:43 ` [PATCH v4 18/29] linux-user: ensure nios2 processes queued work Alex Bennée
2024-02-27 14:43 ` [PATCH v4 19/29] cpu: call plugin init hook asynchronously Alex Bennée
2024-02-27 14:43 ` [PATCH v4 20/29] plugins: Use different helpers when reading registers Alex Bennée
2024-02-27 14:43 ` [PATCH v4 21/29] gdbstub: expose api to find registers Alex Bennée
2024-02-28  5:08   ` Akihiko Odaki
2024-02-27 14:43 ` [PATCH v4 22/29] plugins: create CPUPluginState and migrate plugin_mask Alex Bennée
2024-02-27 14:43 ` [PATCH v4 23/29] plugins: add an API to read registers Alex Bennée
2024-02-28  5:08   ` Akihiko Odaki
2024-02-27 14:43 ` [PATCH v4 24/29] tests/tcg: expand insn test case to exercise register API Alex Bennée
2024-02-27 14:43 ` [PATCH v4 25/29] contrib/plugins: fix imatch Alex Bennée
2024-02-27 14:43 ` [PATCH v4 26/29] contrib/plugins: extend execlog to track register changes Alex Bennée
2024-02-27 14:43 ` [PATCH v4 27/29] docs/devel: lift example and plugin API sections up Alex Bennée
2024-02-27 14:43 ` [PATCH v4 28/29] docs/devel: document some plugin assumptions Alex Bennée
2024-02-27 14:43 ` [PATCH v4 29/29] docs/devel: plugins can trigger a tb flush Alex Bennée

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=20240227144335.1196131-17-alex.bennee@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=alistair.francis@wdc.com \
    --cc=bcain@quicinc.com \
    --cc=bin.meng@windriver.com \
    --cc=bleal@redhat.com \
    --cc=brad@comstyle.com \
    --cc=clg@kaod.org \
    --cc=crosa@redhat.com \
    --cc=danielhb413@gmail.com \
    --cc=david@redhat.com \
    --cc=dbarboza@ventanamicro.com \
    --cc=edgar.iglesias@gmail.com \
    --cc=eduardo@habkost.net \
    --cc=erdnaxe@crans.org \
    --cc=gaosong@loongson.cn \
    --cc=iii@linux.ibm.com \
    --cc=imp@bsdimp.com \
    --cc=jsnow@redhat.com \
    --cc=kevans@freebsd.org \
    --cc=laurent@vivier.eu \
    --cc=liwei1518@gmail.com \
    --cc=ma.mandourr@gmail.com \
    --cc=marcel.apfelbaum@gmail.com \
    --cc=mrolnik@gmail.com \
    --cc=npiggin@gmail.com \
    --cc=palmer@dabbelt.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@linaro.org \
    --cc=pierrick.bouvier@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=qemu-riscv@nongnu.org \
    --cc=qemu-s390x@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=thuth@redhat.com \
    --cc=wainersm@redhat.com \
    --cc=wangyanan55@huawei.com \
    --cc=ysato@users.sourceforge.jp \
    --cc=zhiwei_liu@linux.alibaba.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.