From nobody Tue Feb 10 11:32:44 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1602549086; cv=none; d=zohomail.com; s=zohoarc; b=ccthfxgM2wB1FZ4lV1bvRl8YJ3rDyhyfn5m0PW96sI4L96Wl08SXuf8E2fCHIXrz4/JdYonHKsz3dn0KbCTj+9DkkfzYJMJA5Rhyk+2borh2sBQtRGXmkepiXPuzHqRhHWUhw/OIvoqXe/A1azPbPMoERHVkjOqxLsylm8Los7w= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1602549086; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=njq8LwCRNcFNSShSa2dCop+cPHvQa+V3OA16N3iGgms=; b=ZitjXu+mXKEGkv0W6eFfhPkg3fwDj0OjpHn6dhxqwpZRCdzvxR/uvNQyYYrC4NxRI7Lh2+HHL2Ur8/3ECLLWofRFv8xpB2/rmAYhzH6DzdEHRYYvetqvMs9yxlm3CoSBQX5SU//627WpIqmm3HPff4PD3O2o//klZDfFZ2kvRHA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1602549086381550.2719012524005; Mon, 12 Oct 2020 17:31:26 -0700 (PDT) Received: from localhost ([::1]:44496 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kS8Dt-0001bm-7I for importer@patchew.org; Mon, 12 Oct 2020 20:31:25 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:54856) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kS8B9-00088q-Ni for qemu-devel@nongnu.org; Mon, 12 Oct 2020 20:28:35 -0400 Received: from mail-pl1-x62b.google.com ([2607:f8b0:4864:20::62b]:32910) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kS8B6-0004n0-3V for qemu-devel@nongnu.org; Mon, 12 Oct 2020 20:28:35 -0400 Received: by mail-pl1-x62b.google.com with SMTP id b19so9682087pld.0 for ; Mon, 12 Oct 2020 17:28:31 -0700 (PDT) Received: from localhost.localdomain ([103.94.185.75]) by smtp.googlemail.com with ESMTPSA id y5sm22565925pge.62.2020.10.12.17.28.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 12 Oct 2020 17:28:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=njq8LwCRNcFNSShSa2dCop+cPHvQa+V3OA16N3iGgms=; b=a98uxeOg91xO3hS0pMf1yGSSx/ZSTOUOB20C5DhhPwAdxcIIyybtYfQlafb6J/SjwX RrYM9g316fH7rEcpgzoJF6H0RzBrKXHmlCy7OQoDxkmp9ksjCwQ0k5c1N2ycDy0IaWs3 srZ+jzyFXtdf1Bp2Vg2KJjd77vEBX+PSAv2iFion9T3c0RElZcJpgKFtw9BnnVnJg/pc ondVvasty4WfC5nBbQE+8M4n9u899XdG/gHjPaulkaHP27F2MkeQmBUXWD6i4P5gpw0Q w/0yezFy0GNcD7C3aTV1oKoVWlWAI7q8uMSVwoL+UBoQvxrPY9GxyehrwBWpbsBM0CZB 4YFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=njq8LwCRNcFNSShSa2dCop+cPHvQa+V3OA16N3iGgms=; b=I1JHbKaHl8EXTNWPmUa6nj5WJVxxHQm9hnk+kWSZo0XoKYkdIHwfBCioFqi69BHgQq CYSTgE4Ux7vtHA8a1zNMhPPYf5LZSMoOWPnX9EsbfiTXpcWvLfDPejWQj3Fi1xoLBE5C IqPIAaNhtKznrbsLum3JtKCAV1iDEzMoMHSqzXaUCo6Wr6wDNEcZWAe1ecM19cPDC8At V4OUXp3Y/Zbp0+KiogjvrbZ/l5Beu6UERoR7bLZIWO7bbFybP+ffVwwLhoScclDos1jO vhYeNVpJ5t290tCqyZVXj7oymh+UfUfOAXpZ3vjZsIK8piXV/LVhDt4LRP9gZ39PPM+Z /Gyg== X-Gm-Message-State: AOAM533bfOUiAvHVMR2kEpVSC6dtfl/4xCIBmKb/hLdcpTIN77jX824p 8qAaf9Xt5nQw6GIADwcfzDU2DNdQgsiNKg== X-Google-Smtp-Source: ABdhPJydNAB9G+qbBJYRlrWWCRK5B7AtyOyZJKizm4HaJJPcJ+g82vQt51QjaL0D1OGdzTNVFMulhw== X-Received: by 2002:a17:902:fe86:b029:d4:d451:eb56 with SMTP id x6-20020a170902fe86b02900d4d451eb56mr9774423plm.79.1602548909794; Mon, 12 Oct 2020 17:28:29 -0700 (PDT) From: Yonggang Luo To: qemu-devel@nongnu.org Subject: [PATCH v6 3/5] plugins: Getting qemu-plugin works under win32. Date: Tue, 13 Oct 2020 08:28:04 +0800 Message-Id: <20201013002806.1447-4-luoyonggang@gmail.com> X-Mailer: git-send-email 2.28.0.windows.1 In-Reply-To: <20201013002806.1447-1-luoyonggang@gmail.com> References: <20201013002806.1447-1-luoyonggang@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2607:f8b0:4864:20::62b; envelope-from=luoyonggang@gmail.com; helo=mail-pl1-x62b.google.com X-detected-operating-system: by eggs.gnu.org: No matching host in p0f cache. That's all we know. X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: =?UTF-8?q?Alex=20Benn=C3=A9e?= , Paolo Bonzini , Yonggang Luo , Richard Henderson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Define QEMU_PLUGIN_API_IMPLEMENTATION in api.c and core.c We removed the need of .symbols file, so is the configure script, if we one expose a function to qemu-plugin just need prefix the function with QEMU_PLUGIN_EXPORT We use QEMU_PLUGIN_EXPORT export the functions in api.c and core.c and loading it in qemu_plugin_initialize, so we need a list of function poi= nter deceleration such as QEMU_PLUGIN_EXTERN qemu_plugin_uninstall_t qemu_plugin_uninstall; Signed-off-by: Yonggang Luo --- Makefile | 1 - configure | 71 ------------- include/qemu/qemu-plugin.h | 198 +++++++++++++++++++++++++++-------- meson.build | 6 +- plugins/api.c | 105 +++++++++++-------- plugins/core.c | 15 ++- plugins/loader.c | 50 ++++++++- plugins/meson.build | 10 +- plugins/plugin.h | 1 + plugins/qemu-plugins.symbols | 40 ------- 10 files changed, 277 insertions(+), 220 deletions(-) delete mode 100644 plugins/qemu-plugins.symbols diff --git a/Makefile b/Makefile index c37e513431..cd6079bb78 100644 --- a/Makefile +++ b/Makefile @@ -105,7 +105,6 @@ config-host.mak: $(SRC_PATH)/configure $(SRC_PATH)/pc-b= ios $(SRC_PATH)/VERSION =20 # Force configure to re-run if the API symbols are updated ifeq ($(CONFIG_PLUGIN),y) -config-host.mak: $(SRC_PATH)/plugins/qemu-plugins.symbols =20 .PHONY: plugins plugins: diff --git a/configure b/configure index 0829b7d3b4..f26212813e 100755 --- a/configure +++ b/configure @@ -5201,61 +5201,6 @@ if compile_prog "" "" ; then atomic64=3Dyes fi =20 -######################################### -# See if --dynamic-list is supported by the linker -ld_dynamic_list=3D"no" -if test "$static" =3D "no" ; then - cat > $TMPTXT < $TMPC < -void foo(void); - -void foo(void) -{ - printf("foo\n"); -} - -int main(void) -{ - foo(); - return 0; -} -EOF - - if compile_prog "" "-Wl,--dynamic-list=3D$TMPTXT" ; then - ld_dynamic_list=3D"yes" - fi -fi - -######################################### -# See if -exported_symbols_list is supported by the linker - -ld_exported_symbols_list=3D"no" -if test "$static" =3D "no" ; then - cat > $TMPTXT <> $config_host_mak - # Copy the export object list to the build dir - if test "$ld_dynamic_list" =3D "yes" ; then - echo "CONFIG_HAS_LD_DYNAMIC_LIST=3Dyes" >> $config_host_mak - ld_symbols=3Dqemu-plugins-ld.symbols - cp "$source_path/plugins/qemu-plugins.symbols" $ld_symbols - elif test "$ld_exported_symbols_list" =3D "yes" ; then - echo "CONFIG_HAS_LD_EXPORTED_SYMBOLS_LIST=3Dyes" >> $config_host_mak - ld64_symbols=3Dqemu-plugins-ld64.symbols - echo "# Automatically generated by configure - do not modify" > $ld64_sym= bols - grep 'qemu_' "$source_path/plugins/qemu-plugins.symbols" | sed 's/;//g' |= \ - sed -E 's/^[[:space:]]*(.*)/_\1/' >> $ld64_symbols - else - error_exit \ - "If \$plugins=3Dyes, either \$ld_dynamic_list or " \ - "\$ld_exported_symbols_list should have been set to 'yes'." - fi fi =20 if test -n "$gdb_bin" ; then diff --git a/include/qemu/qemu-plugin.h b/include/qemu/qemu-plugin.h index daac8291b8..af2f6ccb78 100644 --- a/include/qemu/qemu-plugin.h +++ b/include/qemu/qemu-plugin.h @@ -21,11 +21,7 @@ * https://gcc.gnu.org/wiki/Visibility */ #if defined _WIN32 || defined __CYGWIN__ - #ifdef BUILDING_DLL - #define QEMU_PLUGIN_EXPORT __declspec(dllexport) - #else - #define QEMU_PLUGIN_EXPORT __declspec(dllimport) - #endif + #define QEMU_PLUGIN_EXPORT __declspec(dllexport) #define QEMU_PLUGIN_LOCAL #else #if __GNUC__ >=3D 4 @@ -52,7 +48,9 @@ typedef uint64_t qemu_plugin_id_t; =20 extern QEMU_PLUGIN_EXPORT int qemu_plugin_version; =20 -#define QEMU_PLUGIN_VERSION 0 +#define QEMU_PLUGIN_VERSION 1 + +typedef void *(*qemu_plugin_global_dlsym_t)(void *context, const char *nam= e); =20 typedef struct { /* string describing architecture */ @@ -73,8 +71,23 @@ typedef struct { int max_vcpus; } system; }; + void *context; + qemu_plugin_global_dlsym_t dlsym; } qemu_info_t; =20 +/** + * qemu_plugin_initialize() - Initialize a plugin before install + * @info: a block describing some details about the guest + * + * All plugins must export this symbol, and in most case using qemu-plugin= .h + * provided implementation directly. + * For plugin provide this function, the QEMU_PLUGIN_VERSION should >=3D 1 + * + * Note: This function only used to loading qemu's exported functions, not= hing + * else should doding in this function. + */ +QEMU_PLUGIN_EXPORT int qemu_plugin_initialize(const qemu_info_t *info); + /** * qemu_plugin_install() - Install a plugin * @id: this plugin's opaque ID @@ -121,7 +134,7 @@ typedef void (*qemu_plugin_vcpu_udata_cb_t)(unsigned in= t vcpu_index, * * Note: Calling this function from qemu_plugin_install() is a bug. */ -void qemu_plugin_uninstall(qemu_plugin_id_t id, qemu_plugin_simple_cb_t cb= ); +typedef void (*qemu_plugin_uninstall_t)(qemu_plugin_id_t id, qemu_plugin_s= imple_cb_t cb); =20 /** * qemu_plugin_reset() - Reset a plugin @@ -134,7 +147,7 @@ void qemu_plugin_uninstall(qemu_plugin_id_t id, qemu_pl= ugin_simple_cb_t cb); * Plugins are reset asynchronously, and therefore the given plugin receiv= es * callbacks until @cb is called. */ -void qemu_plugin_reset(qemu_plugin_id_t id, qemu_plugin_simple_cb_t cb); +typedef void (*qemu_plugin_reset_t)(qemu_plugin_id_t id, qemu_plugin_simpl= e_cb_t cb); =20 /** * qemu_plugin_register_vcpu_init_cb() - register a vCPU initialization ca= llback @@ -145,7 +158,7 @@ void qemu_plugin_reset(qemu_plugin_id_t id, qemu_plugin= _simple_cb_t cb); * * See also: qemu_plugin_register_vcpu_exit_cb() */ -void qemu_plugin_register_vcpu_init_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_vcpu_init_cb_t)(qemu_plugin_id_t id, qemu_plugin_vcpu_simple_cb_t cb); =20 /** @@ -157,7 +170,7 @@ void qemu_plugin_register_vcpu_init_cb(qemu_plugin_id_t= id, * * See also: qemu_plugin_register_vcpu_init_cb() */ -void qemu_plugin_register_vcpu_exit_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_vcpu_exit_cb_t)(qemu_plugin_id_t id, qemu_plugin_vcpu_simple_cb_t cb); =20 /** @@ -167,7 +180,7 @@ void qemu_plugin_register_vcpu_exit_cb(qemu_plugin_id_t= id, * * The @cb function is called every time a vCPU idles. */ -void qemu_plugin_register_vcpu_idle_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_vcpu_idle_cb_t)(qemu_plugin_id_t id, qemu_plugin_vcpu_simple_cb_t cb); =20 /** @@ -177,7 +190,7 @@ void qemu_plugin_register_vcpu_idle_cb(qemu_plugin_id_t= id, * * The @cb function is called every time a vCPU resumes execution. */ -void qemu_plugin_register_vcpu_resume_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_vcpu_resume_cb_t)(qemu_plugin_id_t id, qemu_plugin_vcpu_simple_cb_t cb); =20 /* @@ -214,7 +227,7 @@ enum qemu_plugin_mem_rw { typedef void (*qemu_plugin_vcpu_tb_trans_cb_t)(qemu_plugin_id_t id, struct qemu_plugin_tb *tb); =20 -void qemu_plugin_register_vcpu_tb_trans_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_vcpu_tb_trans_cb_t)(qemu_plugin_id_t i= d, qemu_plugin_vcpu_tb_trans_cb_t = cb); =20 /** @@ -226,7 +239,7 @@ void qemu_plugin_register_vcpu_tb_trans_cb(qemu_plugin_= id_t id, * * The @cb function is called every time a translated unit executes. */ -void qemu_plugin_register_vcpu_tb_exec_cb(struct qemu_plugin_tb *tb, +typedef void (*qemu_plugin_register_vcpu_tb_exec_cb_t)(struct qemu_plugin_= tb *tb, qemu_plugin_vcpu_udata_cb_t cb, enum qemu_plugin_cb_flags flags, void *userdata); @@ -246,7 +259,7 @@ enum qemu_plugin_op { * Useful if you just want to increment a single counter somewhere in * memory. */ -void qemu_plugin_register_vcpu_tb_exec_inline(struct qemu_plugin_tb *tb, +typedef void (*qemu_plugin_register_vcpu_tb_exec_inline_t)(struct qemu_plu= gin_tb *tb, enum qemu_plugin_op op, void *ptr, uint64_t imm); =20 @@ -259,7 +272,7 @@ void qemu_plugin_register_vcpu_tb_exec_inline(struct qe= mu_plugin_tb *tb, * * The @cb function is called every time an instruction is executed */ -void qemu_plugin_register_vcpu_insn_exec_cb(struct qemu_plugin_insn *insn, +typedef void (*qemu_plugin_register_vcpu_insn_exec_cb_t)(struct qemu_plugi= n_insn *insn, qemu_plugin_vcpu_udata_cb_t cb, enum qemu_plugin_cb_flags flag= s, void *userdata); @@ -275,26 +288,26 @@ void qemu_plugin_register_vcpu_insn_exec_cb(struct qe= mu_plugin_insn *insn, * Insert an inline op to every time an instruction executes. Useful * if you just want to increment a single counter somewhere in memory. */ -void qemu_plugin_register_vcpu_insn_exec_inline(struct qemu_plugin_insn *i= nsn, - enum qemu_plugin_op op, - void *ptr, uint64_t imm); +typedef void (*qemu_plugin_register_vcpu_insn_exec_inline_t)( + struct qemu_plugin_insn *insn, enum qemu_plugin_op op, + void *ptr, uint64_t imm); =20 /* * Helpers to query information about the instructions in a block */ -size_t qemu_plugin_tb_n_insns(const struct qemu_plugin_tb *tb); +typedef size_t (*qemu_plugin_tb_n_insns_t)(const struct qemu_plugin_tb *tb= ); =20 -uint64_t qemu_plugin_tb_vaddr(const struct qemu_plugin_tb *tb); +typedef uint64_t (*qemu_plugin_tb_vaddr_t)(const struct qemu_plugin_tb *tb= ); =20 -struct qemu_plugin_insn * -qemu_plugin_tb_get_insn(const struct qemu_plugin_tb *tb, size_t idx); +typedef struct qemu_plugin_insn * +(*qemu_plugin_tb_get_insn_t)(const struct qemu_plugin_tb *tb, size_t idx); =20 -const void *qemu_plugin_insn_data(const struct qemu_plugin_insn *insn); +typedef const void *(*qemu_plugin_insn_data_t)(const struct qemu_plugin_in= sn *insn); =20 -size_t qemu_plugin_insn_size(const struct qemu_plugin_insn *insn); +typedef size_t (*qemu_plugin_insn_size_t)(const struct qemu_plugin_insn *i= nsn); =20 -uint64_t qemu_plugin_insn_vaddr(const struct qemu_plugin_insn *insn); -void *qemu_plugin_insn_haddr(const struct qemu_plugin_insn *insn); +typedef uint64_t (*qemu_plugin_insn_vaddr_t)(const struct qemu_plugin_insn= *insn); +typedef void *(*qemu_plugin_insn_haddr_t)(const struct qemu_plugin_insn *i= nsn); =20 /* * Memory Instrumentation @@ -307,10 +320,10 @@ typedef uint32_t qemu_plugin_meminfo_t; struct qemu_plugin_hwaddr; =20 /* meminfo queries */ -unsigned int qemu_plugin_mem_size_shift(qemu_plugin_meminfo_t info); -bool qemu_plugin_mem_is_sign_extended(qemu_plugin_meminfo_t info); -bool qemu_plugin_mem_is_big_endian(qemu_plugin_meminfo_t info); -bool qemu_plugin_mem_is_store(qemu_plugin_meminfo_t info); +typedef unsigned int (*qemu_plugin_mem_size_shift_t)(qemu_plugin_meminfo_t= info); +typedef bool (*qemu_plugin_mem_is_sign_extended_t)(qemu_plugin_meminfo_t i= nfo); +typedef bool (*qemu_plugin_mem_is_big_endian_t)(qemu_plugin_meminfo_t info= ); +typedef bool (*qemu_plugin_mem_is_store_t)(qemu_plugin_meminfo_t info); =20 /* * qemu_plugin_get_hwaddr(): @@ -324,7 +337,7 @@ bool qemu_plugin_mem_is_store(qemu_plugin_meminfo_t inf= o); * information about the handle should be recovered before the * callback returns. */ -struct qemu_plugin_hwaddr *qemu_plugin_get_hwaddr(qemu_plugin_meminfo_t in= fo, +typedef struct qemu_plugin_hwaddr *(*qemu_plugin_get_hwaddr_t)(qemu_plugin= _meminfo_t info, uint64_t vaddr); =20 /* @@ -332,21 +345,22 @@ struct qemu_plugin_hwaddr *qemu_plugin_get_hwaddr(qem= u_plugin_meminfo_t info, * to return information about it. For non-IO accesses the device * offset will be into the appropriate block of RAM. */ -bool qemu_plugin_hwaddr_is_io(const struct qemu_plugin_hwaddr *haddr); -uint64_t qemu_plugin_hwaddr_device_offset(const struct qemu_plugin_hwaddr = *haddr); +typedef bool (*qemu_plugin_hwaddr_is_io_t)(const struct qemu_plugin_hwaddr= *haddr); +typedef uint64_t (*qemu_plugin_hwaddr_device_offset_t)( + const struct qemu_plugin_hwaddr *haddr); =20 typedef void (*qemu_plugin_vcpu_mem_cb_t)(unsigned int vcpu_index, qemu_plugin_meminfo_t info, uint64_t vaddr, void *userdata); =20 -void qemu_plugin_register_vcpu_mem_cb(struct qemu_plugin_insn *insn, +typedef void (*qemu_plugin_register_vcpu_mem_cb_t)(struct qemu_plugin_insn= *insn, qemu_plugin_vcpu_mem_cb_t cb, enum qemu_plugin_cb_flags flags, enum qemu_plugin_mem_rw rw, void *userdata); =20 -void qemu_plugin_register_vcpu_mem_inline(struct qemu_plugin_insn *insn, +typedef void (*qemu_plugin_register_vcpu_mem_inline_t)(struct qemu_plugin_= insn *insn, enum qemu_plugin_mem_rw rw, enum qemu_plugin_op op, void *pt= r, uint64_t imm); @@ -359,15 +373,15 @@ typedef void uint64_t a3, uint64_t a4, uint64_t a5, uint64_t a6, uint64_t a7, uint64_t a8); =20 -void qemu_plugin_register_vcpu_syscall_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_vcpu_syscall_cb_t)(qemu_plugin_id_t id, qemu_plugin_vcpu_syscall_cb_t cb= ); =20 typedef void (*qemu_plugin_vcpu_syscall_ret_cb_t)(qemu_plugin_id_t id, unsigned int vcp= u_idx, int64_t num, int64_t ret); =20 -void -qemu_plugin_register_vcpu_syscall_ret_cb(qemu_plugin_id_t id, +typedef void +(*qemu_plugin_register_vcpu_syscall_ret_cb_t)(qemu_plugin_id_t id, qemu_plugin_vcpu_syscall_ret_cb_t= cb); =20 =20 @@ -378,7 +392,7 @@ qemu_plugin_register_vcpu_syscall_ret_cb(qemu_plugin_id= _t id, * Returns an allocated string containing the disassembly */ =20 -char *qemu_plugin_insn_disas(const struct qemu_plugin_insn *insn); +typedef char *(*qemu_plugin_insn_disas_t)(const struct qemu_plugin_insn *i= nsn); =20 /** * qemu_plugin_vcpu_for_each() - iterate over the existing vCPU @@ -389,25 +403,117 @@ char *qemu_plugin_insn_disas(const struct qemu_plugi= n_insn *insn); * * See also: qemu_plugin_register_vcpu_init_cb() */ -void qemu_plugin_vcpu_for_each(qemu_plugin_id_t id, +typedef void (*qemu_plugin_vcpu_for_each_t)(qemu_plugin_id_t id, qemu_plugin_vcpu_simple_cb_t cb); =20 -void qemu_plugin_register_flush_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_flush_cb_t)(qemu_plugin_id_t id, qemu_plugin_simple_cb_t cb); =20 -void qemu_plugin_register_atexit_cb(qemu_plugin_id_t id, +typedef void (*qemu_plugin_register_atexit_cb_t)(qemu_plugin_id_t id, qemu_plugin_udata_cb_t cb, void *userd= ata); =20 /* returns -1 in user-mode */ -int qemu_plugin_n_vcpus(void); +typedef int (*qemu_plugin_n_vcpus_t)(void); =20 /* returns -1 in user-mode */ -int qemu_plugin_n_max_vcpus(void); +typedef int (*qemu_plugin_n_max_vcpus_t)(void); =20 /** * qemu_plugin_outs() - output string via QEMU's logging system * @string: a string */ -void qemu_plugin_outs(const char *string); +typedef void (*qemu_plugin_outs_t)(const char *string); + +#if !defined(QEMU_PLUGIN_API_IMPLEMENTATION) + +#define qemu_plugin_decl_symbol(symbol_name) symbol_name##_t symbol_name + +#define qemu_plugin_load_symbol(info, symbol_name) do {\ + symbol_name =3D info->dlsym(info->context, #symbol_name); \ + if (NULL =3D=3D symbol_name) { \ + return -1; \ + } \ +} while (0) + +qemu_plugin_decl_symbol(qemu_plugin_uninstall); +qemu_plugin_decl_symbol(qemu_plugin_reset); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_init_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_exit_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_idle_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_resume_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_tb_trans_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_tb_exec_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_tb_exec_inline); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_insn_exec_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_insn_exec_inline); +qemu_plugin_decl_symbol(qemu_plugin_tb_n_insns); +qemu_plugin_decl_symbol(qemu_plugin_tb_vaddr); +qemu_plugin_decl_symbol(qemu_plugin_tb_get_insn); +qemu_plugin_decl_symbol(qemu_plugin_insn_data); +qemu_plugin_decl_symbol(qemu_plugin_insn_size); +qemu_plugin_decl_symbol(qemu_plugin_insn_vaddr); +qemu_plugin_decl_symbol(qemu_plugin_insn_haddr); +qemu_plugin_decl_symbol(qemu_plugin_mem_size_shift); +qemu_plugin_decl_symbol(qemu_plugin_mem_is_sign_extended); +qemu_plugin_decl_symbol(qemu_plugin_mem_is_big_endian); +qemu_plugin_decl_symbol(qemu_plugin_mem_is_store); +qemu_plugin_decl_symbol(qemu_plugin_get_hwaddr); +qemu_plugin_decl_symbol(qemu_plugin_hwaddr_is_io); +qemu_plugin_decl_symbol(qemu_plugin_hwaddr_device_offset); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_mem_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_mem_inline); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_syscall_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_vcpu_syscall_ret_cb); +qemu_plugin_decl_symbol(qemu_plugin_insn_disas); +qemu_plugin_decl_symbol(qemu_plugin_vcpu_for_each); +qemu_plugin_decl_symbol(qemu_plugin_register_flush_cb); +qemu_plugin_decl_symbol(qemu_plugin_register_atexit_cb); +qemu_plugin_decl_symbol(qemu_plugin_n_vcpus); +qemu_plugin_decl_symbol(qemu_plugin_n_max_vcpus); +qemu_plugin_decl_symbol(qemu_plugin_outs); + +QEMU_PLUGIN_EXPORT int qemu_plugin_initialize(const qemu_info_t *info) +{ + qemu_plugin_load_symbol(info, qemu_plugin_uninstall); + qemu_plugin_load_symbol(info, qemu_plugin_reset); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_init_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_exit_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_idle_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_resume_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_tb_trans_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_tb_exec_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_tb_exec_inline= ); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_insn_exec_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_insn_exec_inli= ne); + qemu_plugin_load_symbol(info, qemu_plugin_tb_n_insns); + qemu_plugin_load_symbol(info, qemu_plugin_tb_vaddr); + qemu_plugin_load_symbol(info, qemu_plugin_tb_get_insn); + qemu_plugin_load_symbol(info, qemu_plugin_insn_data); + qemu_plugin_load_symbol(info, qemu_plugin_insn_size); + qemu_plugin_load_symbol(info, qemu_plugin_insn_vaddr); + qemu_plugin_load_symbol(info, qemu_plugin_insn_haddr); + qemu_plugin_load_symbol(info, qemu_plugin_mem_size_shift); + qemu_plugin_load_symbol(info, qemu_plugin_mem_is_sign_extended); + qemu_plugin_load_symbol(info, qemu_plugin_mem_is_big_endian); + qemu_plugin_load_symbol(info, qemu_plugin_mem_is_store); + qemu_plugin_load_symbol(info, qemu_plugin_get_hwaddr); + qemu_plugin_load_symbol(info, qemu_plugin_hwaddr_is_io); + qemu_plugin_load_symbol(info, qemu_plugin_hwaddr_device_offset); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_mem_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_mem_inline); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_syscall_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_vcpu_syscall_ret_cb= ); + qemu_plugin_load_symbol(info, qemu_plugin_insn_disas); + qemu_plugin_load_symbol(info, qemu_plugin_vcpu_for_each); + qemu_plugin_load_symbol(info, qemu_plugin_register_flush_cb); + qemu_plugin_load_symbol(info, qemu_plugin_register_atexit_cb); + qemu_plugin_load_symbol(info, qemu_plugin_n_vcpus); + qemu_plugin_load_symbol(info, qemu_plugin_n_max_vcpus); + qemu_plugin_load_symbol(info, qemu_plugin_outs); + return 0; +} + +#endif /* QEMU_PLUGIN_API_IMPLEMENTATION */ + =20 #endif /* QEMU_PLUGIN_API_H */ diff --git a/meson.build b/meson.build index 8ad600a91a..4c073f5460 100644 --- a/meson.build +++ b/meson.build @@ -326,7 +326,11 @@ endif rt =3D cc.find_library('rt', required: false) libdl =3D not_found if 'CONFIG_PLUGIN' in config_host - libdl =3D cc.find_library('dl', required: true) + if targetos =3D=3D 'windows' + libdl =3D declare_dependency(compile_args: [], link_args: []) + else + libdl =3D cc.find_library('dl', required: true) + endif endif libiscsi =3D not_found if 'CONFIG_LIBISCSI' in config_host diff --git a/plugins/api.c b/plugins/api.c index 13177d3578..9964b8bdc7 100644 --- a/plugins/api.c +++ b/plugins/api.c @@ -35,6 +35,7 @@ */ =20 #include "qemu/osdep.h" +#define QEMU_PLUGIN_API_IMPLEMENTATION #include "qemu/plugin.h" #include "cpu.h" #include "sysemu/sysemu.h" @@ -48,14 +49,18 @@ #endif #include "trace/mem.h" =20 +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wmissing-prototypes" + /* Uninstall and Reset handlers */ =20 -void qemu_plugin_uninstall(qemu_plugin_id_t id, qemu_plugin_simple_cb_t cb) +QEMU_PLUGIN_EXPORT void qemu_plugin_uninstall( + qemu_plugin_id_t id, qemu_plugin_simple_cb_t cb) { plugin_reset_uninstall(id, cb, false); } =20 -void qemu_plugin_reset(qemu_plugin_id_t id, qemu_plugin_simple_cb_t cb) +QEMU_PLUGIN_EXPORT void qemu_plugin_reset(qemu_plugin_id_t id, qemu_plugin= _simple_cb_t cb) { plugin_reset_uninstall(id, cb, true); } @@ -67,19 +72,19 @@ void qemu_plugin_reset(qemu_plugin_id_t id, qemu_plugin= _simple_cb_t cb) * during the translation. */ =20 -void qemu_plugin_register_vcpu_init_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_init_cb(qemu_plugin_id_t= id, qemu_plugin_vcpu_simple_cb_t cb) { plugin_register_cb(id, QEMU_PLUGIN_EV_VCPU_INIT, cb); } =20 -void qemu_plugin_register_vcpu_exit_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_exit_cb(qemu_plugin_id_t= id, qemu_plugin_vcpu_simple_cb_t cb) { plugin_register_cb(id, QEMU_PLUGIN_EV_VCPU_EXIT, cb); } =20 -void qemu_plugin_register_vcpu_tb_exec_cb(struct qemu_plugin_tb *tb, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_tb_exec_cb(struct qemu_p= lugin_tb *tb, qemu_plugin_vcpu_udata_cb_t cb, enum qemu_plugin_cb_flags flags, void *udata) @@ -88,25 +93,28 @@ void qemu_plugin_register_vcpu_tb_exec_cb(struct qemu_p= lugin_tb *tb, cb, flags, udata); } =20 -void qemu_plugin_register_vcpu_tb_exec_inline(struct qemu_plugin_tb *tb, - enum qemu_plugin_op op, - void *ptr, uint64_t imm) +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_tb_exec_inline( + struct qemu_plugin_tb *tb, + enum qemu_plugin_op op, + void *ptr, uint64_t imm) { plugin_register_inline_op(&tb->cbs[PLUGIN_CB_INLINE], 0, op, ptr, imm); } =20 -void qemu_plugin_register_vcpu_insn_exec_cb(struct qemu_plugin_insn *insn, - qemu_plugin_vcpu_udata_cb_t cb, - enum qemu_plugin_cb_flags flag= s, - void *udata) +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_insn_exec_cb( + struct qemu_plugin_insn *insn, + qemu_plugin_vcpu_udata_cb_t cb, + enum qemu_plugin_cb_flags flags, + void *udata) { plugin_register_dyn_cb__udata(&insn->cbs[PLUGIN_CB_INSN][PLUGIN_CB_REG= ULAR], cb, flags, udata); } =20 -void qemu_plugin_register_vcpu_insn_exec_inline(struct qemu_plugin_insn *i= nsn, - enum qemu_plugin_op op, - void *ptr, uint64_t imm) +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_insn_exec_inline( + struct qemu_plugin_insn *insn, + enum qemu_plugin_op op, + void *ptr, uint64_t imm) { plugin_register_inline_op(&insn->cbs[PLUGIN_CB_INSN][PLUGIN_CB_INLINE], 0, op, ptr, imm); @@ -114,38 +122,40 @@ void qemu_plugin_register_vcpu_insn_exec_inline(struc= t qemu_plugin_insn *insn, =20 =20 =20 -void qemu_plugin_register_vcpu_mem_cb(struct qemu_plugin_insn *insn, - qemu_plugin_vcpu_mem_cb_t cb, - enum qemu_plugin_cb_flags flags, - enum qemu_plugin_mem_rw rw, - void *udata) +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_mem_cb( + struct qemu_plugin_insn *insn, + qemu_plugin_vcpu_mem_cb_t cb, + enum qemu_plugin_cb_flags flags, + enum qemu_plugin_mem_rw rw, + void *udata) { plugin_register_vcpu_mem_cb(&insn->cbs[PLUGIN_CB_MEM][PLUGIN_CB_REGULA= R], cb, flags, rw, udata); } =20 -void qemu_plugin_register_vcpu_mem_inline(struct qemu_plugin_insn *insn, - enum qemu_plugin_mem_rw rw, - enum qemu_plugin_op op, void *pt= r, - uint64_t imm) +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_mem_inline( + struct qemu_plugin_insn *insn, + enum qemu_plugin_mem_rw rw, + enum qemu_plugin_op op, void *ptr, + uint64_t imm) { plugin_register_inline_op(&insn->cbs[PLUGIN_CB_MEM][PLUGIN_CB_INLINE], rw, op, ptr, imm); } =20 -void qemu_plugin_register_vcpu_tb_trans_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_tb_trans_cb(qemu_plugin_= id_t id, qemu_plugin_vcpu_tb_trans_cb_t = cb) { plugin_register_cb(id, QEMU_PLUGIN_EV_VCPU_TB_TRANS, cb); } =20 -void qemu_plugin_register_vcpu_syscall_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_syscall_cb(qemu_plugin_i= d_t id, qemu_plugin_vcpu_syscall_cb_t cb) { plugin_register_cb(id, QEMU_PLUGIN_EV_VCPU_SYSCALL, cb); } =20 -void +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_syscall_ret_cb(qemu_plugin_id_t id, qemu_plugin_vcpu_syscall_ret_cb_t= cb) { @@ -168,17 +178,17 @@ qemu_plugin_register_vcpu_syscall_ret_cb(qemu_plugin_= id_t id, * each translated instruction. */ =20 -size_t qemu_plugin_tb_n_insns(const struct qemu_plugin_tb *tb) +QEMU_PLUGIN_EXPORT size_t qemu_plugin_tb_n_insns(const struct qemu_plugin_= tb *tb) { return tb->n; } =20 -uint64_t qemu_plugin_tb_vaddr(const struct qemu_plugin_tb *tb) +QEMU_PLUGIN_EXPORT uint64_t qemu_plugin_tb_vaddr(const struct qemu_plugin_= tb *tb) { return tb->vaddr; } =20 -struct qemu_plugin_insn * +QEMU_PLUGIN_EXPORT struct qemu_plugin_insn * qemu_plugin_tb_get_insn(const struct qemu_plugin_tb *tb, size_t idx) { if (unlikely(idx >=3D tb->n)) { @@ -194,27 +204,27 @@ qemu_plugin_tb_get_insn(const struct qemu_plugin_tb *= tb, size_t idx) * instruction being translated. */ =20 -const void *qemu_plugin_insn_data(const struct qemu_plugin_insn *insn) +QEMU_PLUGIN_EXPORT const void *qemu_plugin_insn_data(const struct qemu_plu= gin_insn *insn) { return insn->data->data; } =20 -size_t qemu_plugin_insn_size(const struct qemu_plugin_insn *insn) +QEMU_PLUGIN_EXPORT size_t qemu_plugin_insn_size(const struct qemu_plugin_i= nsn *insn) { return insn->data->len; } =20 -uint64_t qemu_plugin_insn_vaddr(const struct qemu_plugin_insn *insn) +QEMU_PLUGIN_EXPORT uint64_t qemu_plugin_insn_vaddr(const struct qemu_plugi= n_insn *insn) { return insn->vaddr; } =20 -void *qemu_plugin_insn_haddr(const struct qemu_plugin_insn *insn) +QEMU_PLUGIN_EXPORT void *qemu_plugin_insn_haddr(const struct qemu_plugin_i= nsn *insn) { return insn->haddr; } =20 -char *qemu_plugin_insn_disas(const struct qemu_plugin_insn *insn) +QEMU_PLUGIN_EXPORT char *qemu_plugin_insn_disas(const struct qemu_plugin_i= nsn *insn) { CPUState *cpu =3D current_cpu; return plugin_disas(cpu, insn->vaddr, insn->data->len); @@ -225,22 +235,22 @@ char *qemu_plugin_insn_disas(const struct qemu_plugin= _insn *insn) * memory access. */ =20 -unsigned qemu_plugin_mem_size_shift(qemu_plugin_meminfo_t info) +QEMU_PLUGIN_EXPORT unsigned qemu_plugin_mem_size_shift(qemu_plugin_meminfo= _t info) { return info & TRACE_MEM_SZ_SHIFT_MASK; } =20 -bool qemu_plugin_mem_is_sign_extended(qemu_plugin_meminfo_t info) +QEMU_PLUGIN_EXPORT bool qemu_plugin_mem_is_sign_extended(qemu_plugin_memin= fo_t info) { return !!(info & TRACE_MEM_SE); } =20 -bool qemu_plugin_mem_is_big_endian(qemu_plugin_meminfo_t info) +QEMU_PLUGIN_EXPORT bool qemu_plugin_mem_is_big_endian(qemu_plugin_meminfo_= t info) { return !!(info & TRACE_MEM_BE); } =20 -bool qemu_plugin_mem_is_store(qemu_plugin_meminfo_t info) +QEMU_PLUGIN_EXPORT bool qemu_plugin_mem_is_store(qemu_plugin_meminfo_t inf= o) { return !!(info & TRACE_MEM_ST); } @@ -253,8 +263,8 @@ bool qemu_plugin_mem_is_store(qemu_plugin_meminfo_t inf= o) static __thread struct qemu_plugin_hwaddr hwaddr_info; #endif =20 -struct qemu_plugin_hwaddr *qemu_plugin_get_hwaddr(qemu_plugin_meminfo_t in= fo, - uint64_t vaddr) +QEMU_PLUGIN_EXPORT struct qemu_plugin_hwaddr *qemu_plugin_get_hwaddr( + qemu_plugin_meminfo_t info, uint64_t vaddr) { #ifdef CONFIG_SOFTMMU CPUState *cpu =3D current_cpu; @@ -273,7 +283,7 @@ struct qemu_plugin_hwaddr *qemu_plugin_get_hwaddr(qemu_= plugin_meminfo_t info, #endif } =20 -bool qemu_plugin_hwaddr_is_io(const struct qemu_plugin_hwaddr *haddr) +QEMU_PLUGIN_EXPORT bool qemu_plugin_hwaddr_is_io(const struct qemu_plugin_= hwaddr *haddr) { #ifdef CONFIG_SOFTMMU return haddr->is_io; @@ -282,7 +292,8 @@ bool qemu_plugin_hwaddr_is_io(const struct qemu_plugin_= hwaddr *haddr) #endif } =20 -uint64_t qemu_plugin_hwaddr_device_offset(const struct qemu_plugin_hwaddr = *haddr) +QEMU_PLUGIN_EXPORT uint64_t qemu_plugin_hwaddr_device_offset( + const struct qemu_plugin_hwaddr *haddr) { #ifdef CONFIG_SOFTMMU if (haddr) { @@ -313,7 +324,7 @@ static MachineState * get_ms(void) } #endif =20 -int qemu_plugin_n_vcpus(void) +QEMU_PLUGIN_EXPORT int qemu_plugin_n_vcpus(void) { #ifdef CONFIG_USER_ONLY return -1; @@ -322,7 +333,7 @@ int qemu_plugin_n_vcpus(void) #endif } =20 -int qemu_plugin_n_max_vcpus(void) +QEMU_PLUGIN_EXPORT int qemu_plugin_n_max_vcpus(void) { #ifdef CONFIG_USER_ONLY return -1; @@ -334,7 +345,9 @@ int qemu_plugin_n_max_vcpus(void) /* * Plugin output */ -void qemu_plugin_outs(const char *string) +QEMU_PLUGIN_EXPORT void qemu_plugin_outs(const char *string) { qemu_log_mask(CPU_LOG_PLUGIN, "%s", string); } + +#pragma GCC diagnostic pop diff --git a/plugins/core.c b/plugins/core.c index 51bfc94787..3df3e7d9af 100644 --- a/plugins/core.c +++ b/plugins/core.c @@ -32,6 +32,9 @@ #include "trace/mem-internal.h" /* mem_info macros */ #include "plugin.h" =20 +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wmissing-prototypes" + struct qemu_plugin_cb { struct qemu_plugin_ctx *ctx; union qemu_plugin_cb_sig f; @@ -233,7 +236,7 @@ static void plugin_vcpu_for_each(gpointer k, gpointer v= , gpointer udata) args->cb(args->ctx->id, cpu_index); } =20 -void qemu_plugin_vcpu_for_each(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_vcpu_for_each(qemu_plugin_id_t id, qemu_plugin_vcpu_simple_cb_t cb) { struct plugin_for_each_args args; @@ -384,19 +387,19 @@ void qemu_plugin_vcpu_resume_cb(CPUState *cpu) plugin_vcpu_cb__simple(cpu, QEMU_PLUGIN_EV_VCPU_RESUME); } =20 -void qemu_plugin_register_vcpu_idle_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_idle_cb(qemu_plugin_id_t= id, qemu_plugin_vcpu_simple_cb_t cb) { plugin_register_cb(id, QEMU_PLUGIN_EV_VCPU_IDLE, cb); } =20 -void qemu_plugin_register_vcpu_resume_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_vcpu_resume_cb(qemu_plugin_id= _t id, qemu_plugin_vcpu_simple_cb_t cb) { plugin_register_cb(id, QEMU_PLUGIN_EV_VCPU_RESUME, cb); } =20 -void qemu_plugin_register_flush_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_flush_cb(qemu_plugin_id_t id, qemu_plugin_simple_cb_t cb) { plugin_register_cb(id, QEMU_PLUGIN_EV_FLUSH, cb); @@ -463,7 +466,7 @@ void qemu_plugin_atexit_cb(void) plugin_cb__udata(QEMU_PLUGIN_EV_ATEXIT); } =20 -void qemu_plugin_register_atexit_cb(qemu_plugin_id_t id, +QEMU_PLUGIN_EXPORT void qemu_plugin_register_atexit_cb(qemu_plugin_id_t id, qemu_plugin_udata_cb_t cb, void *udata) { @@ -500,3 +503,5 @@ static void __attribute__((__constructor__)) plugin_ini= t(void) QHT_MODE_AUTO_RESIZE); atexit(qemu_plugin_atexit_cb); } + +#pragma GCC diagnostic pop diff --git a/plugins/loader.c b/plugins/loader.c index 8ac5dbc20f..38e050ff9b 100644 --- a/plugins/loader.c +++ b/plugins/loader.c @@ -63,6 +63,7 @@ QemuOptsList qemu_plugin_opts =3D { }, }; =20 +typedef int (*qemu_plugin_initialize_func_t)(const qemu_info_t *); typedef int (*qemu_plugin_install_func_t)(qemu_plugin_id_t, const qemu_inf= o_t *, int, char **); =20 extern struct qemu_plugin_state plugin; @@ -152,10 +153,12 @@ static uint64_t xorshift64star(uint64_t x) =20 static int plugin_load(struct qemu_plugin_desc *desc, const qemu_info_t *i= nfo) { + qemu_plugin_initialize_func_t initialize =3D NULL; qemu_plugin_install_func_t install; struct qemu_plugin_ctx *ctx; gpointer sym; int rc; + int version =3D -1; =20 ctx =3D qemu_memalign(qemu_dcache_linesize, sizeof(*ctx)); memset(ctx, 0, sizeof(*ctx)); @@ -184,7 +187,7 @@ static int plugin_load(struct qemu_plugin_desc *desc, c= onst qemu_info_t *info) desc->path, g_module_error()); goto err_symbol; } else { - int version =3D *(int *)sym; + version =3D *(int *)sym; if (version < QEMU_PLUGIN_MIN_VERSION) { error_report("TCG plugin %s requires API version %d, but " "this QEMU supports only a minimum version of %d", @@ -198,6 +201,21 @@ static int plugin_load(struct qemu_plugin_desc *desc, = const qemu_info_t *info) } } =20 + if (version >=3D QEMU_PLUGIN_VERSION_1) { + /* This version should call to qemu_plugin_initialize first */ + if (!g_module_symbol(ctx->handle, "qemu_plugin_initialize", &sym))= { + error_report("%s: %s", __func__, g_module_error()); + goto err_symbol; + } + initialize =3D (qemu_plugin_initialize_func_t) sym; + /* symbol was found; it could be NULL though */ + if (initialize =3D=3D NULL) { + error_report("%s: %s: qemu_plugin_initialize is NULL", + __func__, desc->path); + goto err_symbol; + } + } + qemu_rec_mutex_lock(&plugin.lock); =20 /* find an unused random id with &ctx as the seed */ @@ -216,6 +234,16 @@ static int plugin_load(struct qemu_plugin_desc *desc, = const qemu_info_t *info) } } QTAILQ_INSERT_TAIL(&plugin.ctxs, ctx, entry); + if (initialize !=3D NULL) { + rc =3D initialize(info); + if (rc) { + error_report("%s: qemu_plugin_initialize returned error code %= d", + __func__, rc); + /* qemu_plugin_initialize only loading function symbols */ + goto err_symbol; + } + } + ctx->installing =3D true; rc =3D install(ctx->id, info, desc->argc, desc->argv); ctx->installing =3D false; @@ -254,6 +282,17 @@ static void plugin_desc_free(struct qemu_plugin_desc *= desc) g_free(desc); } =20 +static void *qemu_plugin_global_dlsym(void* context, const char *name) +{ + GModule *global_handle =3D context; + gpointer sym =3D NULL; + if (!g_module_symbol(global_handle, name, &sym)) { + error_report("%s: %s", __func__, g_module_error()); + return NULL; + } + return sym; +} + /** * qemu_plugin_load_list - load a list of plugins * @head: head of the list of descriptors of the plugins to be loaded @@ -267,6 +306,7 @@ int qemu_plugin_load_list(QemuPluginList *head) { struct qemu_plugin_desc *desc, *next; g_autofree qemu_info_t *info =3D g_new0(qemu_info_t, 1); + GModule *global_handle =3D NULL; =20 info->target_name =3D TARGET_NAME; info->version.min =3D QEMU_PLUGIN_MIN_VERSION; @@ -276,6 +316,12 @@ int qemu_plugin_load_list(QemuPluginList *head) info->system_emulation =3D true; info->system.smp_vcpus =3D ms->smp.cpus; info->system.max_vcpus =3D ms->smp.max_cpus; + global_handle =3D g_module_open(NULL, G_MODULE_BIND_LOCAL); + if (global_handle =3D=3D NULL) { + goto err_dlopen; + } + info->dlsym =3D qemu_plugin_global_dlsym; + info->context =3D (void *)global_handle; #else info->system_emulation =3D false; #endif @@ -289,6 +335,8 @@ int qemu_plugin_load_list(QemuPluginList *head) } QTAILQ_REMOVE(head, desc, entry); } + +err_dlopen: return 0; } =20 diff --git a/plugins/meson.build b/plugins/meson.build index e77723010e..b48d2046ff 100644 --- a/plugins/meson.build +++ b/plugins/meson.build @@ -1,13 +1,5 @@ -if 'CONFIG_HAS_LD_DYNAMIC_LIST' in config_host - plugin_ldflags =3D ['-Wl,--dynamic-list=3D' + (meson.build_root() / 'qem= u-plugins-ld.symbols')] -elif 'CONFIG_HAS_LD_EXPORTED_SYMBOLS_LIST' in config_host - plugin_ldflags =3D ['-Wl,-exported_symbols_list,' + (meson.build_root() = / 'qemu-plugins-ld64.symbols')] -else - plugin_ldflags =3D [] -endif - specific_ss.add(when: 'CONFIG_PLUGIN', if_true: [files( 'loader.c', 'core.c', 'api.c', -), declare_dependency(link_args: plugin_ldflags)]) +), declare_dependency(link_args: [], compile_args: [])]) diff --git a/plugins/plugin.h b/plugins/plugin.h index 1aa29dcadd..406f246a72 100644 --- a/plugins/plugin.h +++ b/plugins/plugin.h @@ -15,6 +15,7 @@ #include =20 #define QEMU_PLUGIN_MIN_VERSION 0 +#define QEMU_PLUGIN_VERSION_1 1 =20 /* global state */ struct qemu_plugin_state { diff --git a/plugins/qemu-plugins.symbols b/plugins/qemu-plugins.symbols deleted file mode 100644 index 4bdb381f48..0000000000 --- a/plugins/qemu-plugins.symbols +++ /dev/null @@ -1,40 +0,0 @@ -{ - qemu_plugin_uninstall; - qemu_plugin_reset; - qemu_plugin_register_vcpu_init_cb; - qemu_plugin_register_vcpu_exit_cb; - qemu_plugin_register_vcpu_idle_cb; - qemu_plugin_register_vcpu_resume_cb; - qemu_plugin_register_vcpu_insn_exec_cb; - qemu_plugin_register_vcpu_insn_exec_inline; - qemu_plugin_register_vcpu_mem_cb; - qemu_plugin_register_vcpu_mem_haddr_cb; - qemu_plugin_register_vcpu_mem_inline; - qemu_plugin_ram_addr_from_host; - qemu_plugin_register_vcpu_tb_trans_cb; - qemu_plugin_register_vcpu_tb_exec_cb; - qemu_plugin_register_vcpu_tb_exec_inline; - qemu_plugin_register_flush_cb; - qemu_plugin_register_vcpu_syscall_cb; - qemu_plugin_register_vcpu_syscall_ret_cb; - qemu_plugin_register_atexit_cb; - qemu_plugin_tb_n_insns; - qemu_plugin_tb_get_insn; - qemu_plugin_tb_vaddr; - qemu_plugin_insn_data; - qemu_plugin_insn_size; - qemu_plugin_insn_vaddr; - qemu_plugin_insn_haddr; - qemu_plugin_insn_disas; - qemu_plugin_mem_size_shift; - qemu_plugin_mem_is_sign_extended; - qemu_plugin_mem_is_big_endian; - qemu_plugin_mem_is_store; - qemu_plugin_get_hwaddr; - qemu_plugin_hwaddr_is_io; - qemu_plugin_hwaddr_to_raddr; - qemu_plugin_vcpu_for_each; - qemu_plugin_n_vcpus; - qemu_plugin_n_max_vcpus; - qemu_plugin_outs; -}; --=20 2.28.0.windows.1