[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 04/10] linux-user: Add '-one-insn-per-tb' option equivalen
From: |
Peter Maydell |
Subject: |
Re: [PATCH v3 04/10] linux-user: Add '-one-insn-per-tb' option equivalent to '-singlestep' |
Date: |
Thu, 20 Apr 2023 10:19:03 +0100 |
On Thu, 20 Apr 2023 at 10:13, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> On 17/4/23 18:40, Peter Maydell wrote:
> > The '-singlestep' option is confusing, because it doesn't actually
> > have anything to do with single-stepping the CPU. What it does do
> > is force TCG emulation to put one guest instruction in each TB,
> > which can be useful in some situations.
> >
> > Create a new command line argument -one-insn-per-tb, so we can
> > document that -singlestep is just a deprecated synonym for it,
> > and eventually perhaps drop it.
> >
> > Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> > Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> > Reviewed-by: Warner Losh <imp@bsdimp.com>
> > ---
> > docs/user/main.rst | 7 ++++++-
> > linux-user/main.c | 9 ++++++---
> > 2 files changed, 12 insertions(+), 4 deletions(-)
>
>
> > @@ -500,8 +500,11 @@ static const struct qemu_argument arg_table[] = {
> > "logfile", "write logs to 'logfile' (default stderr)"},
> > {"p", "QEMU_PAGESIZE", true, handle_arg_pagesize,
> > "pagesize", "set the host page size to 'pagesize'"},
> > - {"singlestep", "QEMU_SINGLESTEP", false, handle_arg_singlestep,
> > - "", "run in singlestep mode"},
> > + {"one-insn-per-tb",
> > + "QEMU_ONE_INSN_PER_TB", false,
> > handle_arg_one_insn_per_tb,
> > + "", "run with one guest instruction per emulated TB"},
> > + {"singlestep", "QEMU_SINGLESTEP", false, handle_arg_one_insn_per_tb,
> > + "", "deprecated synonym for -one-insn-per-tb"},
>
> Maybe worth mentioning QEMU_ONE_INSN_PER_TB too here:
>
> "deprecated synonyms for -one-insn-per-tb and QEMU_ONE_INSN_PER_TB"
There's not a lot of space in the -help output, and I think in
context it's clear enough without.
thanks
-- PMM
- [PATCH v3 01/10] make one-insn-per-tb an accel option, (continued)
- [PATCH v3 08/10] hmp: Add 'one-insn-per-tb' command equivalent to 'singlestep', Peter Maydell, 2023/04/17
- [PATCH v3 06/10] Document that -singlestep command line option is deprecated, Peter Maydell, 2023/04/17
- [PATCH v3 07/10] accel/tcg: Report one-insn-per-tb in 'info jit', not 'info status', Peter Maydell, 2023/04/17
- [PATCH v3 09/10] qapi/run-state.json: Fix missing newline at end of file, Peter Maydell, 2023/04/17