[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 12/22] contrib/plugins/howvec: ensure we don't regress if this pl
From: |
Alex Bennée |
Subject: |
[PATCH 12/22] contrib/plugins/howvec: ensure we don't regress if this plugin is extended |
Date: |
Thu, 9 Jan 2025 17:06:09 +0000 |
From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Message-Id: <20241217224306.2900490-3-pierrick.bouvier@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
contrib/plugins/howvec.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/contrib/plugins/howvec.c b/contrib/plugins/howvec.c
index 9be67f7453..2aa9029c3f 100644
--- a/contrib/plugins/howvec.c
+++ b/contrib/plugins/howvec.c
@@ -253,6 +253,8 @@ static struct qemu_plugin_scoreboard *find_counter(
int i;
uint64_t *cnt = NULL;
uint32_t opcode = 0;
+ /* if opcode is greater than 32 bits, we should refactor insn hash table.
*/
+ G_STATIC_ASSERT(sizeof(opcode) == sizeof(uint32_t));
InsnClassExecCount *class = NULL;
/*
@@ -284,7 +286,7 @@ static struct qemu_plugin_scoreboard *find_counter(
g_mutex_lock(&lock);
icount = (InsnExecCount *) g_hash_table_lookup(insns,
-
GUINT_TO_POINTER(opcode));
+ (gpointer)(intptr_t)
opcode);
if (!icount) {
icount = g_new0(InsnExecCount, 1);
@@ -295,8 +297,7 @@ static struct qemu_plugin_scoreboard *find_counter(
qemu_plugin_scoreboard_new(sizeof(uint64_t));
icount->count = qemu_plugin_scoreboard_u64(score);
- g_hash_table_insert(insns, GUINT_TO_POINTER(opcode),
- (gpointer) icount);
+ g_hash_table_insert(insns, (gpointer)(intptr_t) opcode, icount);
}
g_mutex_unlock(&lock);
--
2.39.5
- [PATCH 04/22] semihosting/uaccess: Include missing 'exec/cpu-all.h' header, (continued)
- [PATCH 04/22] semihosting/uaccess: Include missing 'exec/cpu-all.h' header, Alex Bennée, 2025/01/09
- [PATCH 02/22] semihosting/uaccess: Briefly document returned values, Alex Bennée, 2025/01/09
- [PATCH 01/22] semihosting: add guest_error logging for failed opens, Alex Bennée, 2025/01/09
- [PATCH 09/22] system: squash usb_parse into a single function, Alex Bennée, 2025/01/09
- [PATCH 13/22] tests/tcg/plugins/syscall: fix 32-bit build, Alex Bennée, 2025/01/09
- [PATCH 14/22] tests/tcg/plugins/mem: fix 32-bit build, Alex Bennée, 2025/01/09
- [PATCH 16/22] contrib/plugins/cache: fix 32-bit build, Alex Bennée, 2025/01/09
- [PATCH 07/22] semihosting/meson: Build config.o and console.o once, Alex Bennée, 2025/01/09
- [PATCH 12/22] contrib/plugins/howvec: ensure we don't regress if this plugin is extended,
Alex Bennée <=
- [PATCH 08/22] system/vl: more error exit into config enumeration code, Alex Bennée, 2025/01/09
- [PATCH 06/22] semihosting/console: Avoid including 'cpu.h', Alex Bennée, 2025/01/09
- [PATCH 03/22] semihosting/syscalls: Include missing 'exec/cpu-defs.h' header, Alex Bennée, 2025/01/09
- [PATCH 10/22] system: propagate Error to gdbserver_start (and other device setups), Alex Bennée, 2025/01/09