]> rtime.felk.cvut.cz Git - lisovros/qemu_apohw.git/commitdiff
audio: honor QEMU_AUDIO_TIMER_PERIOD instead of waking up every *nano* second
authorHans de Goede <hdegoede@redhat.com>
Wed, 9 Oct 2013 19:33:44 +0000 (21:33 +0200)
committerMichael Roth <mdroth@linux.vnet.ibm.com>
Tue, 3 Dec 2013 21:59:25 +0000 (15:59 -0600)
Now that we no longer have MIN_REARM_TIMER_NS a bug in the audio subsys has
clearly shown it self by trying to make a timer fire every nano second.

Note we have a similar problem in 1.6, 1.5 and older but there
MIN_REARM_TIMER_NS limits the wakeups caused by audio being active to
4000 times / second. This still causes a host cpu load of 50 % for simply
playing audio, where as with this patch git master is at 13%, so we should
backport this to 1.5 and 1.6 too.

Note this will not apply to 1.5 and 1.6 as is.

Cc: qemu-stable@nongnu.org
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
(cherry picked from commit b4350deed67b95651896ddb60cf9f765093a4848)

Conflicts:

audio/audio.c

*fixed to reflect 1.6 timer function/clock names

Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
audio/audio.c

index 02bb8861f89ae4e7ba969e4a4b478f8cad28a99c..bcd41a99aa90cd52d12908bb480ba416b09e0662 100644 (file)
@@ -1124,7 +1124,8 @@ static int audio_is_timer_needed (void)
 static void audio_reset_timer (AudioState *s)
 {
     if (audio_is_timer_needed ()) {
-        qemu_mod_timer (s->ts, qemu_get_clock_ns (vm_clock) + 1);
+        qemu_mod_timer (s->ts,
+            qemu_get_clock_ns(vm_clock) + conf.period.ticks);
     }
     else {
         qemu_del_timer (s->ts);