qapi: introduce x-query-profile QMP command
This is a counterpart to the HMP "info profile" command. It is being added with an "x-" prefix because this QMP command is intended as an adhoc debugging tool and will thus not be modelled in QAPI as fully structured data, nor will it have long term guaranteed stability. The existing HMP command is rewritten to call the QMP command. Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
parent
dd98234c05
commit
37087fde0e
@ -363,7 +363,7 @@ ERST
|
||||
.args_type = "",
|
||||
.params = "",
|
||||
.help = "show profiling information",
|
||||
.cmd = hmp_info_profile,
|
||||
.cmd_info_hrt = qmp_x_query_profile,
|
||||
},
|
||||
|
||||
SRST
|
||||
|
@ -931,33 +931,6 @@ static void hmp_info_mtree(Monitor *mon, const QDict *qdict)
|
||||
mtree_info(flatview, dispatch_tree, owner, disabled);
|
||||
}
|
||||
|
||||
#ifdef CONFIG_PROFILER
|
||||
|
||||
int64_t dev_time;
|
||||
|
||||
static void hmp_info_profile(Monitor *mon, const QDict *qdict)
|
||||
{
|
||||
static int64_t last_cpu_exec_time;
|
||||
int64_t cpu_exec_time;
|
||||
int64_t delta;
|
||||
|
||||
cpu_exec_time = tcg_cpu_exec_time();
|
||||
delta = cpu_exec_time - last_cpu_exec_time;
|
||||
|
||||
monitor_printf(mon, "async time %" PRId64 " (%0.3f)\n",
|
||||
dev_time, dev_time / (double)NANOSECONDS_PER_SECOND);
|
||||
monitor_printf(mon, "qemu time %" PRId64 " (%0.3f)\n",
|
||||
delta, delta / (double)NANOSECONDS_PER_SECOND);
|
||||
last_cpu_exec_time = cpu_exec_time;
|
||||
dev_time = 0;
|
||||
}
|
||||
#else
|
||||
static void hmp_info_profile(Monitor *mon, const QDict *qdict)
|
||||
{
|
||||
monitor_printf(mon, "Internal profiler not compiled\n");
|
||||
}
|
||||
#endif
|
||||
|
||||
/* Capture support */
|
||||
static QLIST_HEAD (capture_list_head, CaptureState) capture_head;
|
||||
|
||||
|
@ -36,6 +36,7 @@
|
||||
#include "qapi/qapi-commands-machine.h"
|
||||
#include "qapi/qapi-commands-misc.h"
|
||||
#include "qapi/qapi-commands-ui.h"
|
||||
#include "qapi/type-helpers.h"
|
||||
#include "qapi/qmp/qerror.h"
|
||||
#include "hw/mem/memory-device.h"
|
||||
#include "hw/acpi/acpi_dev_interface.h"
|
||||
@ -350,3 +351,34 @@ void qmp_display_reload(DisplayReloadOptions *arg, Error **errp)
|
||||
abort();
|
||||
}
|
||||
}
|
||||
|
||||
#ifdef CONFIG_PROFILER
|
||||
|
||||
int64_t dev_time;
|
||||
|
||||
HumanReadableText *qmp_x_query_profile(Error **errp)
|
||||
{
|
||||
g_autoptr(GString) buf = g_string_new("");
|
||||
static int64_t last_cpu_exec_time;
|
||||
int64_t cpu_exec_time;
|
||||
int64_t delta;
|
||||
|
||||
cpu_exec_time = tcg_cpu_exec_time();
|
||||
delta = cpu_exec_time - last_cpu_exec_time;
|
||||
|
||||
g_string_append_printf(buf, "async time %" PRId64 " (%0.3f)\n",
|
||||
dev_time, dev_time / (double)NANOSECONDS_PER_SECOND);
|
||||
g_string_append_printf(buf, "qemu time %" PRId64 " (%0.3f)\n",
|
||||
delta, delta / (double)NANOSECONDS_PER_SECOND);
|
||||
last_cpu_exec_time = cpu_exec_time;
|
||||
dev_time = 0;
|
||||
|
||||
return human_readable_text_from_str(buf);
|
||||
}
|
||||
#else
|
||||
HumanReadableText *qmp_x_query_profile(Error **errp)
|
||||
{
|
||||
error_setg(errp, "Internal profiler not compiled");
|
||||
return NULL;
|
||||
}
|
||||
#endif
|
||||
|
@ -1412,6 +1412,18 @@
|
||||
'*threads': 'int',
|
||||
'*maxcpus': 'int' } }
|
||||
|
||||
##
|
||||
# @x-query-profile:
|
||||
#
|
||||
# Query TCG profiling information
|
||||
#
|
||||
# Returns: profile information
|
||||
#
|
||||
# Since: 6.2
|
||||
##
|
||||
{ 'command': 'x-query-profile',
|
||||
'returns': 'HumanReadableText' }
|
||||
|
||||
##
|
||||
# @x-query-roms:
|
||||
#
|
||||
|
@ -46,6 +46,9 @@ static int query_error_class(const char *cmd)
|
||||
{ "query-balloon", ERROR_CLASS_DEVICE_NOT_ACTIVE },
|
||||
{ "query-hotpluggable-cpus", ERROR_CLASS_GENERIC_ERROR },
|
||||
{ "query-vm-generation-id", ERROR_CLASS_GENERIC_ERROR },
|
||||
#ifndef CONFIG_PROFILER
|
||||
{ "x-query-profile", ERROR_CLASS_GENERIC_ERROR },
|
||||
#endif
|
||||
{ NULL, -1 }
|
||||
};
|
||||
int i;
|
||||
|
Loading…
Reference in New Issue
Block a user