diff options
author | Emilio G. Cota <cota@braap.org> | 2018-10-10 10:48:53 -0400 |
---|---|---|
committer | Richard Henderson <richard.henderson@linaro.org> | 2018-10-18 18:58:10 -0700 |
commit | 72fd2efbbd52c1a7974000a60a0c2131b1a4aaf2 (patch) | |
tree | 888c2d7d7dfd9ba129d591d6891e002a521b987a /cpus.c | |
parent | dd1d7da23b0abef87f46d9ab39ba9b0974eaec04 (diff) | |
download | qemu-72fd2efbbd52c1a7974000a60a0c2131b1a4aaf2.zip qemu-72fd2efbbd52c1a7974000a60a0c2131b1a4aaf2.tar.gz qemu-72fd2efbbd52c1a7974000a60a0c2131b1a4aaf2.tar.bz2 |
tcg: distribute tcg_time into TCG contexts
When we implemented per-vCPU TCG contexts, we forgot to also
distribute the tcg_time counter, which has remained as a global
accessed without any serialization, leading to potentially missed
counts.
Fix it by distributing the field over the TCG contexts, embedding
it into TCGProfile with a field called "cpu_exec_time", which is more
descriptive than "tcg_time". Add a function to query this value
directly, and for completeness, fill in the field in
tcg_profile_snapshot, even though its callers do not use it.
Signed-off-by: Emilio G. Cota <cota@braap.org>
Message-Id: <20181010144853.13005-5-cota@braap.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'cpus.c')
-rw-r--r-- | cpus.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -1425,7 +1425,8 @@ static int tcg_cpu_exec(CPUState *cpu) ret = cpu_exec(cpu); cpu_exec_end(cpu); #ifdef CONFIG_PROFILER - tcg_time += profile_getclock() - ti; + atomic_set(&tcg_ctx->prof.cpu_exec_time, + tcg_ctx->prof.cpu_exec_time + profile_getclock() - ti); #endif return ret; } |