[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 06/41] accel/tcg: Introduce translator_use_goto_tb
From: |
Richard Henderson |
Subject: |
[PATCH 06/41] accel/tcg: Introduce translator_use_goto_tb |
Date: |
Sat, 10 Jul 2021 08:31:08 -0700 |
Add a generic version of the common use_goto_tb test.
Various targets avoid the page crossing test for CONFIG_USER_ONLY,
but that is wrong: mmap and mprotect can change page permissions.
Reviewed-by: Max Filippov <jcmvbkbc@gmail.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
include/exec/translator.h | 10 ++++++++++
accel/tcg/translator.c | 11 +++++++++++
2 files changed, 21 insertions(+)
diff --git a/include/exec/translator.h b/include/exec/translator.h
index 24232ead41..dd9c06d40d 100644
--- a/include/exec/translator.h
+++ b/include/exec/translator.h
@@ -145,6 +145,16 @@ void translator_loop(const TranslatorOps *ops,
DisasContextBase *db,
void translator_loop_temp_check(DisasContextBase *db);
+/**
+ * translator_use_goto_tb
+ * @db: Disassembly context
+ * @dest: target pc of the goto
+ *
+ * Return true if goto_tb is allowed between the current TB
+ * and the destination PC.
+ */
+bool translator_use_goto_tb(DisasContextBase *db, target_ulong dest);
+
/*
* Translator Load Functions
*
diff --git a/accel/tcg/translator.c b/accel/tcg/translator.c
index 1d32732198..59804af37b 100644
--- a/accel/tcg/translator.c
+++ b/accel/tcg/translator.c
@@ -31,6 +31,17 @@ void translator_loop_temp_check(DisasContextBase *db)
}
}
+bool translator_use_goto_tb(DisasContextBase *db, target_ulong dest)
+{
+ /* Suppress goto_tb in the case of single-steping. */
+ if (db->singlestep_enabled || singlestep) {
+ return false;
+ }
+
+ /* Check for the dest on the same page as the start of the TB. */
+ return ((db->pc_first ^ dest) & TARGET_PAGE_MASK) == 0;
+}
+
void translator_loop(const TranslatorOps *ops, DisasContextBase *db,
CPUState *cpu, TranslationBlock *tb, int max_insns)
{
--
2.25.1
- [PATCH 00/41] tcg patch queue, Richard Henderson, 2021/07/10
- [PATCH 03/41] accel/tcg: Hoist tcg_tb_insert() up above tb_link_page(), Richard Henderson, 2021/07/10
- [PATCH 02/41] tcg: Avoid including 'trace-tcg.h' in target translate.c, Richard Henderson, 2021/07/10
- [PATCH 04/41] tcg: Bake tb_destroy() into tcg_region_tree, Richard Henderson, 2021/07/10
- [PATCH 01/41] tcg: Add separator in INDEX_op_call dump, Richard Henderson, 2021/07/10
- [PATCH 06/41] accel/tcg: Introduce translator_use_goto_tb,
Richard Henderson <=
- [PATCH 05/41] tcg: Move tb_phys_invalidate_count to tb_ctx, Richard Henderson, 2021/07/10
- [PATCH 07/41] target/alpha: Remove use_exit_tb, Richard Henderson, 2021/07/10
- [PATCH 08/41] target/alpha: Remove in_superpage, Richard Henderson, 2021/07/10
- [PATCH 09/41] target/alpha: Use translator_use_goto_tb, Richard Henderson, 2021/07/10
- [PATCH 10/41] target/arm: Use DISAS_TOO_MANY for ISB and SB, Richard Henderson, 2021/07/10
- [PATCH 11/41] target/arm: Use translator_use_goto_tb for aarch64, Richard Henderson, 2021/07/10
- [PATCH 12/41] target/arm: Use translator_use_goto_tb for aarch32, Richard Henderson, 2021/07/10
- [PATCH 14/41] target/avr: Mark some helpers noreturn, Richard Henderson, 2021/07/10
- [PATCH 18/41] target/m68k: Use translator_use_goto_tb, Richard Henderson, 2021/07/10
- [PATCH 13/41] target/avr: Use translator_use_goto_tb, Richard Henderson, 2021/07/10