hmp: disable monitor in preconfig state
authorIgor Mammedov <imammedo@redhat.com>
Fri, 4 May 2018 08:37:42 +0000 (10:37 +0200)
committerEduardo Habkost <ehabkost@redhat.com>
Wed, 30 May 2018 16:16:51 +0000 (13:16 -0300)
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 <imammedo@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <1525423069-61903-5-git-send-email-imammedo@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
monitor.c

index 46814af5332c504b6a6d69aa6f5c8a3121a3cf17..9e50418afe8c93d80bccb9731e382cc1ed630f00 100644 (file)
--- a/monitor.c
+++ b/monitor.c
@@ -3371,6 +3371,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;