From 0944c886e58f30db4272dc9c226621545ef48a69 Mon Sep 17 00:00:00 2001 From: Alex P Date: Tue, 16 Sep 2025 11:27:18 +0300 Subject: [PATCH] [WIP] Maintainability Improvement: Add debug logging throughout the audio system for easy debugging and troubleshooting --- internal/audio/c/audio.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/internal/audio/c/audio.c b/internal/audio/c/audio.c index c8298c2b..20b0461d 100644 --- a/internal/audio/c/audio.c +++ b/internal/audio/c/audio.c @@ -480,34 +480,26 @@ retry_write: // Write PCM to playback device with robust recovery int pcm_rc = snd_pcm_writei(pcm_playback_handle, pcm_buffer, pcm_frames); if (pcm_rc < 0) { - printf("[AUDIO] jetkvm_audio_decode_write: ALSA write failed with error %d (%s), attempt %d/%d\n", - pcm_rc, snd_strerror(pcm_rc), recovery_attempts + 1, max_recovery_attempts); - if (pcm_rc == -EPIPE) { // Buffer underrun - implement progressive recovery recovery_attempts++; if (recovery_attempts > max_recovery_attempts) { - printf("[AUDIO] jetkvm_audio_decode_write: Buffer underrun recovery failed after %d attempts\n", max_recovery_attempts); return -2; } - printf("[AUDIO] jetkvm_audio_decode_write: Buffer underrun detected, attempting recovery (attempt %d)\n", recovery_attempts); // Try to recover with prepare err = snd_pcm_prepare(pcm_playback_handle); if (err < 0) { - printf("[AUDIO] jetkvm_audio_decode_write: snd_pcm_prepare failed (%s), trying drop+prepare\n", snd_strerror(err)); // If prepare fails, try drop and prepare snd_pcm_drop(pcm_playback_handle); err = snd_pcm_prepare(pcm_playback_handle); if (err < 0) { - printf("[AUDIO] jetkvm_audio_decode_write: drop+prepare recovery failed (%s)\n", snd_strerror(err)); return -2; } } // Wait before retry to allow device to stabilize snd_pcm_wait(pcm_playback_handle, sleep_microseconds * recovery_attempts / 1000); - printf("[AUDIO] jetkvm_audio_decode_write: Buffer underrun recovery successful, retrying write\n"); goto retry_write; } else if (pcm_rc == -ESTRPIPE) { // Device suspended, implement robust resume logic