diff options
author | Markus Armbruster <armbru@redhat.com> | 2019-04-17 21:17:54 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2019-04-18 22:18:59 +0200 |
commit | ac7ff4cf5f20eee8cec228209f6b3ca557c60639 (patch) | |
tree | 88938d9d560afb4c6bf5c65d7ee925803e6198d1 /monitor.c | |
parent | 76c8661595fa9414fabf8a164b9adfc93c8a65e2 (diff) | |
download | qemu-ac7ff4cf5f20eee8cec228209f6b3ca557c60639.zip qemu-ac7ff4cf5f20eee8cec228209f6b3ca557c60639.tar.gz qemu-ac7ff4cf5f20eee8cec228209f6b3ca557c60639.tar.bz2 |
qsp: Simplify how qsp_report() prints
qsp_report() takes an fprintf()-like callback and a FILE * to pass to
it.
Its only caller hmp_sync_profile() passes monitor_fprintf() and the
current monitor cast to FILE *. monitor_fprintf() casts it right
back, and is otherwise identical to monitor_printf(). The
type-punning is ugly.
Drop the callback, and call qemu_printf() instead.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20190417191805.28198-7-armbru@redhat.com>
Diffstat (limited to 'monitor.c')
-rw-r--r-- | monitor.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -1336,7 +1336,7 @@ static void hmp_info_sync_profile(Monitor *mon, const QDict *qdict) enum QSPSortBy sort_by; sort_by = mean ? QSP_SORT_BY_AVG_WAIT_TIME : QSP_SORT_BY_TOTAL_WAIT_TIME; - qsp_report((FILE *)mon, monitor_fprintf, max, sort_by, coalesce); + qsp_report(max, sort_by, coalesce); } static void hmp_info_history(Monitor *mon, const QDict *qdict) |