[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 12/37] contrib/plugins/howvec: ensure we don't regress if this plu
From: |
Alex Bennée |
Subject: |
[PULL 12/37] contrib/plugins/howvec: ensure we don't regress if this plugin is extended |
Date: |
Fri, 17 Jan 2025 13:42:31 +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>
Message-Id: <20250116160306.1709518-13-alex.bennee@linaro.org>
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
- [PULL 02/37] semihosting/uaccess: Briefly document returned values, (continued)
- [PULL 02/37] semihosting/uaccess: Briefly document returned values, Alex Bennée, 2025/01/17
- [PULL 07/37] semihosting/meson: Build config.o and console.o once, Alex Bennée, 2025/01/17
- [PULL 21/37] configure: reenable plugins by default for 32-bit hosts, Alex Bennée, 2025/01/17
- [PULL 03/37] semihosting/syscalls: Include missing 'exec/cpu-defs.h' header, Alex Bennée, 2025/01/17
- [PULL 18/37] contrib/plugins/cflow: fix 32-bit build, Alex Bennée, 2025/01/17
- [PULL 16/37] contrib/plugins/cache: fix 32-bit build, Alex Bennée, 2025/01/17
- [PULL 17/37] contrib/plugins/hotblocks: fix 32-bit build, Alex Bennée, 2025/01/17
- [PULL 10/37] system: propagate Error to gdbserver_start (and other device setups), Alex Bennée, 2025/01/17
- [PULL 15/37] contrib/plugins/stoptrigger: fix 32-bit build, Alex Bennée, 2025/01/17
- [PULL 23/37] win32: remove usage of attribute gcc_struct, Alex Bennée, 2025/01/17
- [PULL 12/37] contrib/plugins/howvec: ensure we don't regress if this plugin is extended,
Alex Bennée <=
- [PULL 36/37] docs: add a glossary, Alex Bennée, 2025/01/17
- [PULL 30/37] include/exec: remove warning_printed from MemoryRegion, Alex Bennée, 2025/01/17
- [PULL 29/37] include/exec: fix some copy and paste errors in kdoc, Alex Bennée, 2025/01/17
- [PULL 14/37] tests/tcg/plugins/mem: fix 32-bit build, Alex Bennée, 2025/01/17
- [PULL 27/37] editorconfig: update for perl scripts, Alex Bennée, 2025/01/17
- [PULL 06/37] semihosting/console: Avoid including 'cpu.h', Alex Bennée, 2025/01/17
- [PULL 09/37] system: squash usb_parse into a single function, Alex Bennée, 2025/01/17
- [PULL 22/37] accel/tcg: also suppress asynchronous IRQs for cpu_io_recompile, Alex Bennée, 2025/01/17
- [PULL 28/37] tests/qtest: fix some copy and paste errors in kdoc, Alex Bennée, 2025/01/17
- [PULL 33/37] docs/devel: add b4 for patch retrieval, Alex Bennée, 2025/01/17