[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 43/51] audio: make mixeng optional
From: |
Kővágó, Zoltán |
Subject: |
[Qemu-devel] [PATCH 43/51] audio: make mixeng optional |
Date: |
Thu, 14 Jan 2016 14:45:56 +0100 |
Implementation of the previously added mixeng option.
Signed-off-by: Kővágó, Zoltán <address@hidden>
---
audio/audio.c | 70 +++++++++++++++++++++++++++++++++++++++++++++-----
audio/audio_template.h | 22 +++++++++++-----
2 files changed, 79 insertions(+), 13 deletions(-)
diff --git a/audio/audio.c b/audio/audio.c
index 7c59748..ea3219b 100644
--- a/audio/audio.c
+++ b/audio/audio.c
@@ -793,32 +793,46 @@ static void audio_timer (void *opaque)
*/
size_t AUD_write(SWVoiceOut *sw, void *buf, size_t size)
{
+ HWVoiceOut *hw;
+
if (!sw) {
/* XXX: Consider options */
return size;
}
+ hw = sw->hw;
- if (!sw->hw->enabled) {
+ if (!hw->enabled) {
dolog ("Writing to disabled voice %s\n", SW_NAME (sw));
return 0;
}
- return audio_pcm_sw_write(sw, buf, size);
+ if (hw->s->dev->out->mixeng) {
+ return audio_pcm_sw_write(sw, buf, size);
+ } else {
+ return hw->pcm_ops->write(hw, buf, size);
+ }
}
size_t AUD_read(SWVoiceIn *sw, void *buf, size_t size)
{
+ HWVoiceIn *hw;
+
if (!sw) {
/* XXX: Consider options */
return size;
}
+ hw = sw->hw;
- if (!sw->hw->enabled) {
+ if (!hw->enabled) {
dolog ("Reading from disabled voice %s\n", SW_NAME (sw));
return 0;
}
- return audio_pcm_sw_read(sw, buf, size);
+ if (hw->s->dev->in->mixeng) {
+ return audio_pcm_sw_read(sw, buf, size);
+ } else {
+ return hw->pcm_ops->read(hw, buf, size);
+ }
}
int AUD_get_buffer_size_out (SWVoiceOut *sw)
@@ -1038,6 +1052,26 @@ static void audio_run_out (AudioState *s)
HWVoiceOut *hw = NULL;
SWVoiceOut *sw;
+ if (!s->dev->out->mixeng) {
+ while ((hw = audio_pcm_hw_find_any_enabled_out(s, hw))) {
+ /* there is exactly 1 sw for each hw with no mixeng */
+ sw = hw->sw_head.lh_first;
+
+ if (hw->pending_disable) {
+ hw->enabled = 0;
+ hw->pending_disable = 0;
+ if (hw->pcm_ops->enable_out) {
+ hw->pcm_ops->enable_out(hw, false);
+ }
+ }
+
+ if (sw->active) {
+ sw->callback.fn(sw->callback.opaque, INT_MAX);
+ }
+ }
+ return;
+ }
+
while ((hw = audio_pcm_hw_find_any_enabled_out(s, hw))) {
size_t played, live, prev_rpos, free;
int nb_live, cleanup_required;
@@ -1174,6 +1208,17 @@ static void audio_run_in (AudioState *s)
{
HWVoiceIn *hw = NULL;
+ if (!s->dev->in->mixeng) {
+ while ((hw = audio_pcm_hw_find_any_enabled_in(s, hw))) {
+ /* there is exactly 1 sw for each hw with no mixeng */
+ SWVoiceIn *sw = hw->sw_head.lh_first;
+ if (sw->active) {
+ sw->callback.fn(sw->callback.opaque, INT_MAX);
+ }
+ }
+ return;
+ }
+
while ((hw = audio_pcm_hw_find_any_enabled_in(s, hw))) {
SWVoiceIn *sw;
size_t captured, min;
@@ -1654,6 +1699,11 @@ CaptureVoiceOut *AUD_add_capture(
s = QTAILQ_FIRST(&audio_states);
}
+ if (!s->dev->out->mixeng) {
+ dolog("Can't capture with mixeng disabled\n");
+ goto err0;
+ }
+
if (audio_validate_settings (as)) {
dolog ("Invalid settings were passed when trying to add capture\n");
audio_print_settings (as);
@@ -1811,9 +1861,13 @@ QemuOptsList qemu_audiodev_opts = {
static void validate_per_direction_opts(AudiodevPerDirectionOptions *pdo,
Error **errp)
{
+ if (!pdo->has_mixeng) {
+ pdo->has_mixeng = true;
+ pdo->mixeng = true;
+ }
if (!pdo->has_fixed_settings) {
pdo->has_fixed_settings = true;
- pdo->fixed_settings = true;
+ pdo->fixed_settings = pdo->mixeng;
}
if (!pdo->fixed_settings &&
(pdo->has_frequency || pdo->has_channels || pdo->has_format)) {
@@ -1821,6 +1875,10 @@ static void
validate_per_direction_opts(AudiodevPerDirectionOptions *pdo,
"You can't use frequency, channels or format with
fixed-settings=off");
return;
}
+ if (!pdo->mixeng && pdo->fixed_settings) {
+ error_setg(errp, "You can't use fixed-settings without mixeng");
+ return;
+ }
if (!pdo->has_frequency) {
pdo->has_frequency = true;
@@ -1832,7 +1890,7 @@ static void
validate_per_direction_opts(AudiodevPerDirectionOptions *pdo,
}
if (!pdo->has_voices) {
pdo->has_voices = true;
- pdo->voices = 1;
+ pdo->voices = pdo->mixeng ? 1 : INT_MAX;
}
if (!pdo->has_format) {
pdo->has_format = true;
diff --git a/audio/audio_template.h b/audio/audio_template.h
index 762efea..f3cbc66 100644
--- a/audio/audio_template.h
+++ b/audio/audio_template.h
@@ -78,15 +78,19 @@ static void glue (audio_pcm_hw_free_resources_, TYPE) (HW
*hw)
static void glue(audio_pcm_hw_alloc_resources_, TYPE)(HW *hw)
{
- size_t samples;
- if (hw->pcm_ops->glue(buffer_size_, TYPE)) {
- samples = hw->pcm_ops->glue(buffer_size_, TYPE)(hw);
+ if (hw->s->dev->TYPE->mixeng) {
+ size_t samples;
+ if (hw->pcm_ops->glue(buffer_size_, TYPE)) {
+ samples = hw->pcm_ops->glue(buffer_size_, TYPE)(hw);
+ } else {
+ samples = 1024; /* todo better default */
+ }
+
+ HWBUF = g_malloc0(sizeof(STSampleBuffer) + sizeof(st_sample)*samples);
+ HWBUF->size = samples;
} else {
- samples = 1024; /* todo better default */
+ HWBUF = NULL;
}
-
- HWBUF = g_malloc0(sizeof(STSampleBuffer) + sizeof(st_sample)*samples);
- HWBUF->size = samples;
}
static void glue (audio_pcm_sw_free_resources_, TYPE) (SW *sw)
@@ -105,6 +109,10 @@ static int glue (audio_pcm_sw_alloc_resources_, TYPE) (SW
*sw)
{
int samples;
+ if (!sw->s->dev->TYPE->mixeng) {
+ return 0;
+ }
+
samples = ((int64_t) sw->HWBUF->size << 32) / sw->ratio;
sw->buf = audio_calloc (AUDIO_FUNC, samples, sizeof (struct st_sample));
--
2.7.0
- [Qemu-devel] [PATCH 36/51] wavaudio: port to the new audio backend api, (continued)
- [Qemu-devel] [PATCH 36/51] wavaudio: port to the new audio backend api, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 33/51] paaudio: port to the new audio backend api, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 17/51] audio: reduce glob_audio_state usage, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 44/51] paaudio: get/put_buffer functions, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 45/51] audio: support more than two channels in volume setting, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 18/51] audio: basic support for multi backend audio, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 37/51] audio: remove remains of the old backend api, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 20/51] audio: audiodev= parameters no longer optional when -audiodev present, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 32/51] ossaudio: port to the new audio backend api, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 42/51] audio: add mixeng option (documentation), Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 43/51] audio: make mixeng optional,
Kővágó, Zoltán <=
- [Qemu-devel] [PATCH 47/51] audio: basic support for multichannel audio, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 39/51] audio: remove hw->samples, buffer_size_in/out pcm_ops, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 41/51] audio: split ctl_* functions into enable_* and volume_*, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 49/51] usb-audio: do not count on avail bytes actually available, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 40/51] audio: common rate control code for timer based outputs, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 51/51] usbaudio: change playback counters to 64 bit, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 24/51] audio: do not run each backend in audio_run, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 23/51] audio: remove audio_MIN, audio_MAX, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 48/51] paaudio: channel-map option, Kővágó, Zoltán, 2016/01/14
- [Qemu-devel] [PATCH 46/51] audio: replace shift in audio_pcm_info with bytes_per_frame, Kővágó, Zoltán, 2016/01/14