qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

             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).