qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] target/ppc: Use tcg_gen_lookup_an


From: Daniel Henrique Barboza
Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v2] target/ppc: Use tcg_gen_lookup_and_goto_ptr
Date: Tue, 14 Nov 2017 14:05:51 -0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0



On 10/27/2017 02:58 PM, Richard Henderson wrote:
From: Richard Henderson <address@hidden>

Signed-off-by: Richard Henderson <address@hidden>
---
Rebased and updated for changes to tcg_gen_lookup_and_goto_ptr.

---

I am experiencing a remarkable performance boost running a simple scenario
up to SLOF prompt with this patch. It takes 50 seconds on average to get to SLOF
prompt as is now on my Lenovo laptop, upstream qemu and default pseries
machine. With this patch it takes 21 seconds average.


Reviewed-by: Daniel Henrique Barboza <address@hidden>

  target/ppc/translate.c | 23 ++++++++---------------
  1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/target/ppc/translate.c b/target/ppc/translate.c
index 998fbed848..4075fc8589 100644
--- a/target/ppc/translate.c
+++ b/target/ppc/translate.c
@@ -3419,7 +3419,7 @@ static inline bool use_goto_tb(DisasContext *ctx, 
target_ulong dest)
  }

  /***                                Branch                                 
***/
-static inline void gen_goto_tb(DisasContext *ctx, int n, target_ulong dest)
+static void gen_goto_tb(DisasContext *ctx, int n, target_ulong dest)
  {
      if (NARROW_MODE(ctx)) {
          dest = (uint32_t) dest;
@@ -3441,7 +3441,7 @@ static inline void gen_goto_tb(DisasContext *ctx, int n, 
target_ulong dest)
                  gen_debug_exception(ctx);
              }
          }
-        tcg_gen_exit_tb(0);
+        tcg_gen_lookup_and_goto_ptr();
      }
  }

@@ -3479,7 +3479,7 @@ static void gen_b(DisasContext *ctx)
  #define BCOND_CTR 2
  #define BCOND_TAR 3

-static inline void gen_bcond(DisasContext *ctx, int type)
+static void gen_bcond(DisasContext *ctx, int type)
  {
      uint32_t bo = BO(ctx->opcode);
      TCGLabel *l1;
@@ -3543,26 +3543,19 @@ static inline void gen_bcond(DisasContext *ctx, int 
type)
          } else {
              gen_goto_tb(ctx, 0, li);
          }
-        if ((bo & 0x14) != 0x14) {
-            gen_set_label(l1);
-            gen_goto_tb(ctx, 1, ctx->nip);
-        }
      } else {
          if (NARROW_MODE(ctx)) {
              tcg_gen_andi_tl(cpu_nip, target, (uint32_t)~3);
          } else {
              tcg_gen_andi_tl(cpu_nip, target, ~3);
          }
-        tcg_gen_exit_tb(0);
-        if ((bo & 0x14) != 0x14) {
-            gen_set_label(l1);
-            gen_update_nip(ctx, ctx->nip);
-            tcg_gen_exit_tb(0);
-        }
-    }
-    if (type == BCOND_LR || type == BCOND_CTR || type == BCOND_TAR) {
+        tcg_gen_lookup_and_goto_ptr();
          tcg_temp_free(target);
      }
+    if ((bo & 0x14) != 0x14) {
+        gen_set_label(l1);
+        gen_goto_tb(ctx, 1, ctx->nip);
+    }
  }

  static void gen_bc(DisasContext *ctx)




reply via email to

[Prev in Thread] Current Thread [Next in Thread]