[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper
From: |
Emilio G. Cota |
Subject: |
Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper |
Date: |
Wed, 26 Apr 2017 19:17:32 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Wed, Apr 26, 2017 at 10:40:45 +0200, Richard Henderson wrote:
> On 04/26/2017 08:23 AM, Emilio G. Cota wrote:
> >This paves the way for upcoming work.
> >
> >Reviewed-by: Richard Henderson <address@hidden>
> >Signed-off-by: Emilio G. Cota <address@hidden>
> >---
> > tcg-runtime.c | 21 +++++++++++++++++++++
> > tcg/tcg-runtime.h | 2 ++
> > tcg/tcg.h | 1 +
> > 3 files changed, 24 insertions(+)
> >
> >diff --git a/tcg-runtime.c b/tcg-runtime.c
> >index 4c60c96..90d2d4b 100644
> >--- a/tcg-runtime.c
> >+++ b/tcg-runtime.c
> >@@ -27,6 +27,7 @@
> > #include "exec/helper-proto.h"
> > #include "exec/cpu_ldst.h"
> > #include "exec/exec-all.h"
> >+#include "exec/tb-hash.h"
> > /* 32-bit helpers */
> >@@ -141,6 +142,26 @@ uint64_t HELPER(ctpop_i64)(uint64_t arg)
> > return ctpop64(arg);
> > }
> >+ cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags);
> >+ tb = atomic_rcu_read(&cpu->tb_jmp_cache[tb_jmp_cache_hash_func(addr)]);
> >+ if (likely(tb && tb->pc == addr && tb->cs_base == cs_base &&
> >+ tb->flags == flags)) {
>
> This comparison is wrong. It will incorrectly reject a TB for i386 guest
> when CS_BASE != 0. You really want
>
> tb = atomic_rcu_read(&cpu->tb_jmp_cache[tb_jmp_cache_hash_func(addr)]);
> if (tb) {
> cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags);
> if (tb->pc == pc && tb->cs_base == cs_base && tb->flags == flags) {
> return tb->tc_ptr;
> }
> }
> return tcg_ctx.code_gen_epilogue;
>
> where you don't even load the cpu state if there isn't a preliminary hit in
> the cache.
Yes, I like this.
> (Note to self: That minor optimization would also apply to tb_find.)
FWIW I looked at tb_find -- you need the pc though, which comes from
loading the CPU state:
cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags);
^^
tb = atomic_rcu_read(&cpu->tb_jmp_cache[tb_jmp_cache_hash_func(pc)]);
^^
If we wanted to really avoid getting all the state I guess we'd have to add
another function that returned just the pc.
E.
- [Qemu-devel] [PATCH v3 09/10] target/i386: optimize indirect branches, (continued)
- [Qemu-devel] [PATCH v3 09/10] target/i386: optimize indirect branches, Emilio G. Cota, 2017/04/26
- [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Emilio G. Cota, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Paolo Bonzini, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Richard Henderson, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Emilio G. Cota, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Richard Henderson, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Emilio G. Cota, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Emilio G. Cota, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Emilio G. Cota, 2017/04/26
- Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper,
Emilio G. Cota <=
Re: [Qemu-devel] [PATCH v3 01/10] tcg-runtime: add lookup_tb_ptr helper, Alex Bennée, 2017/04/26
[Qemu-devel] [PATCH v3 06/10] target/arm: optimize indirect branches, Emilio G. Cota, 2017/04/26
[Qemu-devel] [PATCH v3 10/10] tb-hash: improve tb_jmp_cache hash function in user mode, Emilio G. Cota, 2017/04/26