qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 1/1] log hmp/qmp command


From: Denis V. Lunev
Subject: Re: [Qemu-devel] [PATCH 1/1] log hmp/qmp command
Date: Tue, 13 Oct 2015 16:18:24 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0

On 10/13/2015 04:05 PM, Luiz Capitulino wrote:
On Mon, 12 Oct 2015 11:41:35 +0300
"Denis V. Lunev" <address@hidden> wrote:

From: Pavel Butsykin <address@hidden>

This log would be very welcome for long-term diagnostics of the system
in the production. This log is at least necessary to understand what
has been happened on the system and to identify issues at higher-level
subsystems (libvirt, etc).
This is good to have, and probably we should have had it since day one.
But I have the following considerations:

  - libvirt would need support for this to use it the way you describe,
    although libvirt already has its logging facility (well, one could
    wrap qemu in bash and pass this by default but this is not ideal)
OK, I'll add task to the queue for our libvirt guys

  - A true logging facility would also log responses and events
responses are done alreade internally in the scope of preparations
to the next submission, events are missed. Thank you for pointing
this out

  - I don't think it's important to log HMP

OK, but I do not that that this will hurt us :)

Den

Signed-off-by: Pavel Butsykin <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
CC: Markus Armbruster <address@hidden>
CC: Luiz Capitulino <address@hidden>
CC: Eric Blake <address@hidden>
---
  include/qemu/log.h | 1 +
  monitor.c          | 4 ++++
  qemu-log.c         | 1 +
  3 files changed, 6 insertions(+)

diff --git a/include/qemu/log.h b/include/qemu/log.h
index f880e66..dfb587e 100644
--- a/include/qemu/log.h
+++ b/include/qemu/log.h
@@ -41,6 +41,7 @@ static inline bool qemu_log_enabled(void)
  #define LOG_UNIMP          (1 << 10)
  #define LOG_GUEST_ERROR    (1 << 11)
  #define CPU_LOG_MMU        (1 << 12)
+#define LOG_CMD            (1 << 13)
/* Returns true if a bit is set in the current loglevel mask
   */
diff --git a/monitor.c b/monitor.c
index 4f1ba2f..a22a798 100644
--- a/monitor.c
+++ b/monitor.c
@@ -2848,6 +2848,8 @@ static void handle_hmp_command(Monitor *mon, const char 
*cmdline)
      QDict *qdict;
      const mon_cmd_t *cmd;
+ qemu_log_mask(LOG_CMD, "hmp \"%s\" requested\n", cmdline);
+
      cmd = monitor_parse_command(mon, &cmdline, mon->cmd_table);
      if (!cmd) {
          return;
@@ -3822,6 +3824,8 @@ static void handle_qmp_command(JSONMessageParser *parser, 
QList *tokens)
          error_setg(&local_err, QERR_JSON_PARSING);
          goto err_out;
      }
+    qemu_log_mask(LOG_CMD, "qmp \"%s\" requested\n",
+                  qobject_to_json(obj)->string);
input = qmp_check_input_obj(obj, &local_err);
      if (!input) {
diff --git a/qemu-log.c b/qemu-log.c
index 13f3813..66b15ec 100644
--- a/qemu-log.c
+++ b/qemu-log.c
@@ -119,6 +119,7 @@ const QEMULogItem qemu_log_items[] = {
      { LOG_GUEST_ERROR, "guest_errors",
        "log when the guest OS does something invalid (eg accessing a\n"
        "non-existent register)" },
+    { LOG_CMD, "cmd", "log the hmp/qmp commands execution" },
      { 0, NULL, NULL },
  };




reply via email to

[Prev in Thread] Current Thread [Next in Thread]