Blame 0507-Do-not-use-pa_simple-PulseAudio-API.patch

Hans de Goede 56753f
From 5c8ce128ec78d17a8a0f49115dc07726a3d3f0c1 Mon Sep 17 00:00:00 2001
Hans de Goede 56753f
From: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= <marcandre.lureau@gmail.com>
Hans de Goede 56753f
Date: Tue, 17 Apr 2012 14:32:41 +0200
Hans de Goede 56753f
Subject: [PATCH 507/509] Do not use pa_simple PulseAudio API
Hans de Goede 56753f
Hans de Goede 56753f
Unfortunately, pa_simple is a limited API which doesn't let us
Hans de Goede 56753f
retrieve the associated pa_stream. It is needed to control the volume
Hans de Goede 56753f
of the stream.
Hans de Goede 56753f
Hans de Goede 56753f
In v4:
Hans de Goede 56753f
- add missing braces
Hans de Goede 56753f
Hans de Goede 56753f
Signed-off-by: Marc-Andr? Lureau <marcandre.lureau@redhat.com>
Hans de Goede 56753f
Signed-off-by: malc <av1474@comtv.ru>
Hans de Goede 56753f
---
Hans de Goede 56753f
 audio/paaudio.c |  377 +++++++++++++++++++++++++++++++++++++++++++++++++------
Hans de Goede 56753f
 1 file changed, 339 insertions(+), 38 deletions(-)
Hans de Goede 56753f
Hans de Goede 56753f
diff --git a/audio/paaudio.c b/audio/paaudio.c
Hans de Goede 56753f
index d1f3912..6f50c1c 100644
Hans de Goede 56753f
--- a/audio/paaudio.c
Hans de Goede 56753f
+++ b/audio/paaudio.c
Hans de Goede 56753f
@@ -2,8 +2,7 @@
Hans de Goede 56753f
 #include "qemu-common.h"
Hans de Goede 56753f
 #include "audio.h"
Hans de Goede 56753f
 
Hans de Goede 56753f
-#include <pulse/simple.h>
Hans de Goede 56753f
-#include <pulse/error.h>
Hans de Goede 56753f
+#include <pulse/pulseaudio.h>
Hans de Goede 56753f
 
Hans de Goede 56753f
 #define AUDIO_CAP "pulseaudio"
Hans de Goede 56753f
 #include "audio_int.h"
Hans de Goede 56753f
@@ -15,7 +14,7 @@ typedef struct {
Hans de Goede 56753f
     int live;
Hans de Goede 56753f
     int decr;
Hans de Goede 56753f
     int rpos;
Hans de Goede 56753f
-    pa_simple *s;
Hans de Goede 56753f
+    pa_stream *stream;
Hans de Goede 56753f
     void *pcm_buf;
Hans de Goede 56753f
     struct audio_pt pt;
Hans de Goede 56753f
 } PAVoiceOut;
Hans de Goede 56753f
@@ -26,17 +25,23 @@ typedef struct {
Hans de Goede 56753f
     int dead;
Hans de Goede 56753f
     int incr;
Hans de Goede 56753f
     int wpos;
Hans de Goede 56753f
-    pa_simple *s;
Hans de Goede 56753f
+    pa_stream *stream;
Hans de Goede 56753f
     void *pcm_buf;
Hans de Goede 56753f
     struct audio_pt pt;
Hans de Goede 56753f
+    const void *read_data;
Hans de Goede 56753f
+    size_t read_index, read_length;
Hans de Goede 56753f
 } PAVoiceIn;
Hans de Goede 56753f
 
Hans de Goede 56753f
-static struct {
Hans de Goede 56753f
+typedef struct {
Hans de Goede 56753f
     int samples;
Hans de Goede 56753f
     char *server;
Hans de Goede 56753f
     char *sink;
Hans de Goede 56753f
     char *source;
Hans de Goede 56753f
-} conf = {
Hans de Goede 56753f
+    pa_threaded_mainloop *mainloop;
Hans de Goede 56753f
+    pa_context *context;
Hans de Goede 56753f
+} paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+static paaudio glob_paaudio = {
Hans de Goede 56753f
     .samples = 4096,
Hans de Goede 56753f
 };
Hans de Goede 56753f
 
Hans de Goede 56753f
@@ -51,6 +56,126 @@ static void GCC_FMT_ATTR (2, 3) qpa_logerr (int err, const char *fmt, ...)
Hans de Goede 56753f
     AUD_log (AUDIO_CAP, "Reason: %s\n", pa_strerror (err));
Hans de Goede 56753f
 }
Hans de Goede 56753f
 
Hans de Goede 56753f
+#define CHECK_SUCCESS_GOTO(c, rerror, expression, label)        \
Hans de Goede 56753f
+    do {                                                        \
Hans de Goede 56753f
+        if (!(expression)) {                                    \
Hans de Goede 56753f
+            if (rerror) {                                       \
Hans de Goede 56753f
+                *(rerror) = pa_context_errno ((c)->context);    \
Hans de Goede 56753f
+            }                                                   \
Hans de Goede 56753f
+            goto label;                                         \
Hans de Goede 56753f
+        }                                                       \
Hans de Goede 56753f
+    } while (0);
Hans de Goede 56753f
+
Hans de Goede 56753f
+#define CHECK_DEAD_GOTO(c, stream, rerror, label)                       \
Hans de Goede 56753f
+    do {                                                                \
Hans de Goede 56753f
+        if (!(c)->context || !PA_CONTEXT_IS_GOOD (pa_context_get_state((c)->context)) || \
Hans de Goede 56753f
+            !(stream) || !PA_STREAM_IS_GOOD (pa_stream_get_state ((stream)))) { \
Hans de Goede 56753f
+            if (((c)->context && pa_context_get_state ((c)->context) == PA_CONTEXT_FAILED) || \
Hans de Goede 56753f
+                ((stream) && pa_stream_get_state ((stream)) == PA_STREAM_FAILED)) { \
Hans de Goede 56753f
+                if (rerror) {                                           \
Hans de Goede 56753f
+                    *(rerror) = pa_context_errno ((c)->context);        \
Hans de Goede 56753f
+                }                                                       \
Hans de Goede 56753f
+            } else {                                                    \
Hans de Goede 56753f
+                if (rerror) {                                           \
Hans de Goede 56753f
+                    *(rerror) = PA_ERR_BADSTATE;                        \
Hans de Goede 56753f
+                }                                                       \
Hans de Goede 56753f
+            }                                                           \
Hans de Goede 56753f
+            goto label;                                                 \
Hans de Goede 56753f
+        }                                                               \
Hans de Goede 56753f
+    } while (0);
Hans de Goede 56753f
+
Hans de Goede 56753f
+static int qpa_simple_read (PAVoiceIn *p, void *data, size_t length, int *rerror)
Hans de Goede 56753f
+{
Hans de Goede 56753f
+    paaudio *g = &glob_paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_lock (g->mainloop);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    CHECK_DEAD_GOTO (g, p->stream, rerror, unlock_and_fail);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    while (length > 0) {
Hans de Goede 56753f
+        size_t l;
Hans de Goede 56753f
+
Hans de Goede 56753f
+        while (!p->read_data) {
Hans de Goede 56753f
+            int r;
Hans de Goede 56753f
+
Hans de Goede 56753f
+            r = pa_stream_peek (p->stream, &p->read_data, &p->read_length);
Hans de Goede 56753f
+            CHECK_SUCCESS_GOTO (g, rerror, r == 0, unlock_and_fail);
Hans de Goede 56753f
+
Hans de Goede 56753f
+            if (!p->read_data) {
Hans de Goede 56753f
+                pa_threaded_mainloop_wait (g->mainloop);
Hans de Goede 56753f
+                CHECK_DEAD_GOTO (g, p->stream, rerror, unlock_and_fail);
Hans de Goede 56753f
+            } else {
Hans de Goede 56753f
+                p->read_index = 0;
Hans de Goede 56753f
+            }
Hans de Goede 56753f
+        }
Hans de Goede 56753f
+
Hans de Goede 56753f
+        l = p->read_length < length ? p->read_length : length;
Hans de Goede 56753f
+        memcpy (data, (const uint8_t *) p->read_data+p->read_index, l);
Hans de Goede 56753f
+
Hans de Goede 56753f
+        data = (uint8_t *) data + l;
Hans de Goede 56753f
+        length -= l;
Hans de Goede 56753f
+
Hans de Goede 56753f
+        p->read_index += l;
Hans de Goede 56753f
+        p->read_length -= l;
Hans de Goede 56753f
+
Hans de Goede 56753f
+        if (!p->read_length) {
Hans de Goede 56753f
+            int r;
Hans de Goede 56753f
+
Hans de Goede 56753f
+            r = pa_stream_drop (p->stream);
Hans de Goede 56753f
+            p->read_data = NULL;
Hans de Goede 56753f
+            p->read_length = 0;
Hans de Goede 56753f
+            p->read_index = 0;
Hans de Goede 56753f
+
Hans de Goede 56753f
+            CHECK_SUCCESS_GOTO (g, rerror, r == 0, unlock_and_fail);
Hans de Goede 56753f
+        }
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+    return 0;
Hans de Goede 56753f
+
Hans de Goede 56753f
+unlock_and_fail:
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+    return -1;
Hans de Goede 56753f
+}
Hans de Goede 56753f
+
Hans de Goede 56753f
+static int qpa_simple_write (PAVoiceOut *p, const void *data, size_t length, int *rerror)
Hans de Goede 56753f
+{
Hans de Goede 56753f
+    paaudio *g = &glob_paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_lock (g->mainloop);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    CHECK_DEAD_GOTO (g, p->stream, rerror, unlock_and_fail);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    while (length > 0) {
Hans de Goede 56753f
+        size_t l;
Hans de Goede 56753f
+        int r;
Hans de Goede 56753f
+
Hans de Goede 56753f
+        while (!(l = pa_stream_writable_size (p->stream))) {
Hans de Goede 56753f
+            pa_threaded_mainloop_wait (g->mainloop);
Hans de Goede 56753f
+            CHECK_DEAD_GOTO (g, p->stream, rerror, unlock_and_fail);
Hans de Goede 56753f
+        }
Hans de Goede 56753f
+
Hans de Goede 56753f
+        CHECK_SUCCESS_GOTO (g, rerror, l != (size_t) -1, unlock_and_fail);
Hans de Goede 56753f
+
Hans de Goede 56753f
+        if (l > length) {
Hans de Goede 56753f
+            l = length;
Hans de Goede 56753f
+        }
Hans de Goede 56753f
+
Hans de Goede 56753f
+        r = pa_stream_write (p->stream, data, l, NULL, 0LL, PA_SEEK_RELATIVE);
Hans de Goede 56753f
+        CHECK_SUCCESS_GOTO (g, rerror, r >= 0, unlock_and_fail);
Hans de Goede 56753f
+
Hans de Goede 56753f
+        data = (const uint8_t *) data + l;
Hans de Goede 56753f
+        length -= l;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+    return 0;
Hans de Goede 56753f
+
Hans de Goede 56753f
+unlock_and_fail:
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+    return -1;
Hans de Goede 56753f
+}
Hans de Goede 56753f
+
Hans de Goede 56753f
 static void *qpa_thread_out (void *arg)
Hans de Goede 56753f
 {
Hans de Goede 56753f
     PAVoiceOut *pa = arg;
Hans de Goede 56753f
@@ -77,7 +202,7 @@ static void *qpa_thread_out (void *arg)
Hans de Goede 56753f
             }
Hans de Goede 56753f
         }
Hans de Goede 56753f
 
Hans de Goede 56753f
-        decr = to_mix = audio_MIN (pa->live, conf.samples >> 2);
Hans de Goede 56753f
+        decr = to_mix = audio_MIN (pa->live, glob_paaudio.samples >> 2);
Hans de Goede 56753f
         rpos = pa->rpos;
Hans de Goede 56753f
 
Hans de Goede 56753f
         if (audio_pt_unlock (&pa->pt, AUDIO_FUNC)) {
Hans de Goede 56753f
@@ -91,8 +216,8 @@ static void *qpa_thread_out (void *arg)
Hans de Goede 56753f
 
Hans de Goede 56753f
             hw->clip (pa->pcm_buf, src, chunk);
Hans de Goede 56753f
 
Hans de Goede 56753f
-            if (pa_simple_write (pa->s, pa->pcm_buf,
Hans de Goede 56753f
-                                 chunk << hw->info.shift, &error) < 0) {
Hans de Goede 56753f
+            if (qpa_simple_write (pa, pa->pcm_buf,
Hans de Goede 56753f
+                                  chunk << hw->info.shift, &error) < 0) {
Hans de Goede 56753f
                 qpa_logerr (error, "pa_simple_write failed\n");
Hans de Goede 56753f
                 return NULL;
Hans de Goede 56753f
             }
Hans de Goede 56753f
@@ -169,7 +294,7 @@ static void *qpa_thread_in (void *arg)
Hans de Goede 56753f
             }
Hans de Goede 56753f
         }
Hans de Goede 56753f
 
Hans de Goede 56753f
-        incr = to_grab = audio_MIN (pa->dead, conf.samples >> 2);
Hans de Goede 56753f
+        incr = to_grab = audio_MIN (pa->dead, glob_paaudio.samples >> 2);
Hans de Goede 56753f
         wpos = pa->wpos;
Hans de Goede 56753f
 
Hans de Goede 56753f
         if (audio_pt_unlock (&pa->pt, AUDIO_FUNC)) {
Hans de Goede 56753f
@@ -181,8 +306,8 @@ static void *qpa_thread_in (void *arg)
Hans de Goede 56753f
             int chunk = audio_MIN (to_grab, hw->samples - wpos);
Hans de Goede 56753f
             void *buf = advance (pa->pcm_buf, wpos);
Hans de Goede 56753f
 
Hans de Goede 56753f
-            if (pa_simple_read (pa->s, buf,
Hans de Goede 56753f
-                                chunk << hw->info.shift, &error) < 0) {
Hans de Goede 56753f
+            if (qpa_simple_read (pa, buf,
Hans de Goede 56753f
+                                 chunk << hw->info.shift, &error) < 0) {
Hans de Goede 56753f
                 qpa_logerr (error, "pa_simple_read failed\n");
Hans de Goede 56753f
                 return NULL;
Hans de Goede 56753f
             }
Hans de Goede 56753f
@@ -283,6 +408,109 @@ static audfmt_e pa_to_audfmt (pa_sample_format_t fmt, int *endianness)
Hans de Goede 56753f
     }
Hans de Goede 56753f
 }
Hans de Goede 56753f
 
Hans de Goede 56753f
+static void context_state_cb (pa_context *c, void *userdata)
Hans de Goede 56753f
+{
Hans de Goede 56753f
+    paaudio *g = &glob_paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    switch (pa_context_get_state(c)) {
Hans de Goede 56753f
+    case PA_CONTEXT_READY:
Hans de Goede 56753f
+    case PA_CONTEXT_TERMINATED:
Hans de Goede 56753f
+    case PA_CONTEXT_FAILED:
Hans de Goede 56753f
+        pa_threaded_mainloop_signal (g->mainloop, 0);
Hans de Goede 56753f
+        break;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    case PA_CONTEXT_UNCONNECTED:
Hans de Goede 56753f
+    case PA_CONTEXT_CONNECTING:
Hans de Goede 56753f
+    case PA_CONTEXT_AUTHORIZING:
Hans de Goede 56753f
+    case PA_CONTEXT_SETTING_NAME:
Hans de Goede 56753f
+        break;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+}
Hans de Goede 56753f
+
Hans de Goede 56753f
+static void stream_state_cb (pa_stream *s, void * userdata)
Hans de Goede 56753f
+{
Hans de Goede 56753f
+    paaudio *g = &glob_paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    switch (pa_stream_get_state (s)) {
Hans de Goede 56753f
+
Hans de Goede 56753f
+    case PA_STREAM_READY:
Hans de Goede 56753f
+    case PA_STREAM_FAILED:
Hans de Goede 56753f
+    case PA_STREAM_TERMINATED:
Hans de Goede 56753f
+        pa_threaded_mainloop_signal (g->mainloop, 0);
Hans de Goede 56753f
+        break;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    case PA_STREAM_UNCONNECTED:
Hans de Goede 56753f
+    case PA_STREAM_CREATING:
Hans de Goede 56753f
+        break;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+}
Hans de Goede 56753f
+
Hans de Goede 56753f
+static void stream_request_cb (pa_stream *s, size_t length, void *userdata)
Hans de Goede 56753f
+{
Hans de Goede 56753f
+    paaudio *g = &glob_paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_signal (g->mainloop, 0);
Hans de Goede 56753f
+}
Hans de Goede 56753f
+
Hans de Goede 56753f
+static pa_stream *qpa_simple_new (
Hans de Goede 56753f
+        const char *server,
Hans de Goede 56753f
+        const char *name,
Hans de Goede 56753f
+        pa_stream_direction_t dir,
Hans de Goede 56753f
+        const char *dev,
Hans de Goede 56753f
+        const char *stream_name,
Hans de Goede 56753f
+        const pa_sample_spec *ss,
Hans de Goede 56753f
+        const pa_channel_map *map,
Hans de Goede 56753f
+        const pa_buffer_attr *attr,
Hans de Goede 56753f
+        int *rerror)
Hans de Goede 56753f
+{
Hans de Goede 56753f
+    paaudio *g = &glob_paaudio;
Hans de Goede 56753f
+    int r;
Hans de Goede 56753f
+    pa_stream *stream;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_lock (g->mainloop);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    stream = pa_stream_new (g->context, name, ss, map);
Hans de Goede 56753f
+    if (!stream) {
Hans de Goede 56753f
+        goto fail;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_stream_set_state_callback (stream, stream_state_cb, g);
Hans de Goede 56753f
+    pa_stream_set_read_callback (stream, stream_request_cb, g);
Hans de Goede 56753f
+    pa_stream_set_write_callback (stream, stream_request_cb, g);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (dir == PA_STREAM_PLAYBACK) {
Hans de Goede 56753f
+        r = pa_stream_connect_playback (stream, dev, attr,
Hans de Goede 56753f
+                                        PA_STREAM_INTERPOLATE_TIMING
Hans de Goede 56753f
+                                        |PA_STREAM_ADJUST_LATENCY
Hans de Goede 56753f
+                                        |PA_STREAM_AUTO_TIMING_UPDATE, NULL, NULL);
Hans de Goede 56753f
+    } else {
Hans de Goede 56753f
+        r = pa_stream_connect_record (stream, dev, attr,
Hans de Goede 56753f
+                                      PA_STREAM_INTERPOLATE_TIMING
Hans de Goede 56753f
+                                      |PA_STREAM_ADJUST_LATENCY
Hans de Goede 56753f
+                                      |PA_STREAM_AUTO_TIMING_UPDATE);
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (r < 0) {
Hans de Goede 56753f
+      goto fail;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    return stream;
Hans de Goede 56753f
+
Hans de Goede 56753f
+fail:
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (stream) {
Hans de Goede 56753f
+        pa_stream_unref (stream);
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    qpa_logerr (pa_context_errno (g->context),
Hans de Goede 56753f
+                "stream_new() failed\n");
Hans de Goede 56753f
+
Hans de Goede 56753f
+    return NULL;
Hans de Goede 56753f
+}
Hans de Goede 56753f
+
Hans de Goede 56753f
 static int qpa_init_out (HWVoiceOut *hw, struct audsettings *as)
Hans de Goede 56753f
 {
Hans de Goede 56753f
     int error;
Hans de Goede 56753f
@@ -306,24 +534,24 @@ static int qpa_init_out (HWVoiceOut *hw, struct audsettings *as)
Hans de Goede 56753f
 
Hans de Goede 56753f
     obt_as.fmt = pa_to_audfmt (ss.format, &obt_as.endianness);
Hans de Goede 56753f
 
Hans de Goede 56753f
-    pa->s = pa_simple_new (
Hans de Goede 56753f
-        conf.server,
Hans de Goede 56753f
+    pa->stream = qpa_simple_new (
Hans de Goede 56753f
+        glob_paaudio.server,
Hans de Goede 56753f
         "qemu",
Hans de Goede 56753f
         PA_STREAM_PLAYBACK,
Hans de Goede 56753f
-        conf.sink,
Hans de Goede 56753f
+        glob_paaudio.sink,
Hans de Goede 56753f
         "pcm.playback",
Hans de Goede 56753f
         &ss,
Hans de Goede 56753f
         NULL,                   /* channel map */
Hans de Goede 56753f
         &ba,                    /* buffering attributes */
Hans de Goede 56753f
         &error
Hans de Goede 56753f
         );
Hans de Goede 56753f
-    if (!pa->s) {
Hans de Goede 56753f
+    if (!pa->stream) {
Hans de Goede 56753f
         qpa_logerr (error, "pa_simple_new for playback failed\n");
Hans de Goede 56753f
         goto fail1;
Hans de Goede 56753f
     }
Hans de Goede 56753f
 
Hans de Goede 56753f
     audio_pcm_init_info (&hw->info, &obt_as);
Hans de Goede 56753f
-    hw->samples = conf.samples;
Hans de Goede 56753f
+    hw->samples = glob_paaudio.samples;
Hans de Goede 56753f
     pa->pcm_buf = audio_calloc (AUDIO_FUNC, hw->samples, 1 << hw->info.shift);
Hans de Goede 56753f
     pa->rpos = hw->rpos;
Hans de Goede 56753f
     if (!pa->pcm_buf) {
Hans de Goede 56753f
@@ -342,8 +570,10 @@ static int qpa_init_out (HWVoiceOut *hw, struct audsettings *as)
Hans de Goede 56753f
     g_free (pa->pcm_buf);
Hans de Goede 56753f
     pa->pcm_buf = NULL;
Hans de Goede 56753f
  fail2:
Hans de Goede 56753f
-    pa_simple_free (pa->s);
Hans de Goede 56753f
-    pa->s = NULL;
Hans de Goede 56753f
+    if (pa->stream) {
Hans de Goede 56753f
+        pa_stream_unref (pa->stream);
Hans de Goede 56753f
+        pa->stream = NULL;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
  fail1:
Hans de Goede 56753f
     return -1;
Hans de Goede 56753f
 }
Hans de Goede 56753f
@@ -361,24 +591,24 @@ static int qpa_init_in (HWVoiceIn *hw, struct audsettings *as)
Hans de Goede 56753f
 
Hans de Goede 56753f
     obt_as.fmt = pa_to_audfmt (ss.format, &obt_as.endianness);
Hans de Goede 56753f
 
Hans de Goede 56753f
-    pa->s = pa_simple_new (
Hans de Goede 56753f
-        conf.server,
Hans de Goede 56753f
+    pa->stream = qpa_simple_new (
Hans de Goede 56753f
+        glob_paaudio.server,
Hans de Goede 56753f
         "qemu",
Hans de Goede 56753f
         PA_STREAM_RECORD,
Hans de Goede 56753f
-        conf.source,
Hans de Goede 56753f
+        glob_paaudio.source,
Hans de Goede 56753f
         "pcm.capture",
Hans de Goede 56753f
         &ss,
Hans de Goede 56753f
         NULL,                   /* channel map */
Hans de Goede 56753f
         NULL,                   /* buffering attributes */
Hans de Goede 56753f
         &error
Hans de Goede 56753f
         );
Hans de Goede 56753f
-    if (!pa->s) {
Hans de Goede 56753f
+    if (!pa->stream) {
Hans de Goede 56753f
         qpa_logerr (error, "pa_simple_new for capture failed\n");
Hans de Goede 56753f
         goto fail1;
Hans de Goede 56753f
     }
Hans de Goede 56753f
 
Hans de Goede 56753f
     audio_pcm_init_info (&hw->info, &obt_as);
Hans de Goede 56753f
-    hw->samples = conf.samples;
Hans de Goede 56753f
+    hw->samples = glob_paaudio.samples;
Hans de Goede 56753f
     pa->pcm_buf = audio_calloc (AUDIO_FUNC, hw->samples, 1 << hw->info.shift);
Hans de Goede 56753f
     pa->wpos = hw->wpos;
Hans de Goede 56753f
     if (!pa->pcm_buf) {
Hans de Goede 56753f
@@ -397,8 +627,10 @@ static int qpa_init_in (HWVoiceIn *hw, struct audsettings *as)
Hans de Goede 56753f
     g_free (pa->pcm_buf);
Hans de Goede 56753f
     pa->pcm_buf = NULL;
Hans de Goede 56753f
  fail2:
Hans de Goede 56753f
-    pa_simple_free (pa->s);
Hans de Goede 56753f
-    pa->s = NULL;
Hans de Goede 56753f
+    if (pa->stream) {
Hans de Goede 56753f
+        pa_stream_unref (pa->stream);
Hans de Goede 56753f
+        pa->stream = NULL;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
  fail1:
Hans de Goede 56753f
     return -1;
Hans de Goede 56753f
 }
Hans de Goede 56753f
@@ -413,9 +645,9 @@ static void qpa_fini_out (HWVoiceOut *hw)
Hans de Goede 56753f
     audio_pt_unlock_and_signal (&pa->pt, AUDIO_FUNC);
Hans de Goede 56753f
     audio_pt_join (&pa->pt, &ret, AUDIO_FUNC);
Hans de Goede 56753f
 
Hans de Goede 56753f
-    if (pa->s) {
Hans de Goede 56753f
-        pa_simple_free (pa->s);
Hans de Goede 56753f
-        pa->s = NULL;
Hans de Goede 56753f
+    if (pa->stream) {
Hans de Goede 56753f
+        pa_stream_unref (pa->stream);
Hans de Goede 56753f
+        pa->stream = NULL;
Hans de Goede 56753f
     }
Hans de Goede 56753f
 
Hans de Goede 56753f
     audio_pt_fini (&pa->pt, AUDIO_FUNC);
Hans de Goede 56753f
@@ -433,9 +665,9 @@ static void qpa_fini_in (HWVoiceIn *hw)
Hans de Goede 56753f
     audio_pt_unlock_and_signal (&pa->pt, AUDIO_FUNC);
Hans de Goede 56753f
     audio_pt_join (&pa->pt, &ret, AUDIO_FUNC);
Hans de Goede 56753f
 
Hans de Goede 56753f
-    if (pa->s) {
Hans de Goede 56753f
-        pa_simple_free (pa->s);
Hans de Goede 56753f
-        pa->s = NULL;
Hans de Goede 56753f
+    if (pa->stream) {
Hans de Goede 56753f
+        pa_stream_unref (pa->stream);
Hans de Goede 56753f
+        pa->stream = NULL;
Hans de Goede 56753f
     }
Hans de Goede 56753f
 
Hans de Goede 56753f
     audio_pt_fini (&pa->pt, AUDIO_FUNC);
Hans de Goede 56753f
@@ -460,37 +692,106 @@ static int qpa_ctl_in (HWVoiceIn *hw, int cmd, ...)
Hans de Goede 56753f
 /* common */
Hans de Goede 56753f
 static void *qpa_audio_init (void)
Hans de Goede 56753f
 {
Hans de Goede 56753f
-    return &conf;
Hans de Goede 56753f
+    paaudio *g = &glob_paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    g->mainloop = pa_threaded_mainloop_new ();
Hans de Goede 56753f
+    if (!g->mainloop) {
Hans de Goede 56753f
+        goto fail;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    g->context = pa_context_new (pa_threaded_mainloop_get_api (g->mainloop), glob_paaudio.server);
Hans de Goede 56753f
+    if (!g->context) {
Hans de Goede 56753f
+        goto fail;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_context_set_state_callback (g->context, context_state_cb, g);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (pa_context_connect (g->context, glob_paaudio.server, 0, NULL) < 0) {
Hans de Goede 56753f
+        qpa_logerr (pa_context_errno (g->context),
Hans de Goede 56753f
+                    "pa_context_connect() failed\n");
Hans de Goede 56753f
+        goto fail;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_lock (g->mainloop);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (pa_threaded_mainloop_start (g->mainloop) < 0) {
Hans de Goede 56753f
+        goto unlock_and_fail;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    for (;;) {
Hans de Goede 56753f
+        pa_context_state_t state;
Hans de Goede 56753f
+
Hans de Goede 56753f
+        state = pa_context_get_state (g->context);
Hans de Goede 56753f
+
Hans de Goede 56753f
+        if (state == PA_CONTEXT_READY) {
Hans de Goede 56753f
+            break;
Hans de Goede 56753f
+        }
Hans de Goede 56753f
+
Hans de Goede 56753f
+        if (!PA_CONTEXT_IS_GOOD (state)) {
Hans de Goede 56753f
+            qpa_logerr (pa_context_errno (g->context),
Hans de Goede 56753f
+                        "Wrong context state\n");
Hans de Goede 56753f
+            goto unlock_and_fail;
Hans de Goede 56753f
+        }
Hans de Goede 56753f
+
Hans de Goede 56753f
+        /* Wait until the context is ready */
Hans de Goede 56753f
+        pa_threaded_mainloop_wait (g->mainloop);
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+
Hans de Goede 56753f
+    return &glob_paaudio;
Hans de Goede 56753f
+
Hans de Goede 56753f
+unlock_and_fail:
Hans de Goede 56753f
+    pa_threaded_mainloop_unlock (g->mainloop);
Hans de Goede 56753f
+fail:
Hans de Goede 56753f
+    AUD_log (AUDIO_CAP, "Failed to initialize PA context");
Hans de Goede 56753f
+    return NULL;
Hans de Goede 56753f
 }
Hans de Goede 56753f
 
Hans de Goede 56753f
 static void qpa_audio_fini (void *opaque)
Hans de Goede 56753f
 {
Hans de Goede 56753f
-    (void) opaque;
Hans de Goede 56753f
+    paaudio *g = opaque;
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (g->mainloop) {
Hans de Goede 56753f
+        pa_threaded_mainloop_stop (g->mainloop);
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (g->context) {
Hans de Goede 56753f
+        pa_context_disconnect (g->context);
Hans de Goede 56753f
+        pa_context_unref (g->context);
Hans de Goede 56753f
+        g->context = NULL;
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    if (g->mainloop) {
Hans de Goede 56753f
+        pa_threaded_mainloop_free (g->mainloop);
Hans de Goede 56753f
+    }
Hans de Goede 56753f
+
Hans de Goede 56753f
+    g->mainloop = NULL;
Hans de Goede 56753f
 }
Hans de Goede 56753f
 
Hans de Goede 56753f
 struct audio_option qpa_options[] = {
Hans de Goede 56753f
     {
Hans de Goede 56753f
         .name  = "SAMPLES",
Hans de Goede 56753f
         .tag   = AUD_OPT_INT,
Hans de Goede 56753f
-        .valp  = &conf.samples,
Hans de Goede 56753f
+        .valp  = &glob_paaudio.samples,
Hans de Goede 56753f
         .descr = "buffer size in samples"
Hans de Goede 56753f
     },
Hans de Goede 56753f
     {
Hans de Goede 56753f
         .name  = "SERVER",
Hans de Goede 56753f
         .tag   = AUD_OPT_STR,
Hans de Goede 56753f
-        .valp  = &conf.server,
Hans de Goede 56753f
+        .valp  = &glob_paaudio.server,
Hans de Goede 56753f
         .descr = "server address"
Hans de Goede 56753f
     },
Hans de Goede 56753f
     {
Hans de Goede 56753f
         .name  = "SINK",
Hans de Goede 56753f
         .tag   = AUD_OPT_STR,
Hans de Goede 56753f
-        .valp  = &conf.sink,
Hans de Goede 56753f
+        .valp  = &glob_paaudio.sink,
Hans de Goede 56753f
         .descr = "sink device name"
Hans de Goede 56753f
     },
Hans de Goede 56753f
     {
Hans de Goede 56753f
         .name  = "SOURCE",
Hans de Goede 56753f
         .tag   = AUD_OPT_STR,
Hans de Goede 56753f
-        .valp  = &conf.source,
Hans de Goede 56753f
+        .valp  = &glob_paaudio.source,
Hans de Goede 56753f
         .descr = "source device name"
Hans de Goede 56753f
     },
Hans de Goede 56753f
     { /* End of list */ }
Hans de Goede 56753f
-- 
Hans de Goede 56753f
1.7.10
Hans de Goede 56753f