yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone

Blame SOURCES/kvm-audio-honor-QEMU_AUDIO_TIMER_PERIOD-instead-of-wakin.patch

0a122b
From 0bdb440567cacd42675c39d97fb19b20e77d93da Mon Sep 17 00:00:00 2001
0a122b
From: Miroslav Rezanina <mrezanin@redhat.com>
0a122b
Date: Wed, 15 Jan 2014 09:14:31 -0500
0a122b
Subject: [PATCH 04/14] audio: honor QEMU_AUDIO_TIMER_PERIOD instead of waking
0a122b
 up every *nano* second.
0a122b
0a122b
Message-id: <23a0c5622fb294ab7cbcf3a812f5ab2f2e47b461.1389776721.git.mrezanin@redhat.com>
0a122b
Patchwork-id: 56719
0a122b
O-Subject: [RHEL7 qemu-kvm PATCH 1/3] audio: honor QEMU_AUDIO_TIMER_PERIOD instead of waking up every *nano* second.
0a122b
Bugzilla: 1017636
0a122b
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
0a122b
RH-Acked-by: Gerd Hoffmann <kraxel@redhat.com>
0a122b
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
0a122b
0a122b
From: Miroslav Rezanina <mrezanin@redhat.com>
0a122b
0a122b
This is manual backport of upstream commit b4350deed:
0a122b
0a122b
    Now that we no longer have MIN_REARM_TIMER_NS a bug in the audio subsys has
0a122b
    clearly shown it self by trying to make a timer fire every nano second.
0a122b
0a122b
    Note we have a similar problem in 1.6, 1.5 and older but there
0a122b
    MIN_REARM_TIMER_NS limits the wakeups caused by audio being active to
0a122b
    4000 times / second. This still causes a host cpu load of 50 % for simply
0a122b
    playing audio, where as with this patch git master is at 13%, so we should
0a122b
    backport this to 1.5 and 1.6 too.
0a122b
0a122b
    Note this will not apply to 1.5 and 1.6 as is.
0a122b
0a122b
    Cc: qemu-stable@nongnu.org
0a122b
    Signed-off-by: Hans de Goede <hdegoede@redhat.com>
0a122b
    Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
0a122b
0a122b
RHEL does not use new timer api so patch couldn't be cherry picked.
0a122b
0a122b
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
0a122b
Upstream: b4350deed67b95651896ddb60cf9f765093a4848
0a122b
---
0a122b
 audio/audio.c | 3 ++-
0a122b
 1 file changed, 2 insertions(+), 1 deletion(-)
0a122b
0a122b
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
0a122b
---
0a122b
 audio/audio.c | 3 ++-
0a122b
 1 file changed, 2 insertions(+), 1 deletion(-)
0a122b
0a122b
diff --git a/audio/audio.c b/audio/audio.c
0a122b
index 02bb886..ebf7d3f 100644
0a122b
--- a/audio/audio.c
0a122b
+++ b/audio/audio.c
0a122b
@@ -1124,7 +1124,8 @@ static int audio_is_timer_needed (void)
0a122b
 static void audio_reset_timer (AudioState *s)
0a122b
 {
0a122b
     if (audio_is_timer_needed ()) {
0a122b
-        qemu_mod_timer (s->ts, qemu_get_clock_ns (vm_clock) + 1);
0a122b
+        qemu_mod_timer (s->ts, 
0a122b
+            qemu_get_clock_ns (vm_clock) + conf.period.ticks);
0a122b
     }
0a122b
     else {
0a122b
         qemu_del_timer (s->ts);
0a122b
-- 
0a122b
1.8.3.1
0a122b