From nobody Fri Nov 7 04:15:01 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=gmail.com Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1545598957101806.1451338652885; Sun, 23 Dec 2018 13:02:37 -0800 (PST) Received: from localhost ([127.0.0.1]:42216 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gbAtP-0005DF-HZ for importer@patchew.org; Sun, 23 Dec 2018 16:02:35 -0500 Received: from eggs.gnu.org ([208.118.235.92]:52198) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gbAju-0007aP-Gf for qemu-devel@nongnu.org; Sun, 23 Dec 2018 15:52:48 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gbAjq-00021E-GP for qemu-devel@nongnu.org; Sun, 23 Dec 2018 15:52:45 -0500 Received: from mail-wm1-x336.google.com ([2a00:1450:4864:20::336]:37497) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gbAjq-0001y5-5M for qemu-devel@nongnu.org; Sun, 23 Dec 2018 15:52:42 -0500 Received: by mail-wm1-x336.google.com with SMTP id g67so10420651wmd.2 for ; Sun, 23 Dec 2018 12:52:42 -0800 (PST) Received: from nullptr.home.dirty-ice.org (2a01-036c-0113-24a3-0000-0000-0000-0005.pool6.digikabel.hu. [2a01:36c:113:24a3::5]) by smtp.gmail.com with ESMTPSA id g198sm25456920wmd.23.2018.12.23.12.52.40 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 23 Dec 2018 12:52:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=vcB7bxTnfRfgqQXaotat+XXmhaSKEHoQVPmLY3YbJ7E=; b=pxWjfR1CLpm7a1rnjRA4Nw/cjseNQNMFmwL2nBumeq68Yevu1uhyjtEUgo1TOluZ24 PAfMwK8NEhx+u9rFMPBQCPy4AjKxhhJ+zb9t/xShTGKUH5RWrht3ig5/B+TreRsHVY7L rz4hp9bBQaRJMvV06fZ+W/0mvuiOGpEj/YWzlu4y9EtOlTMXEb4o8GowRHI3o+w/hgGX h2wRg1JeTXrSX6PoK1n9ZFQaX5MQ8F6nPk3p7QV5PU5YvvU7J9GdmdmIq0qxxVFswuOw 4EueUFqyoLAsrhhziVLwCtqRbSlEttB+HnTLBXAkb41CtrPrHXB3UXTSBU02e+cA6kNs A8Hw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=vcB7bxTnfRfgqQXaotat+XXmhaSKEHoQVPmLY3YbJ7E=; b=SvLiAFwGclIRpUiZuHsdrlleMCQ1axv5UDLNXH7nDZ2m3omKcCqEL+luNR32guEpt+ 8ZeiITfEzvSGh/g0tmJPisAJ7NSQWxdlEHVuRJtxguSda023nBIdFvfL8QccEvej/o85 ShL8QZEkq6+eVuL+mEgzCtV4qBdRnY1K0JNySL8e7aRUNDKtJZQiBRvWSlLjZ3ymfVaf m/J4rIWX31x2pyMc92pKVYQvQAiPyJIiUlK4JTtlZgmjh87ymeGoU5Nf9E1bghW95r5y NxmLot9RE1ooZACHHVJzo27GSz2rF+6020Xr+91LXJJ+BXHM9M7Lph8nt2yvrX3bDonT cZ2g== X-Gm-Message-State: AA+aEWZNiBT2hwvwP5z442FQC6PWtmbzRII7MZ2ky7zWaquRsimLLD0Z R9KclLrrAIZCCQIGvY6iAKYqWFqwenc= X-Google-Smtp-Source: AFSGD/VVCS2AvzNQP8GLZNnX2daBOvHHocs/VGAUmD5CaMfXFDWjsbCP8QcHEYP2Nr2gjoIiURhJ/Q== X-Received: by 2002:a1c:e1d5:: with SMTP id y204mr9942241wmg.65.1545598360800; Sun, 23 Dec 2018 12:52:40 -0800 (PST) From: "=?UTF-8?q?K=C5=91v=C3=A1g=C3=B3=2C=20Zolt=C3=A1n?=" X-Google-Original-From: =?UTF-8?q?K=C5=91v=C3=A1g=C3=B3=2C=20Zolt=C3=A1n?= To: qemu-devel@nongnu.org Date: Sun, 23 Dec 2018 21:51:48 +0100 Message-Id: X-Mailer: git-send-email 2.20.1 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2a00:1450:4864:20::336 Subject: [Qemu-devel] [PATCH v2 12/52] paaudio: port to -audiodev config X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Gerd Hoffmann Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Signed-off-by: K=C5=91v=C3=A1g=C3=B3, Zolt=C3=A1n --- audio/audio_legacy.c | 11 ++++++ audio/paaudio.c | 80 ++++++++++++++++---------------------------- 2 files changed, 40 insertions(+), 51 deletions(-) diff --git a/audio/audio_legacy.c b/audio/audio_legacy.c index f37407586b..1f99d21803 100644 --- a/audio/audio_legacy.c +++ b/audio/audio_legacy.c @@ -96,6 +96,15 @@ SimpleEnvMap oss_map[] =3D { { /* End of list */ } }; =20 +SimpleEnvMap pa_map[] =3D { + { "QEMU_PA_SAMPLES", "buffer", ENV_TRANSFORM_SAMPLES_TO_USECS_OUT }, + { "QEMU_PA_SERVER", "server" }, + { "QEMU_PA_SINK", "sink.name" }, + { "QEMU_PA_SOURCE", "source.name" }, + + { /* End of list */ } +}; + static unsigned long long toull(const char *str) { unsigned long long ret; @@ -272,6 +281,8 @@ static void legacy_opt(const char *drv) handle_env_opts(opts, dsound_map); } else if (strcmp(drv, "oss") =3D=3D 0) { handle_env_opts(opts, oss_map); + } else if (strcmp(drv, "pa") =3D=3D 0) { + handle_env_opts(opts, pa_map); } } =20 diff --git a/audio/paaudio.c b/audio/paaudio.c index 0981f010c9..6de6ed97fc 100644 --- a/audio/paaudio.c +++ b/audio/paaudio.c @@ -2,6 +2,7 @@ #include "qemu/osdep.h" #include "qemu-common.h" #include "audio.h" +#include "qapi/opts-visitor.h" =20 #include =20 @@ -10,14 +11,7 @@ #include "audio_pt_int.h" =20 typedef struct { - int samples; - char *server; - char *sink; - char *source; -} PAConf; - -typedef struct { - PAConf conf; + Audiodev *dev; pa_threaded_mainloop *mainloop; pa_context *context; } paaudio; @@ -32,6 +26,7 @@ typedef struct { void *pcm_buf; struct audio_pt pt; paaudio *g; + int samples; } PAVoiceOut; =20 typedef struct { @@ -46,6 +41,7 @@ typedef struct { const void *read_data; size_t read_index, read_length; paaudio *g; + int samples; } PAVoiceIn; =20 static void qpa_audio_fini(void *opaque); @@ -227,7 +223,7 @@ static void *qpa_thread_out (void *arg) } } =20 - decr =3D to_mix =3D audio_MIN(pa->live, pa->g->conf.samples >> 5); + decr =3D to_mix =3D audio_MIN(pa->live, pa->samples >> 5); rpos =3D pa->rpos; =20 if (audio_pt_unlock(&pa->pt, __func__)) { @@ -319,7 +315,7 @@ static void *qpa_thread_in (void *arg) } } =20 - incr =3D to_grab =3D audio_MIN(pa->dead, pa->g->conf.samples >> 5); + incr =3D to_grab =3D audio_MIN(pa->dead, pa->samples >> 5); wpos =3D pa->wpos; =20 if (audio_pt_unlock(&pa->pt, __func__)) { @@ -546,6 +542,8 @@ static int qpa_init_out(HWVoiceOut *hw, struct audsetti= ngs *as, struct audsettings obt_as =3D *as; PAVoiceOut *pa =3D (PAVoiceOut *) hw; paaudio *g =3D pa->g =3D drv_opaque; + AudiodevPaOptions *popts =3D &g->dev->u.pa; + AudiodevPaPerDirectionOptions *ppdo =3D popts->sink; =20 ss.format =3D audfmt_to_pa (as->fmt, as->endianness); ss.channels =3D as->nchannels; @@ -566,7 +564,7 @@ static int qpa_init_out(HWVoiceOut *hw, struct audsetti= ngs *as, g, "qemu", PA_STREAM_PLAYBACK, - g->conf.sink, + ppdo->has_name ? ppdo->name : NULL, &ss, NULL, /* channel map */ &ba, /* buffering attributes */ @@ -578,7 +576,8 @@ static int qpa_init_out(HWVoiceOut *hw, struct audsetti= ngs *as, } =20 audio_pcm_init_info (&hw->info, &obt_as); - hw->samples =3D g->conf.samples; + hw->samples =3D pa->samples =3D audio_buffer_samples(g->dev->out, &obt= _as, + 46440); pa->pcm_buf =3D audio_calloc(__func__, hw->samples, 1 << hw->info.shif= t); pa->rpos =3D hw->rpos; if (!pa->pcm_buf) { @@ -612,6 +611,8 @@ static int qpa_init_in(HWVoiceIn *hw, struct audsetting= s *as, void *drv_opaque) struct audsettings obt_as =3D *as; PAVoiceIn *pa =3D (PAVoiceIn *) hw; paaudio *g =3D pa->g =3D drv_opaque; + AudiodevPaOptions *popts =3D &g->dev->u.pa; + AudiodevPaPerDirectionOptions *ppdo =3D popts->source; =20 ss.format =3D audfmt_to_pa (as->fmt, as->endianness); ss.channels =3D as->nchannels; @@ -623,7 +624,7 @@ static int qpa_init_in(HWVoiceIn *hw, struct audsetting= s *as, void *drv_opaque) g, "qemu", PA_STREAM_RECORD, - g->conf.source, + ppdo->has_name ? ppdo->name : NULL, &ss, NULL, /* channel map */ NULL, /* buffering attributes */ @@ -635,7 +636,8 @@ static int qpa_init_in(HWVoiceIn *hw, struct audsetting= s *as, void *drv_opaque) } =20 audio_pcm_init_info (&hw->info, &obt_as); - hw->samples =3D g->conf.samples; + hw->samples =3D pa->samples =3D audio_buffer_samples(g->dev->in, &obt_= as, + 46440); pa->pcm_buf =3D audio_calloc(__func__, hw->samples, 1 << hw->info.shif= t); pa->wpos =3D hw->wpos; if (!pa->pcm_buf) { @@ -808,14 +810,19 @@ static int qpa_ctl_in (HWVoiceIn *hw, int cmd, ...) } =20 /* common */ -static PAConf glob_conf =3D { - .samples =3D 4096, -}; - static void *qpa_audio_init(Audiodev *dev) { - paaudio *g =3D g_malloc(sizeof(paaudio)); - g->conf =3D glob_conf; + paaudio *g; + AudiodevPaOptions *popts; + const char *server; + + assert(dev->driver =3D=3D AUDIODEV_DRIVER_PA); + + g =3D g_malloc(sizeof(paaudio)); + popts =3D &dev->u.pa; + server =3D popts->has_server ? popts->server : NULL; + + g->dev =3D dev; g->mainloop =3D NULL; g->context =3D NULL; =20 @@ -825,14 +832,14 @@ static void *qpa_audio_init(Audiodev *dev) } =20 g->context =3D pa_context_new (pa_threaded_mainloop_get_api (g->mainlo= op), - g->conf.server); + server); if (!g->context) { goto fail; } =20 pa_context_set_state_callback (g->context, context_state_cb, g); =20 - if (pa_context_connect (g->context, g->conf.server, 0, NULL) < 0) { + if (pa_context_connect (g->context, server, 0, NULL) < 0) { qpa_logerr (pa_context_errno (g->context), "pa_context_connect() failed\n"); goto fail; @@ -895,34 +902,6 @@ static void qpa_audio_fini (void *opaque) g_free(g); } =20 -struct audio_option qpa_options[] =3D { - { - .name =3D "SAMPLES", - .tag =3D AUD_OPT_INT, - .valp =3D &glob_conf.samples, - .descr =3D "buffer size in samples" - }, - { - .name =3D "SERVER", - .tag =3D AUD_OPT_STR, - .valp =3D &glob_conf.server, - .descr =3D "server address" - }, - { - .name =3D "SINK", - .tag =3D AUD_OPT_STR, - .valp =3D &glob_conf.sink, - .descr =3D "sink device name" - }, - { - .name =3D "SOURCE", - .tag =3D AUD_OPT_STR, - .valp =3D &glob_conf.source, - .descr =3D "source device name" - }, - { /* End of list */ } -}; - static struct audio_pcm_ops qpa_pcm_ops =3D { .init_out =3D qpa_init_out, .fini_out =3D qpa_fini_out, @@ -940,7 +919,6 @@ static struct audio_pcm_ops qpa_pcm_ops =3D { static struct audio_driver pa_audio_driver =3D { .name =3D "pa", .descr =3D "http://www.pulseaudio.org/", - .options =3D qpa_options, .init =3D qpa_audio_init, .fini =3D qpa_audio_fini, .pcm_ops =3D &qpa_pcm_ops, --=20 2.20.1