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


Reply via email to