[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: |
malc |
Subject: |
Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3) |
Date: |
Tue, 13 Mar 2012 00:51:16 +0400 (MSK) |
User-agent: |
Alpine 2.00 (LNX 1167 2008-08-23) |
On Mon, 12 Mar 2012, Marc-Andr? Lureau wrote:
> Hi Vassili
>
> Thanks for your review!
>
> On Mon, Mar 12, 2012 at 8:44 PM, malc <address@hidden> wrote:
> > a. Pulse/Spice have per connection volume
>
> Each playback/recorde stream has it's own volume, and that's what we
> control. We get the client full range thanks to flat-volume logic
> (implemented by PulseAudio and Windows Vista+)
Okay.
>
> > b. Other drivers are not affected
>
> I don't see yet how, but I will review other drivers ctl_{in,out}
> implementations
You don't see how they can be affected or you don't see how you can
figure that out?
>
> > c. mixeng is not the default
>
> So you mean the last patch shouldn't be applied? Can you explain the
> rationale? Why would we want broken behaviour by default?
>
Because i hate change.
--
mailto:address@hidden
- [Qemu-devel] [PATCH 06/11] hw/ac97: add support for volume control, (continued)
- [Qemu-devel] [PATCH 06/11] hw/ac97: add support for volume control, Marc-André Lureau, 2012/03/12
- [Qemu-devel] [PATCH 07/11] audio/spice: add support for volume control, Marc-André Lureau, 2012/03/12
- [Qemu-devel] [PATCH 11/11] Make mixemu mandatory, Marc-André Lureau, 2012/03/12
- [Qemu-devel] [PATCH 08/11] Do not use pa_simple PulseAudio API, Marc-André Lureau, 2012/03/12
- [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 <=
- 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, 2012/03/13
- 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