On 2/9/21 10:27 AM, Alex Bennée wrote: > When icount is enabled and we recompile an MMIO access we end up > double counting the instruction execution. To avoid this we introduce > the CF_NOINSTR cflag which disables instrumentation for the next TB. > As this is part of the hashed compile flags we will only execute the > generated TB while coming out of a cpu_io_recompile. > > While we are at it delete the old TODO. We might as well keep the > translation handy as it's likely you will repeatedly hit it on each > MMIO access. > > Reported-by: Aaron Lindsay <aa...@os.amperecomputing.com> > Signed-off-by: Alex Bennée <alex.ben...@linaro.org> > --- > include/exec/exec-all.h | 3 ++- > accel/tcg/translate-all.c | 17 ++++++++--------- > accel/tcg/translator.c | 2 +- > 3 files changed, 11 insertions(+), 11 deletions(-)
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~