[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig stat
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state |
Date: |
Thu, 24 May 2018 18:00:18 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Igor Mammedov <address@hidden> writes:
> Ban it for now, if someone would need it to work early,
> one would have to implement checks if HMP command is valid
> at preconfig state.
>
> Signed-off-by: Igor Mammedov <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> ---
> v5:
> * add 'use QMP instead" to error message, suggesting user
> the right interface to use
> v4:
> * v3 was only printing error but not preventing command execution,
> Fix it by returning after printing error message.
> ("Dr. David Alan Gilbert" <address@hidden>)
> ---
> monitor.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/monitor.c b/monitor.c
> index 39f8ee1..0ffdf1d 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -3374,6 +3374,12 @@ static void handle_hmp_command(Monitor *mon, const
> char *cmdline)
>
> trace_handle_hmp_command(mon, cmdline);
>
> + if (runstate_check(RUN_STATE_PRECONFIG)) {
> + monitor_printf(mon, "HMP not available in preconfig state, "
> + "use QMP instead\n");
> + return;
> + }
> +
> cmd = monitor_parse_command(mon, cmdline, &cmdline, mon->cmd_table);
> if (!cmd) {
> return;
So we offer the user an HMP monitor, but we summarily fail all commands.
I'm sorry, but that's... searching for polite word... embarrassing. We
should accept HMP output only when we're ready to accept it. Yes, that
would involve a bit more surgery rather than this cheap hack. The whole
preconfig thing smells like a cheap hack to me, but let's not overdo it.
- [Qemu-devel] [PATCH v7 00/11] enable numa configuration before machine_init() from QMP, Igor Mammedov, 2018/05/04
- [Qemu-devel] [PATCH v7 01/11] numa: postpone options post-processing till machine_run_board_init(), Igor Mammedov, 2018/05/04
- [Qemu-devel] [PATCH v7 02/11] numa: split out NumaOptions parsing into set_numa_options(), Igor Mammedov, 2018/05/04
- [Qemu-devel] [PATCH v7 03/11] qapi: introduce preconfig runstate, Igor Mammedov, 2018/05/04
- [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state, Igor Mammedov, 2018/05/04
- Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state, Markus Armbruster, 2018/05/24
- Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state, Eduardo Habkost, 2018/05/24
- Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state, Markus Armbruster, 2018/05/25
- Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state, Eduardo Habkost, 2018/05/25
- Re: [Qemu-devel] [PATCH v7 04/11] hmp: disable monitor in preconfig state, Igor Mammedov, 2018/05/28
[Qemu-devel] [PATCH v7 06/11] tests: qapi-schema tests for allow-preconfig, Igor Mammedov, 2018/05/04
[Qemu-devel] [PATCH v7 05/11] qapi: introduce new cmd option "allowed-in-preconfig", Igor Mammedov, 2018/05/04