Re: [Qemu-devel] [PATCH v2 00/49] audio: -audiodev option, multiple options

[Eric Blake]

On 09/03/2015 04:15 AM, Gerd Hoffmann wrote:
> On Fr, 2015-08-21 at 17:36 +0200, Kővágó, Zoltán wrote:
>> This patch series adds support to multiple audio backends.  Afterwards I
>> add support to multiple backends.  Audio fronteds gain a new audiodev
>> option to specify the id of the audiodev to use.  The audiodev= option
>> is required, unless you use the old environment variable based
>> configuration, in that case it must not used (and you can't use multiple
>> backends). 
>>
>> Finally I also make mixeng usage optional, it can save us some useless
>> format converting when not needed.  Also makes easier to support formats
>> currently not supported by qemu (as only the backend has to support it,
>> not the mixeng).
>>
>> For easier testing pull https://github.com/DirtYiCE/qemu.git tag 
>> audio-multi-v2.
>>
>> Please review.
> 
> Adding qapi & net folks to Cc.  Ping.
> 
> How to go forward with the QAPI bits in this series (Patches 1-9)?
> Patch #1 got reviews from block folks.  Patch #3 has a comment from
> Eduardo pending.  What about the other ones?  Fine as-is?  I remember
> from the pre-2.4 freeze discussions that there at least was agreement
> that flattening the qapi structs is the way to go.

Still on my list of patches to review (seems to be a rather large list,
sadly).  I should get to it before Monday.

> 
> I'd prefer if the qapi maintainers can pick up and merge these patches.
> But I can also merge them via audio queue if I get reviews from the qapi
> maintainers.

I'll leave it up to Markus on which tree is better for the qapi patches
to go through.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature