[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 25/40] plugins/lockstep: mention the one-insn-per-tb optio
From: |
Manos Pitsidianakis |
Subject: |
Re: [PATCH v2 25/40] plugins/lockstep: mention the one-insn-per-tb option |
Date: |
Fri, 5 Jul 2024 13:04:26 +0300 |
On Fri, 5 Jul 2024 at 11:48, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> This really helps with lockstep although its super slow on big jobs.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> contrib/plugins/lockstep.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/contrib/plugins/lockstep.c b/contrib/plugins/lockstep.c
> index 761bcdf363..353bf12dfb 100644
> --- a/contrib/plugins/lockstep.c
> +++ b/contrib/plugins/lockstep.c
> @@ -14,7 +14,8 @@
> * particular run may execute the exact same sequence of blocks. An
> * asynchronous event (for example X11 graphics update) may cause a
> * block to end early and a new partial block to start. This means
> - * serial only test cases are a better bet. -d nochain may also help.
> + * serial only test cases are a better bet. -d nochain may also help
> + * as well as -accel tcg,one-insn-per-tb=on
> *
> * This code is not thread safe!
> *
> --
> 2.39.2
>
Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
- [PATCH v2 29/40] accel/tcg: Move qemu_plugin_vcpu_init__async() to plugins/, (continued)
- [PATCH v2 29/40] accel/tcg: Move qemu_plugin_vcpu_init__async() to plugins/, Alex Bennée, 2024/07/05
- [PATCH v2 28/40] plugins: Free CPUPluginState before destroying vCPU state, Alex Bennée, 2024/07/05
- [PATCH v2 30/40] gdbstub: Clean up process_string_cmd, Alex Bennée, 2024/07/05
- [PATCH v2 16/40] tests/tcg/arm: Manually register allocate half-precision numbers, Alex Bennée, 2024/07/05
- [PATCH v2 39/40] gdbstub: Add support for MTE in user mode, Alex Bennée, 2024/07/05
- [PATCH v2 37/40] gdbstub: Pass CPU context to command handler, Alex Bennée, 2024/07/05
- [PATCH v2 35/40] target/arm: Factor out code for setting MTE TCF0 field, Alex Bennée, 2024/07/05
- [PATCH v2 25/40] plugins/lockstep: mention the one-insn-per-tb option, Alex Bennée, 2024/07/05
- Re: [PATCH v2 25/40] plugins/lockstep: mention the one-insn-per-tb option,
Manos Pitsidianakis <=
- [PATCH v2 19/40] linux-user/main: Suppress out-of-range comparison warning for clang, Alex Bennée, 2024/07/05
- [PATCH v2 34/40] target/arm: Make some MTE helpers widely available, Alex Bennée, 2024/07/05
- [PATCH v2 32/40] gdbstub: Add support for target-specific stubs, Alex Bennée, 2024/07/05
- [PATCH v2 26/40] plugins/lockstep: clean-up output, Alex Bennée, 2024/07/05
- [PATCH v2 24/40] plugins/lockstep: make mixed-mode safe, Alex Bennée, 2024/07/05
- [PATCH v2 14/40] tests/tcg/arm: Drop -N from LDFLAGS, Alex Bennée, 2024/07/05
- [PATCH v2 36/40] gdbstub: Make hex conversion function non-internal, Alex Bennée, 2024/07/05
- [PATCH v2 21/40] test/plugin: make insn plugin less noisy by default, Alex Bennée, 2024/07/05
- [PATCH v2 13/40] tests/tcg/arm: Fix fcvt result messages, Alex Bennée, 2024/07/05