From: "Marc-André Lureau" <marcandre.lureau@gmail.com>
To: qemu-devel@nongnu.org
Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>,
dnb@redhat.com, dlaor@redhat.com, kraxel@redhat.com
Subject: [Qemu-devel] [PATCH v4 00/11] apply volume on client side
Date: Tue, 13 Mar 2012 16:20:48 +0100 [thread overview]
Message-ID: <1331652059-10090-1-git-send-email-marcandre.lureau@redhat.com> (raw)
Hi,
This patch series implements client-side audio volume support. This reduces confusion of guest users when volume control is not effective (because mixemu is disabled or because client-side is muted and can't be unmuted by the guest..)
Instead, the backend is responsible for applying volume giving the guest control over the full range of the client, and avoiding multiple level of volume/mute effects.
Although I was mainly interested in having the Spice audio backend support, I also added PulseAudio backend support (which unfortunately requires full-blown API, even after quick discussion with upstream).
v4:
- code style fixes (only warnings left - mostly spaces)
- dropped the 1 << 32 -1 change
- added QEMU_MIXEMU runtime option, enabled by default
- rfc: removed some deprecated audio options
Marc-André Lureau (11):
audio: add VOICE_VOLUME ctl
audio: don't apply volume effect if backend has VOICE_VOLUME_CAP
hw/ac97: remove USE_MIXER code
hw/ac97: the volume mask is not only 0x1f
hw/ac97: add support for volume control
audio/spice: add support for volume control
Do not use pa_simple PulseAudio API
configure: pa_simple is not needed anymore
Allow controlling volume with PulseAudio backend
Enable mixemu by default, add runtime option
audio/rfc: remove PLIVE and PERIOD options
audio/audio.c | 64 +++----
audio/audio_int.h | 6 +
audio/audio_template.h | 28 +---
audio/mixeng.c | 6 -
audio/paaudio.c | 476 +++++++++++++++++++++++++++++++++++++++++++-----
audio/spiceaudio.c | 41 ++++
configure | 14 +-
hw/ac97.c | 140 +++++---------
hw/hda-audio.c | 4 -
9 files changed, 561 insertions(+), 218 deletions(-)
--
1.7.7.6
next reply other threads:[~2012-03-13 15:21 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-13 15:20 Marc-André Lureau [this message]
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 01/11] audio: add VOICE_VOLUME ctl Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 02/11] audio: don't apply volume effect if backend has VOICE_VOLUME_CAP Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 03/11] hw/ac97: remove USE_MIXER code Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 04/11] hw/ac97: the volume mask is not only 0x1f Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 05/11] hw/ac97: add support for volume control Marc-André Lureau
2012-03-13 15:37 ` malc
2012-03-13 16:04 ` Marc-André Lureau
2012-03-13 16:07 ` malc
2012-03-13 16:23 ` Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 06/11] audio/spice: " Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 07/11] Do not use pa_simple PulseAudio API Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 08/11] configure: pa_simple is not needed anymore Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 09/11] Allow controlling volume with PulseAudio backend Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 10/11] Enable mixemu by default, add runtime option Marc-André Lureau
2012-03-13 15:20 ` [Qemu-devel] [PATCH v4 11/11] audio/rfc: remove PLIVE and PERIOD options Marc-André Lureau
2012-03-13 15:47 ` malc
2012-03-14 9:22 ` Gerd Hoffmann
2012-03-14 11:20 ` Marc-André Lureau
2012-03-14 11:49 ` Gerd Hoffmann
2012-03-14 12:13 ` Marc-André Lureau
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=1331652059-10090-1-git-send-email-marcandre.lureau@redhat.com \
--to=marcandre.lureau@gmail.com \
--cc=dlaor@redhat.com \
--cc=dnb@redhat.com \
--cc=kraxel@redhat.com \
--cc=marcandre.lureau@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).