On 5/28/20 10:37 AM, Kevin Wolf wrote:
cur_mon is updated relatively early in the command handling code even
though only the command handler actually needs it. Move it to
handle_hmp_command().
Signed-off-by: Kevin Wolf <kw...@redhat.com>
---
monitor/hmp.c | 23 ++++++++++++-----------
monitor/misc.c | 7 -------
2 files changed, 12 insertions(+), 18 deletions(-)
+++ b/monitor/misc.c
@@ -258,7 +252,6 @@ static void monitor_init_qmp_commands(void)
/* Set the current CPU defined by the user. Callers must hold BQL. */
int monitor_set_cpu(Monitor *mon, int cpu_index)
{
- Monitor *cur_mon = monitor_cur();
CPUState *cpu;
cpu = qemu_get_cpu(cpu_index);
Bogus churn, will disappear after you fix 2/12. For the rest of the patch:
Reviewed-by: Eric Blake <ebl...@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org