commit 9b990ee5a3cc6aa38f81266fb0c6ef37a36c45b9
author Richard Henderson <richard.henderson@linaro.org> Fri Oct 13 10:50:02 2017 -0700
committer Richard Henderson <richard.henderson@linaro.org> Tue Oct 24 13:53:41 2017 -0700
tree ed59ca42a7bfa9d7bac6785f2ad2852d2ef919f9
parent 4e2ca83e71b51577b06b1468e836556912bd5b6e

tcg: Add CPUState cflags_next_tb

We were generating code during tb_invalidate_phys_page_range,
check_watchpoint, cpu_io_recompile, and (seemingly) discarding
the TB, assuming that it would magically be picked up during
the next iteration through the cpu_exec loop.

Instead, record the desired cflags in CPUState so that we request
the proper TB so that there is no more magic.

Reviewed-by: Emilio G. Cota <cota@braap.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

exec.c

diff --git a/exec.c b/exec.c
index 3e0a3da..97a24a8 100644
--- a/exec.c
+++ b/exec.c
@@ -2431,11 +2431,8 @@
 {
     CPUState *cpu = current_cpu;
     CPUClass *cc = CPU_GET_CLASS(cpu);
-    CPUArchState *env = cpu->env_ptr;
-    target_ulong pc, cs_base;
     target_ulong vaddr;
     CPUWatchpoint *wp;
-    uint32_t cpu_flags;
 
     assert(tcg_enabled());
     if (cpu->watchpoint_hit) {
@@ -2475,8 +2472,8 @@
                     cpu->exception_index = EXCP_DEBUG;
                     cpu_loop_exit(cpu);
                 } else {
-                    cpu_get_tb_cpu_state(env, &pc, &cs_base, &cpu_flags);
-                    tb_gen_code(cpu, pc, cs_base, cpu_flags, 1 | curr_cflags());
+                    /* Force execution of one insn next time.  */
+                    cpu->cflags_next_tb = 1 | curr_cflags();
                     cpu_loop_exit_noexc(cpu);
                 }
             }