aboutsummaryrefslogtreecommitdiff
path: root/accel/tcg
diff options
context:
space:
mode:
authorRichard Henderson <richard.henderson@linaro.org>2022-11-06 11:12:33 +1100
committerRichard Henderson <richard.henderson@linaro.org>2022-11-09 12:29:03 +1100
commit344b63b380541a63c02ef7a8a6ae66cb0b6f0273 (patch)
tree3c1652bd446d7eb53281147fe0ad72b06176e16b /accel/tcg
parent35abb009b22e93e89cc627de74fa90339b680882 (diff)
downloadqemu-344b63b380541a63c02ef7a8a6ae66cb0b6f0273.zip
qemu-344b63b380541a63c02ef7a8a6ae66cb0b6f0273.tar.gz
qemu-344b63b380541a63c02ef7a8a6ae66cb0b6f0273.tar.bz2
accel/tcg: Split out setjmp_gen_code
Isolate the code protected by setjmp. Fixes: translate-all.c: In function ‘tb_gen_code’: translate-all.c:748:51: error: argument ‘cflags’ might be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered] Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'accel/tcg')
-rw-r--r--accel/tcg/translate-all.c58
1 files changed, 33 insertions, 25 deletions
diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c
index 9ee21f7..ac3ee37 100644
--- a/accel/tcg/translate-all.c
+++ b/accel/tcg/translate-all.c
@@ -742,6 +742,37 @@ void page_collection_unlock(struct page_collection *set)
#endif /* !CONFIG_USER_ONLY */
+/*
+ * Isolate the portion of code gen which can setjmp/longjmp.
+ * Return the size of the generated code, or negative on error.
+ */
+static int setjmp_gen_code(CPUArchState *env, TranslationBlock *tb,
+ target_ulong pc, void *host_pc,
+ int *max_insns, int64_t *ti)
+{
+ int ret = sigsetjmp(tcg_ctx->jmp_trans, 0);
+ if (unlikely(ret != 0)) {
+ return ret;
+ }
+
+ tcg_func_start(tcg_ctx);
+
+ tcg_ctx->cpu = env_cpu(env);
+ gen_intermediate_code(env_cpu(env), tb, *max_insns, pc, host_pc);
+ assert(tb->size != 0);
+ tcg_ctx->cpu = NULL;
+ *max_insns = tb->icount;
+
+#ifdef CONFIG_PROFILER
+ qatomic_set(&tcg_ctx->prof.tb_count, tcg_ctx->prof.tb_count + 1);
+ qatomic_set(&tcg_ctx->prof.interm_time,
+ tcg_ctx->prof.interm_time + profile_getclock() - *ti);
+ *ti = profile_getclock();
+#endif
+
+ return tcg_gen_code(tcg_ctx, tb, pc);
+}
+
/* Called with mmap_lock held for user mode emulation. */
TranslationBlock *tb_gen_code(CPUState *cpu,
target_ulong pc, target_ulong cs_base,
@@ -754,8 +785,8 @@ TranslationBlock *tb_gen_code(CPUState *cpu,
int gen_code_size, search_size, max_insns;
#ifdef CONFIG_PROFILER
TCGProfile *prof = &tcg_ctx->prof;
- int64_t ti;
#endif
+ int64_t ti;
void *host_pc;
assert_memory_lock();
@@ -805,33 +836,10 @@ TranslationBlock *tb_gen_code(CPUState *cpu,
ti = profile_getclock();
#endif
- gen_code_size = sigsetjmp(tcg_ctx->jmp_trans, 0);
- if (unlikely(gen_code_size != 0)) {
- goto error_return;
- }
-
- tcg_func_start(tcg_ctx);
-
- tcg_ctx->cpu = env_cpu(env);
- gen_intermediate_code(cpu, tb, max_insns, pc, host_pc);
- assert(tb->size != 0);
- tcg_ctx->cpu = NULL;
- max_insns = tb->icount;
-
trace_translate_block(tb, pc, tb->tc.ptr);
- /* generate machine code */
-
-#ifdef CONFIG_PROFILER
- qatomic_set(&prof->tb_count, prof->tb_count + 1);
- qatomic_set(&prof->interm_time,
- prof->interm_time + profile_getclock() - ti);
- ti = profile_getclock();
-#endif
-
- gen_code_size = tcg_gen_code(tcg_ctx, tb, pc);
+ gen_code_size = setjmp_gen_code(env, tb, pc, host_pc, &max_insns, &ti);
if (unlikely(gen_code_size < 0)) {
- error_return:
switch (gen_code_size) {
case -1:
/*