From: Marc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
---
audio/audio-mixeng-be.c | 32 ++++++++------------------------
audio/trace-events | 5 +++++
2 files changed, 13 insertions(+), 24 deletions(-)
diff --git a/audio/audio-mixeng-be.c b/audio/audio-mixeng-be.c
index c3234aafb1b..5303c08a285 100644
--- a/audio/audio-mixeng-be.c
+++ b/audio/audio-mixeng-be.c
@@ -29,8 +29,6 @@
#define AUDIO_CAP "audio"
#include "audio_int.h"
-/* #define DEBUG_OUT */
-
#define SW_NAME(sw) (sw)->name ? (sw)->name : "unknown"
const struct mixeng_volume nominal_volume = {
@@ -568,9 +566,7 @@ static size_t audio_pcm_sw_write(SWVoiceOut *sw, void *buf, size_t buf_len)
}
if (live == hw->mix_buf.size) {
-#ifdef DEBUG_OUT
- dolog ("%s is full %zu\n", sw->name, live);
-#endif
+ trace_audio_out_full(sw->name, live);
return 0;
}
@@ -618,15 +614,8 @@ static size_t audio_pcm_sw_write(SWVoiceOut *sw, void *buf, size_t buf_len)
sw->resample_buf.pos = 0;
}
-#ifdef DEBUG_OUT
- dolog (
- "%s: write size %zu written %zu total mixed %zu\n",
- SW_NAME(sw),
- buf_len / sw->info.bytes_per_frame,
- total_in,
- sw->total_hw_samples_mixed
- );
-#endif
+ trace_audio_sw_write(SW_NAME(sw), buf_len / sw->info.bytes_per_frame,
+ total_in, sw->total_hw_samples_mixed);
return total_in * sw->info.bytes_per_frame;
}
@@ -900,10 +889,8 @@ static size_t audio_get_free(SWVoiceOut *sw)
dead = sw->hw->mix_buf.size - live;
-#ifdef DEBUG_OUT
- dolog("%s: get_free live %zu dead %zu frontend frames %u\n",
- SW_NAME(sw), live, dead, st_rate_frames_in(sw->rate, dead));
-#endif
+ trace_audio_get_free(SW_NAME(sw), live, dead,
+ st_rate_frames_in(sw->rate, dead));
return dead;
}
@@ -1058,9 +1045,8 @@ static void audio_run_out(AudioMixengBackend *s)
if (hw->pending_disable && !nb_live) {
SWVoiceCap *sc;
-#ifdef DEBUG_OUT
- dolog ("Disabling voice\n");
-#endif
+
+ trace_audio_out_disable();
hw->enabled = false;
hw->pending_disable = false;
if (k->enable_out) {
@@ -1089,9 +1075,7 @@ static void audio_run_out(AudioMixengBackend *s)
hw->mix_buf.pos = 0;
}
-#ifdef DEBUG_OUT
- dolog("played=%zu\n", played);
-#endif
+ trace_audio_out_played(played);
if (played) {
audio_capture_mix_and_clear (hw, prev_rpos, played);
diff --git a/audio/trace-events b/audio/trace-events
index 3f8ebf9c6cb..7a73d6eb84f 100644
--- a/audio/trace-events
+++ b/audio/trace-events
@@ -65,6 +65,11 @@ audio_capture_attach(const char *name, bool active) "capture %s active=%d"
audio_get_avail(const char *name, size_t live, uint32_t frontend_frames) "%s: get_avail live %zu frontend frames %u"
audio_run_poll(const char *msg, int64_t elapsed_us) "Elapsed since last %s: %" PRId64 " us"
audio_capture_free_sw(const char *name) "freeing %s"
+audio_out_full(const char *name, size_t live) "%s is full %zu"
+audio_sw_write(const char *name, size_t size, size_t written, size_t total_mixed) "%s: write size %zu written %zu total mixed %zu"
+audio_get_free(const char *name, size_t live, size_t dead, uint32_t frontend_frames) "%s: get_free live %zu dead %zu frontend frames %u"
+audio_out_disable(void) "Disabling voice"
+audio_out_played(size_t played) "played=%zu"
# audio_template.h
audio_open_out(const char *name, int freq, int nchannels, int fmt) "open %s, freq %d, nchannels %d, fmt %d"
--
2.52.0