[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3)
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3) |
Date: |
Tue, 13 Mar 2012 11:34:26 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120216 Thunderbird/10.0.1 |
On 03/13/12 11:33, Kevin Wolf wrote:
> Am 13.03.2012 11:19, schrieb Gerd Hoffmann:
>> Hi,
>>
>>>> I do to, except when they fix broken behaviour. More seriously, do you
>>>> have other concerns with the mixemu code?
>>>
>>> Sure - it adds overhead.
>>
>> The point of this patchset is to kill the overhead if possible, i.e. try
>> to pass down the volume the guest asked for all the way down to the
>> hosts's audio hardware.
>>
>> This patchset starts with spice and pulse, adding support to other audio
>> backends (oss, alsa, windows, ...) shouldn't be hard, mixemu shouldn't
>> be used in the common case then.
>
> Then the default should probably to use it at least if the backend
> support passing it through? Leaving it broken by default wouldn't be
> very nice.
Agree. I would make the mixemu runtime switch have an effect only in
case the audio backend doesn't support volume, so the user can pick
between non-working volume control + zero overhead and working volume
control at the price of mixemu overhead then.
cheers,
Gerd
- Re: [Qemu-devel] [PATCH 08/11] Do not use pa_simple PulseAudio API, (continued)
- [Qemu-devel] [PATCH 09/11] configure: pa_simple is not needed anymore, Marc-André Lureau, 2012/03/12
- [Qemu-devel] [PATCH 10/11] Allow controlling volume with PulseAudio backend, Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Gerd Hoffmann, 2012/03/13
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Kevin Wolf, 2012/03/13
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3),
Gerd Hoffmann <=
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Marc-André Lureau, 2012/03/13
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), malc, 2012/03/13