cputlb: bring back tlb_flush_count under !TLB_DEBUG
commit83974cf4f8a46513f799ff0d7c7eb151acafda7b
authorEmilio G. Cota <cota@braap.org>
Thu, 6 Jul 2017 18:42:26 +0000 (6 14:42 -0400)
committerRichard Henderson <richard.henderson@linaro.org>
Tue, 10 Oct 2017 14:37:10 +0000 (10 07:37 -0700)
treece7c8add92d4cd60e832564b066be0c6a12c0e2b
parent567d0a19c7998fa366598b83d5a6e5f0759d3ea9
cputlb: bring back tlb_flush_count under !TLB_DEBUG

Commit f0aff0f124 ("cputlb: add assert_cpu_is_self checks") buried
the increment of tlb_flush_count under TLB_DEBUG. This results in
"info jit" always (mis)reporting 0 TLB flushes when !TLB_DEBUG.

Besides, under MTTCG tlb_flush_count is updated by several threads,
so in order not to lose counts we'd either have to use atomic ops
or distribute the counter, which is more scalable.

This patch does the latter by embedding tlb_flush_count in CPUArchState.
The global count is then easily obtained by iterating over the CPU list.

Note that this change also requires updating the accessors to
tlb_flush_count to use atomic_read/set whenever there may be conflicting
accesses (as defined in C11) to it.

Reviewed-by: Richard Henderson <rth@twiddle.net>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Emilio G. Cota <cota@braap.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
accel/tcg/cputlb.c
accel/tcg/translate-all.c
include/exec/cpu-defs.h
include/exec/cputlb.h