Browse Source

ALSA: pcm: detect xruns in no-period-wakeup mode

When period wakeups are disabled, successive calls to the pointer update
function do not have a maximum allowed distance, so xruns cannot be
detected with the pointer value only.

To detect xruns, compare the actually elapsed time with the time that
should have theoretically elapsed since the last update.  When the
hardware pointer has wrapped around due to an xrun, the actually elapsed
time will be too big by about hw_ptr_buffer_jiffies.

Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Clemens Ladisch 14 years ago
parent
commit
59ff878ffb
1 changed files with 16 additions and 2 deletions
  1. 16 2
      sound/core/pcm_lib.c

+ 16 - 2
sound/core/pcm_lib.c

@@ -374,9 +374,23 @@ static int snd_pcm_update_hw_ptr0(struct snd_pcm_substream *substream,
 			   (unsigned long)runtime->hw_ptr_base);
 			   (unsigned long)runtime->hw_ptr_base);
 	}
 	}
 
 
-	/* without period interrupts, there are no regular pointer updates */
-	if (runtime->no_period_wakeup)
+	if (runtime->no_period_wakeup) {
+		/*
+		 * Without regular period interrupts, we have to check
+		 * the elapsed time to detect xruns.
+		 */
+		jdelta = jiffies - runtime->hw_ptr_jiffies;
+		hdelta = jdelta - delta * HZ / runtime->rate;
+		while (hdelta > runtime->hw_ptr_buffer_jiffies / 2 + 1) {
+			delta += runtime->buffer_size;
+			hw_base += runtime->buffer_size;
+			if (hw_base >= runtime->boundary)
+				hw_base = 0;
+			new_hw_ptr = hw_base + pos;
+			hdelta -= runtime->hw_ptr_buffer_jiffies;
+		}
 		goto no_delta_check;
 		goto no_delta_check;
+	}
 
 
 	/* something must be really wrong */
 	/* something must be really wrong */
 	if (delta >= runtime->buffer_size + runtime->period_size) {
 	if (delta >= runtime->buffer_size + runtime->period_size) {