[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 01/28] accel/tcg: Introduce translator_use_goto_tb
From: |
Richard Henderson |
Subject: |
[PATCH v2 01/28] accel/tcg: Introduce translator_use_goto_tb |
Date: |
Wed, 30 Jun 2021 11:31:59 -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 v2 00/28] accel/tcg: Introduce translator_use_goto_tb, Richard Henderson, 2021/06/30
- [PATCH v2 01/28] accel/tcg: Introduce translator_use_goto_tb,
Richard Henderson <=
- [PATCH v2 02/28] target/alpha: Remove use_exit_tb, Richard Henderson, 2021/06/30
- [PATCH v2 06/28] target/arm: Use translator_use_goto_tb for aarch64, Richard Henderson, 2021/06/30
- [PATCH v2 09/28] target/avr: Mark some helpers noreturn, Richard Henderson, 2021/06/30
- [PATCH v2 05/28] target/arm: Use gen_jmp for ISB and SB, Richard Henderson, 2021/06/30
- [PATCH v2 03/28] target/alpha: Remove in_superpage, Richard Henderson, 2021/06/30
- [PATCH v2 08/28] target/avr: Use translator_use_goto_tb, Richard Henderson, 2021/06/30
- [PATCH v2 04/28] target/alpha: Use translator_use_goto_tb, Richard Henderson, 2021/06/30
- [PATCH v2 15/28] target/mips: Use translator_use_goto_tb, Richard Henderson, 2021/06/30
- [PATCH v2 12/28] target/i386: Use translator_use_goto_tb, Richard Henderson, 2021/06/30
- [PATCH v2 17/28] target/nios2: Use translator_use_goto_tb, Richard Henderson, 2021/06/30