From: "Zoltán Kővágó" <dirty.ice.hu@gmail.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: Markus Armbruster <armbru@redhat.com>,
qemu-devel@nongnu.org, Maxim Levitsky <mlevitsk@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v2 4/4] audio: paaudio: ability to specify stream name
Date: Wed, 28 Aug 2019 00:43:49 +0200 [thread overview]
Message-ID: <2fdc616c-0690-a898-0247-765f90153903@gmail.com> (raw)
In-Reply-To: <20190827054223.nuiz7fz27tld3hpt@sirius.home.kraxel.org>
On 2019-08-27 07:42, Gerd Hoffmann wrote:
> On Mon, Aug 26, 2019 at 09:59:04PM +0200, Kővágó, Zoltán wrote:
>> This can be used to identify stream in tools like pavucontrol when one
>> creates multiple -audiodevs or runs multiple qemu instances.
>
> Hmm, can we create an useful name automatically, without yet another
> config option?
>
> Useful choices could be the device name (usb-audio, ...) or the device
> id (whatever -device id=xxx was specified on the command line).
I'm afraid this is not going to work with the current architecture: due
to mixeng even if you have multiple devices, they'll be mixed to a
single stream and the audio backend will only see this one mixed stream.
As a workaround we could do something like concat all device names or
ids, but I don't like that idea.
Alternatively we could use the id of the audiodev instead, and no more
problems with mixeng. However, with mixeng off (implemented in my next
patch series) suddenly soundcards will have suddenly end up as different
streams. (This can be worked around by creating multiple audiodevs,
like what you have to use now to get multiple streams from pa, so this
is probably a smaller problem.)
Currently I'm leaning for the audiodev's id option, unless someone
proposes something better.
Regards,
Zoltan
next prev parent reply other threads:[~2019-08-27 22:44 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-26 19:59 [Qemu-devel] [PATCH v2 0/4] Audio: misc fixes for "Audio 20190821 patches" Kővágó, Zoltán
2019-08-26 19:59 ` [Qemu-devel] [PATCH v2 1/4] audio: fix invalid malloc size in audio_create_pdos Kővágó, Zoltán
2019-08-26 19:59 ` [Qemu-devel] [PATCH v2 2/4] audio: omitting audiodev= parameter is only deprecated Kővágó, Zoltán
2019-08-28 11:53 ` Gerd Hoffmann
2019-08-26 19:59 ` [Qemu-devel] [PATCH v2 3/4] audio: paaudio: fix client name Kővágó, Zoltán
2019-08-27 5:37 ` Gerd Hoffmann
2019-08-27 11:26 ` Maxim Levitsky
2019-08-26 19:59 ` [Qemu-devel] [PATCH v2 4/4] audio: paaudio: ability to specify stream name Kővágó, Zoltán
2019-08-27 5:42 ` Gerd Hoffmann
2019-08-27 22:43 ` Zoltán Kővágó [this message]
2019-08-28 7:33 ` Gerd Hoffmann
2019-08-28 9:12 ` Maxim Levitsky
2019-08-28 9:53 ` Daniel P. Berrangé
2019-08-28 10:14 ` Maxim Levitsky
2019-08-28 10:26 ` Daniel P. Berrangé
2019-08-28 10:39 ` Maxim Levitsky
2019-09-09 23:42 ` Zoltán Kővágó
2019-09-10 7:00 ` Gerd Hoffmann
2019-08-26 21:43 ` [Qemu-devel] [PATCH v2 0/4] Audio: misc fixes for "Audio 20190821 patches" Maxim Levitsky
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2fdc616c-0690-a898-0247-765f90153903@gmail.com \
--to=dirty.ice.hu@gmail.com \
--cc=armbru@redhat.com \
--cc=kraxel@redhat.com \
--cc=mlevitsk@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).