On 29/04/16 19:32, Richard Henderson wrote: > On 04/29/2016 06:58 AM, Sergey Fedorov wrote: >> On 29/04/16 16:54, Alex Bennée wrote: >>> Sergey Fedorov <sergey.fedo...@linaro.org> writes: >>>> diff --git a/cpu-exec.c b/cpu-exec.c >>>> index f49a436e1a5a..5f23c0660d6e 100644 >>>> --- a/cpu-exec.c >>>> +++ b/cpu-exec.c >>>> @@ -320,7 +320,9 @@ found: >>>> return tb; >>>> } >>>> >>>> -static inline TranslationBlock *tb_find_fast(CPUState *cpu) >>>> +static inline TranslationBlock *tb_find_fast(CPUState *cpu, >>>> + TranslationBlock **last_tb, >>>> + int tb_exit) >>>> { >>>> CPUArchState *env = (CPUArchState *)cpu->env_ptr; >>>> TranslationBlock *tb; >>>> @@ -331,11 +333,24 @@ static inline TranslationBlock >>>> *tb_find_fast(CPUState *cpu) >>>> always be the same before a given translated block >>>> is executed. */ >>>> cpu_get_tb_cpu_state(env, &pc, &cs_base, &flags); >>>> + tb_lock(); >>>> tb = cpu->tb_jmp_cache[tb_jmp_cache_hash_func(pc)]; >>>> if (unlikely(!tb || tb->pc != pc || tb->cs_base != cs_base || >>>> tb->flags != flags)) { >>>> tb = tb_find_slow(cpu, pc, cs_base, flags); >>>> } >>>> + if (cpu->tb_flushed) { >>>> + /* Ensure that no TB jump will be modified as the >>>> + * translation buffer has been flushed. >>>> + */ >>>> + *last_tb = NULL; >>>> + cpu->tb_flushed = false; >>>> + } >>>> + /* See if we can patch the calling TB. */ >>>> + if (*last_tb && qemu_loglevel_mask(CPU_LOG_TB_NOCHAIN)) { >>> This should be !qemu_loglevel_mask(CPU_LOG_TB_NOCHAIN) >> Probably, it's mine rebase conflict resolution mistake. Nice catch, thanks! > Fixed while applying all to tcg-next.
Thanks! Kind regards, Sergey