[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v7 22/22] tcg: fix cpu_io_recompile
From: |
Pavel Dovgalyuk |
Subject: |
[Qemu-devel] [PATCH v7 22/22] tcg: fix cpu_io_recompile |
Date: |
Tue, 27 Feb 2018 12:53:38 +0300 |
User-agent: |
StGit/0.17.1-dirty |
cpu_io_recompile() function was broken by
the commit 9b990ee5a3cc6aa38f81266fb0c6ef37a36c45b9. Instead of regenerating
the block starting from PC of the original block, it just set the instruction
counter for TCG. In most cases this was unnoticed, but in icount mode
there was an exception for incorrect usage of CF_LAST_IO flag.
This patch recovers recompilation of the original block and also
configures translation for executing single IO instruction which
caused a recompilation.
Signed-off-by: Pavel Dovgalyuk <address@hidden>
---
accel/tcg/translate-all.c | 18 +++++++++++++++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c
index 67795cd..5ad1b91 100644
--- a/accel/tcg/translate-all.c
+++ b/accel/tcg/translate-all.c
@@ -1728,7 +1728,8 @@ void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr)
CPUArchState *env = cpu->env_ptr;
#endif
TranslationBlock *tb;
- uint32_t n;
+ uint32_t n, flags;
+ target_ulong pc, cs_base;
tb_lock();
tb = tb_find_pc(retaddr);
@@ -1766,8 +1767,14 @@ void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr)
cpu_abort(cpu, "TB too big during recompile");
}
- /* Adjust the execution state of the next TB. */
- cpu->cflags_next_tb = curr_cflags() | CF_LAST_IO | n;
+ pc = tb->pc;
+ cs_base = tb->cs_base;
+ flags = tb->flags;
+ tb_phys_invalidate(tb, -1);
+
+ /* Execute one IO instruction without caching
+ instead of creating large TB. */
+ cpu->cflags_next_tb = curr_cflags() | CF_LAST_IO | CF_NOCACHE | 1;
if (tb->cflags & CF_NOCACHE) {
if (tb->orig_tb) {
@@ -1778,6 +1785,11 @@ void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr)
tb_remove(tb);
}
+ /* Generate new TB instead of the current one. */
+ /* FIXME: In theory this could raise an exception. In practice
+ we have already translated the block once so it's probably ok. */
+ tb_gen_code(cpu, pc, cs_base, flags, curr_cflags() | CF_LAST_IO | n);
+
/* TODO: If env->pc != tb->pc (i.e. the faulting instruction was not
* the first in the TB) then we end up generating a whole new TB and
* repeating the fault, which is horribly inefficient.
- [Qemu-devel] [PATCH v7 12/22] replay: make locking visible outside replay code, (continued)
- [Qemu-devel] [PATCH v7 12/22] replay: make locking visible outside replay code, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 13/22] replay: push replay_mutex_lock up the call tree, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 14/22] replay: don't destroy mutex at exit, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 15/22] replay: check return values of fwrite, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 16/22] replay: avoid recursive call of checkpoints, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 17/22] scripts/replay-dump.py: replay log dumper, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 18/22] replay: don't process async events when warping the clock, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 19/22] replay: save vmstate of the asynchronous events, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 20/22] replay: don't drain/flush bdrv queue while RR is working, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 21/22] replay: update documentation, Pavel Dovgalyuk, 2018/02/27
- [Qemu-devel] [PATCH v7 22/22] tcg: fix cpu_io_recompile,
Pavel Dovgalyuk <=