[Qemu-devel] [PATCH 16/25] paaudio: properly disconnect streams in fini_*

[Kővágó, Zoltán]

Currently this needs a workaround due to bug #74624 in pulseaudio.

Signed-off-by: Kővágó, Zoltán <dirty.ice...@gmail.com>
---
 audio/paaudio.c | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/audio/paaudio.c b/audio/paaudio.c
index e3b8207..2aee22f 100644
--- a/audio/paaudio.c
+++ b/audio/paaudio.c
@@ -678,6 +678,25 @@ static int qpa_init_in(HWVoiceIn *hw, struct audsettings 
*as, void *drv_opaque)
     return -1;
 }
 
+static void qpa_simple_disconnect(PAConnection *c, pa_stream *stream)
+{
+    int err;
+
+    pa_threaded_mainloop_lock(c->mainloop);
+    /* wait until actually connects. workaround pa bug #74624
+     * https://bugs.freedesktop.org/show_bug.cgi?id=74624 */
+    while (pa_stream_get_state(stream) == PA_STREAM_CREATING) {
+        pa_threaded_mainloop_wait(c->mainloop);
+    }
+
+    err = pa_stream_disconnect(stream);
+    if (err != 0) {
+        dolog("Failed to dissconnect! err=%d\n", err);
+    }
+    pa_stream_unref(stream);
+    pa_threaded_mainloop_unlock(c->mainloop);
+}
+
 static void qpa_fini_out (HWVoiceOut *hw)
 {
     void *ret;
@@ -689,7 +708,7 @@ static void qpa_fini_out (HWVoiceOut *hw)
     audio_pt_join (&pa->pt, &ret, AUDIO_FUNC);
 
     if (pa->stream) {
-        pa_stream_unref (pa->stream);
+        qpa_simple_disconnect(pa->g->conn, pa->stream);
         pa->stream = NULL;
     }
 
@@ -709,7 +728,7 @@ static void qpa_fini_in (HWVoiceIn *hw)
     audio_pt_join (&pa->pt, &ret, AUDIO_FUNC);
 
     if (pa->stream) {
-        pa_stream_unref (pa->stream);
+        qpa_simple_disconnect(pa->g->conn, pa->stream);
         pa->stream = NULL;
     }
 
-- 
2.4.5