On 10/12/2015 02:41 AM, Denis V. Lunev 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).
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>
---
@@ -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);
In addition to the leak already pointed out, qobject_to_json() can be
expensive, and it looks like we are doing that work unconditionally even
if the logging is not turned on. Is there a way to optimize so that the
conversion is only done when logging is enabled?