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: Luiz Capitulino
Subject: Re: [Qemu-devel] [PATCH 1/1] log hmp/qmp command
Date: Tue, 13 Oct 2015 09:05:07 -0400

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)

 - A true logging facility would also log responses and events

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

> 
> 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]