[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 10/11] alsaaudio: change default playback settings
From: |
Volker Rümelin |
Subject: |
[PATCH 10/11] alsaaudio: change default playback settings |
Date: |
Sun, 18 Dec 2022 18:15:38 +0100 |
The currently used default playback settings in the ALSA audio
backend are a bit unfortunate. With a few emulated audio devices,
audio playback does not work properly. Here is a short part of
the debug log while audio is playing (elapsed time in seconds).
audio: Elapsed since last alsa run (running): 0.046244
audio: Elapsed since last alsa run (running): 0.023137
audio: Elapsed since last alsa run (running): 0.023170
audio: Elapsed since last alsa run (running): 0.023650
audio: Elapsed since last alsa run (running): 0.060802
audio: Elapsed since last alsa run (running): 0.031931
For some audio devices the time of more than 23ms between updates
is too long.
Set the period time to 5.8ms so that the maximum time between
two updates typically does not exceed 11ms. This roughly matches
the 10ms period time when doing playback with the audio timer.
After this patch the debug log looks like this.
audio: Elapsed since last alsa run (running): 0.011919
audio: Elapsed since last alsa run (running): 0.005788
audio: Elapsed since last alsa run (running): 0.005995
audio: Elapsed since last alsa run (running): 0.011069
audio: Elapsed since last alsa run (running): 0.005901
audio: Elapsed since last alsa run (running): 0.006084
Signed-off-by: Volker Rümelin <vr_qemu@t-online.de>
---
audio/alsaaudio.c | 11 ++++-------
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/audio/alsaaudio.c b/audio/alsaaudio.c
index 5f50dfa0bf..0cc982e61f 100644
--- a/audio/alsaaudio.c
+++ b/audio/alsaaudio.c
@@ -913,17 +913,14 @@ static void *alsa_audio_init(Audiodev *dev)
alsa_init_per_direction(aopts->in);
alsa_init_per_direction(aopts->out);
- /*
- * need to define them, as otherwise alsa produces no sound
- * doesn't set has_* so alsa_open can identify it wasn't set by the user
- */
+ /* don't set has_* so alsa_open can identify it wasn't set by the user */
if (!dev->u.alsa.out->has_period_length) {
- /* 1024 frames assuming 44100Hz */
- dev->u.alsa.out->period_length = 1024 * 1000000 / 44100;
+ /* 256 frames assuming 44100Hz */
+ dev->u.alsa.out->period_length = 5805;
}
if (!dev->u.alsa.out->has_buffer_length) {
/* 4096 frames assuming 44100Hz */
- dev->u.alsa.out->buffer_length = 4096ll * 1000000 / 44100;
+ dev->u.alsa.out->buffer_length = 92880;
}
/*
--
2.35.3
- Re: [PATCH 03/11] audio: rename hardware store to backend, (continued)
- [PATCH 06/11] audio/alsaaudio: use g_new0() instead of audio_calloc(), Volker Rümelin, 2022/12/18
- [PATCH 11/11] alsaaudio: reintroduce default recording settings, Volker Rümelin, 2022/12/18
- [PATCH 08/11] audio/audio_template: use g_new0() to replace audio_calloc(), Volker Rümelin, 2022/12/18
- [PATCH 04/11] audio: remove unused #define AUDIO_STRINGIFY, Volker Rümelin, 2022/12/18
- [PATCH 10/11] alsaaudio: change default playback settings,
Volker Rümelin <=
- Re: [PATCH 10/11] alsaaudio: change default playback settings, Christian Schoenebeck, 2022/12/21
- Re: [PATCH 10/11] alsaaudio: change default playback settings, Volker Rümelin, 2022/12/26
- Re: [PATCH 10/11] alsaaudio: change default playback settings, Volker Rümelin, 2022/12/26
- Re: [PATCH 10/11] alsaaudio: change default playback settings, Christian Schoenebeck, 2022/12/28
- Re: [PATCH 10/11] alsaaudio: change default playback settings, Volker Rümelin, 2022/12/29
- Re: [PATCH 10/11] alsaaudio: change default playback settings, Volker Rümelin, 2022/12/30
- Re: [PATCH 10/11] alsaaudio: change default playback settings, Christian Schoenebeck, 2022/12/30
[PATCH 09/11] audio: remove audio_calloc() function, Volker Rümelin, 2022/12/18