main-loop: For tools, initialize timers as part of qemu_init_main_loop()
commitd34e8f6e9d3a396c3327aa9807c83f9e1f4a7bd7
authorMichael Roth <mdroth@linux.vnet.ibm.com>
Sat, 21 Jan 2012 17:13:53 +0000 (21 11:13 -0600)
committerAnthony Liguori <aliguori@us.ibm.com>
Wed, 1 Feb 2012 20:45:02 +0000 (1 14:45 -0600)
tree02c78a7b8b95cf5da36f5a7e6dec710b1444a6ec
parentee77dfb26a5c7972bd7100e910c7427bcb1c8cae
main-loop: For tools, initialize timers as part of qemu_init_main_loop()

In some cases initializing the alarm timers can lead to non-negligable
overhead from programs that link against qemu-tool.o. At least,
setting a max-resolution WinMM alarm timer via mm_start_timer() (the
current default for Windows) can increase the "tick rate" on Windows
OSs and affect frequency scaling, and in the case of tools that run
in guest OSs such has qemu-ga, the impact can be fairly dramatic
(+20%/20% user/sys time on a core 2 processor was observed from an idle
Windows XP guest).

This patch doesn't address the issue directly (not sure what a good
solution would be for Windows, or what other situations it might be
noticeable), but it at least limits the scope of the issue to programs
that "opt-in" to using the main-loop.c functions by only enabling alarm
timers when qemu_init_main_loop() is called, which is already required
to make use of those facilities, so existing users shouldn't be
affected.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
main-loop.c
main-loop.h
qemu-tool.c
vl.c