On Sun, Sep 27, 2020 at 4:45 PM Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > > On 9/27/20 10:18 AM, Yonggang Luo wrote: > > Signed-off-by: Yonggang Luo <luoyongg...@gmail.com> > > Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > > --- > > contrib/plugins/hotblocks.c | 2 +- > > tests/plugin/bb.c | 4 ++-- > > 2 files changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/contrib/plugins/hotblocks.c b/contrib/plugins/hotblocks.c > > index 3942a2ca54..37435a3fc7 100644 > > --- a/contrib/plugins/hotblocks.c > > +++ b/contrib/plugins/hotblocks.c > > @@ -102,7 +102,7 @@ static void vcpu_tb_trans(qemu_plugin_id_t id, struct qemu_plugin_tb *tb) > > { > > ExecCount *cnt; > > uint64_t pc = qemu_plugin_tb_vaddr(tb); > > - unsigned long insns = qemu_plugin_tb_n_insns(tb); > > + size_t insns = qemu_plugin_tb_n_insns(tb); > > uint64_t hash = pc ^ insns; > > > > g_mutex_lock(&lock); > > diff --git a/tests/plugin/bb.c b/tests/plugin/bb.c > > index e4cc7fdd6e..de09bdde4e 100644 > > --- a/tests/plugin/bb.c > > +++ b/tests/plugin/bb.c > > @@ -72,7 +72,7 @@ static void vcpu_tb_exec(unsigned int cpu_index, void *udata) > > CPUCount *count = max_cpus ? > > g_ptr_array_index(counts, cpu_index) : &inline_count; > > > > - unsigned long n_insns = (unsigned long)udata; > > + uintptr_t n_insns = (uintptr_t)udata; > > Shouldn't this be: > > uint64_t n_insns = (uint64_t)(uintptr_t)udata; don't know, may i assmue uintptr_t <= uint64_t , then that's not a prolem > > ? > > > g_mutex_lock(&count->lock); > > count->insn_count += n_insns; > > count->bb_count++; > > @@ -81,7 +81,7 @@ static void vcpu_tb_exec(unsigned int cpu_index, void *udata) > > > > static void vcpu_tb_trans(qemu_plugin_id_t id, struct qemu_plugin_tb *tb) > > { > > - unsigned long n_insns = qemu_plugin_tb_n_insns(tb); > > + size_t n_insns = qemu_plugin_tb_n_insns(tb); > > > > if (do_inline) { > > qemu_plugin_register_vcpu_tb_exec_inline(tb, QEMU_PLUGIN_INLINE_ADD_U64, > >
-- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo