[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/5] Stop adding HMP-only commands, allow QMP for all
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 0/5] Stop adding HMP-only commands, allow QMP for all |
Date: |
Thu, 09 Sep 2021 10:13:32 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Markus Armbruster <armbru@redhat.com> writes:
[...]
> Limitations:
>
> 1. If we convert a long-running HMP command to this technique, we print
> its output only after it completed its work. We also end up with a
> long-running QMP command, which is bad, because it stops the main
> loop and makes the QMP monitor unresponsive (except for OOB commands,
> if the client is careful). The former can be mitigated with
> 'coroutine': true. The latter can't.
>
> 2. We can't prompt for input.
>
> The only current use I can see is HMP "change vnc passwd" prompting
> for a password. Except you currently have to say "change vnc passwd
> wtf" to get it to prompt (suspect logic error in commit cfb5387a1de).
Subject: [PATCH 1/2] hmp: Unbreak "change vnc"
Message-Id: <20210909081219.308065-2-armbru@redhat.com>
>
>
> [...]
- Re: [PATCH 4/5] qapi: introduce x-query-registers QMP command, (continued)
- [PATCH 5/5] monitor: rewrite 'info registers' in terms of 'x-query-registers', Daniel P . Berrangé, 2021/09/08
- Re: [PATCH 0/5] Stop adding HMP-only commands, allow QMP for all, Ján Tomko, 2021/09/08
- Re: [PATCH 0/5] Stop adding HMP-only commands, allow QMP for all, Markus Armbruster, 2021/09/08
- Re: [PATCH 0/5] Stop adding HMP-only commands, allow QMP for all, Philippe Mathieu-Daudé, 2021/09/08
- Re: [PATCH 0/5] Stop adding HMP-only commands, allow QMP for all, Paolo Bonzini, 2021/09/09