[pulseaudio-commits] r1168 - in /trunk/src/pulsecore: sink-input.c sink-input.h

svnmailer-noreply at 0pointer.de svnmailer-noreply at 0pointer.de
Sat Jul 29 08:06:50 PDT 2006


Author: lennart
Date: Sat Jul 29 17:06:49 2006
New Revision: 1168

URL: http://0pointer.de/cgi-bin/viewcvs.cgi?rev=1168&root=pulseaudio&view=rev
Log:
* implement "hot" moving of playback streams between sinks (pa_sink_input_move_to()). 
* optimize the adjusting of the volume in pa_sink_input_peek() a little

Modified:
    trunk/src/pulsecore/sink-input.c
    trunk/src/pulsecore/sink-input.h

Modified: trunk/src/pulsecore/sink-input.c
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/trunk/src/pulsecore/sink-input.c?rev=1168&root=pulseaudio&r1=1167&r2=1168&view=diff
==============================================================================
--- trunk/src/pulsecore/sink-input.c (original)
+++ trunk/src/pulsecore/sink-input.c Sat Jul 29 17:06:49 2006
@@ -34,10 +34,13 @@
 #include <pulsecore/sample-util.h>
 #include <pulsecore/core-subscribe.h>
 #include <pulsecore/log.h>
+#include <pulsecore/play-memblockq.h>
 
 #include "sink-input.h"
 
 #define CONVERT_BUFFER_LENGTH 4096
+#define MOVE_BUFFER_LENGTH (1024*1024)
+#define SILENCE_BUFFER_LENGTH (64*1024)
 
 #define CHECK_VALIDITY_RETURN_NULL(condition) \
 do {\
@@ -89,9 +92,11 @@
         resample_method = s->core->resample_method;
     
     if (variable_rate || !pa_sample_spec_equal(spec, &s->sample_spec) || !pa_channel_map_equal(map, &s->channel_map))
-        if (!(resampler = pa_resampler_new(spec, map, &s->sample_spec, &s->channel_map, s->core->memblock_stat, resample_method)))
+        if (!(resampler = pa_resampler_new(spec, map, &s->sample_spec, &s->channel_map, s->core->memblock_stat, resample_method))) {
+            pa_log_warn(__FILE__": Unsupported resampling operation.");
             return NULL;
-    
+        }
+
     i = pa_xnew(pa_sink_input, 1);
     i->ref = 1;
     i->state = PA_SINK_INPUT_DRAINED;
@@ -111,9 +116,14 @@
     i->get_latency = NULL;
     i->underrun = NULL;
     i->userdata = NULL;
+    i->move_silence = 0;
 
     pa_memchunk_reset(&i->resampled_chunk);
     i->resampler = resampler;
+    i->resample_method = resample_method;
+    i->variable_rate = variable_rate;
+
+    i->silence_memblock = NULL;
     
     assert(s->core);
     r = pa_idxset_put(s->core->sink_inputs, i, &i->index);
@@ -125,7 +135,8 @@
     pa_log_info(__FILE__": created %u \"%s\" on %u with sample spec \"%s\"", i->index, i->name, s->index, st);
 
     pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SINK_INPUT|PA_SUBSCRIPTION_EVENT_NEW, i->index);
-    
+    pa_sink_notify(i->sink);
+
     return i;    
 }
 
@@ -164,6 +175,9 @@
     if (i->resampler)
         pa_resampler_free(i->resampler);
 
+    if (i->silence_memblock)
+        pa_memblock_unref(i->silence_memblock);
+    
     pa_xfree(i->name);
     pa_xfree(i->driver);
     pa_xfree(i);
@@ -203,7 +217,10 @@
         r += i->get_latency(i);
 
     if (i->resampled_chunk.memblock)
-        r += pa_bytes_to_usec(i->resampled_chunk.length, &i->sample_spec);
+        r += pa_bytes_to_usec(i->resampled_chunk.length, &i->sink->sample_spec);
+
+    if (i->move_silence)
+        r += pa_bytes_to_usec(i->move_silence, &i->sink->sample_spec);
 
     return r;
 }
@@ -211,6 +228,7 @@
 int pa_sink_input_peek(pa_sink_input *i, pa_memchunk *chunk, pa_cvolume *volume) {
     int ret = -1;
     int do_volume_adj_here;
+    int volume_is_norm;
     
     assert(i);
     assert(i->ref >= 1);
@@ -223,6 +241,23 @@
         goto finish;
 
     assert(i->state == PA_SINK_INPUT_RUNNING || i->state == PA_SINK_INPUT_DRAINED);
+
+    if (i->move_silence > 0) {
+
+        /* We have just been moved and shall play some silence for a
+         * while until the old sink has drained its playback buffer */
+        
+        if (!i->silence_memblock)
+            i->silence_memblock = pa_silence_memblock_new(&i->sink->sample_spec, SILENCE_BUFFER_LENGTH, i->sink->core->memblock_stat);
+
+        chunk->memblock = pa_memblock_ref(i->silence_memblock);
+        chunk->index = 0;
+        chunk->length = i->move_silence < chunk->memblock->length ? i->move_silence : chunk->memblock->length;
+
+        ret = 0;
+        do_volume_adj_here = 1;
+        goto finish;
+    }
     
     if (!i->resampler) {
         do_volume_adj_here = 0;
@@ -231,6 +266,7 @@
     }
 
     do_volume_adj_here = !pa_channel_map_equal(&i->channel_map, &i->sink->channel_map);
+    volume_is_norm = pa_cvolume_is_norm(&i->volume);
     
     while (!i->resampled_chunk.memblock) {
         pa_memchunk tchunk;
@@ -250,7 +286,7 @@
         tchunk.length = l;
 
         /* It might be necessary to adjust the volume here */
-        if (do_volume_adj_here) {
+        if (do_volume_adj_here && !volume_is_norm) {
             pa_memchunk_make_writable(&tchunk, i->sink->core->memblock_stat, 0);
             pa_volume_memchunk(&tchunk, &i->sample_spec, &i->volume);
         }
@@ -299,6 +335,30 @@
     assert(i->ref >= 1);
     assert(length > 0);
 
+    if (i->move_silence > 0) {
+
+        if (chunk) {
+
+            if (chunk->memblock != i->silence_memblock ||
+                chunk->index != 0 ||
+                (chunk->memblock && (chunk->length != (i->silence_memblock->length < i->move_silence ? i->silence_memblock->length : i->move_silence)))) 
+                return;
+            
+        }
+
+        assert(i->move_silence >= length);
+        
+        i->move_silence -= length;
+
+        if (i->move_silence <= 0) {
+            assert(i->silence_memblock);
+            pa_memblock_unref(i->silence_memblock);
+            i->silence_memblock = NULL;
+        }
+
+        return;
+    }
+
     if (!i->resampler) {
         if (i->drop)
             i->drop(i, chunk, length);
@@ -367,6 +427,8 @@
 
     i->sample_spec.rate = rate;
     pa_resampler_set_input_rate(i->resampler, rate);
+
+    pa_subscription_post(i->sink->core, PA_SUBSCRIPTION_EVENT_SINK_INPUT|PA_SUBSCRIPTION_EVENT_CHANGE, i->index);
 }
 
 void pa_sink_input_set_name(pa_sink_input *i, const char *name) {
@@ -388,3 +450,162 @@
 
     return pa_resampler_get_method(i->resampler);
 }
+
+int pa_sink_input_move_to(pa_sink_input *i, pa_sink *dest, int immediately) {
+    pa_resampler *new_resampler = NULL;
+    pa_memblockq *buffer = NULL;
+    pa_sink *origin;
+    
+    assert(i);
+    assert(dest);
+
+    origin = i->sink;
+
+    if (dest == origin)
+        return 0;
+
+    if (pa_idxset_size(dest->inputs) >= PA_MAX_INPUTS_PER_SINK) {
+        pa_log_warn(__FILE__": Failed to move sink input: too many inputs per sink.");
+        return -1;
+    }
+
+    if (i->resampler &&
+        pa_sample_spec_equal(&origin->sample_spec, &dest->sample_spec) &&
+        pa_channel_map_equal(&origin->channel_map, &dest->channel_map))
+
+        /* Try to reuse the old resampler if possible */
+        new_resampler = i->resampler;
+    
+    else if (i->variable_rate ||
+        !pa_sample_spec_equal(&i->sample_spec, &dest->sample_spec) ||
+        !pa_channel_map_equal(&i->channel_map, &dest->channel_map)) {
+
+        /* Okey, we need a new resampler for the new sink */
+        
+        if (!(new_resampler = pa_resampler_new(
+                      &i->sample_spec, &i->channel_map,
+                      &dest->sample_spec, &dest->channel_map,
+                      dest->core->memblock_stat,
+                      i->resample_method))) {
+            pa_log_warn(__FILE__": Unsupported resampling operation.");
+            return -1;
+        }
+    }
+
+    if (!immediately) {
+        pa_usec_t old_latency, new_latency;
+        pa_usec_t silence_usec = 0;
+
+        buffer = pa_memblockq_new(0, MOVE_BUFFER_LENGTH, 0, pa_frame_size(&origin->sample_spec), 0, 0, NULL, NULL);
+        
+        /* Let's do a little bit of Voodoo for compensating latency
+         * differences */
+
+        old_latency = pa_sink_get_latency(origin);
+        new_latency = pa_sink_get_latency(dest);
+        
+        /* The already resampled data should go to the old sink */
+
+        if (old_latency >= new_latency) {
+
+            /* The latency of the old sink is larger than the latency
+             * of the new sink. Therefore to compensate for the
+             * difference we to play silence on the new one for a
+             * while */
+
+            silence_usec = old_latency - new_latency;
+            
+        } else {
+            size_t l;
+            int volume_is_norm;
+            
+            /* The latency of new sink is larger than the latency of
+             * the old sink. Therefore we have to precompute a little
+             * and make sure that this is still played on the old
+             * sink, until we can play the first sample on the new
+             * sink.*/
+            
+            l = pa_usec_to_bytes(new_latency - old_latency, &origin->sample_spec);
+
+            volume_is_norm = pa_cvolume_is_norm(&i->volume);
+
+            while (l > 0) {
+                pa_memchunk chunk;
+                pa_cvolume volume;
+                size_t n;
+
+                if (pa_sink_input_peek(i, &chunk, &volume) < 0)
+                    break;
+
+                n = chunk.length > l ? l : chunk.length;
+                pa_sink_input_drop(i, &chunk, n);
+                chunk.length = n;
+
+                if (!volume_is_norm) {
+                    pa_memchunk_make_writable(&chunk, origin->core->memblock_stat, 0);
+                    pa_volume_memchunk(&chunk, &origin->sample_spec, &volume);
+                }
+
+                if (pa_memblockq_push(buffer, &chunk) < 0) {
+                    pa_memblock_unref(chunk.memblock);
+                    break;
+                }
+
+                pa_memblock_unref(chunk.memblock);
+                l -= n;
+            }
+        }
+
+        if (i->resampled_chunk.memblock) {
+
+            /* There is still some data left in the already resampled
+             * memory block. Hence, let's output it on the old sink
+             * and sleep so long on the new sink */
+
+            pa_memblockq_push(buffer, &i->resampled_chunk);
+            silence_usec += pa_bytes_to_usec(i->resampled_chunk.length, &origin->sample_spec);
+        }
+
+        /* Calculate the new sleeping time */
+        i->move_silence = pa_usec_to_bytes(
+                pa_bytes_to_usec(i->move_silence, &i->sample_spec) +
+                silence_usec,
+                &i->sample_spec);
+    }
+
+    /* Okey, let's move it */
+    pa_idxset_remove_by_data(i->sink->inputs, i, NULL);
+    i->sink = dest;
+    pa_idxset_put(i->sink->inputs, i, NULL);
+
+    /* Replace resampler */
+    if (new_resampler != i->resampler) {
+        if (i->resampler)
+            pa_resampler_free(i->resampler);
+        i->resampler = new_resampler;
+
+        /* if the resampler changed, the silence memblock is
+         * probably invalid now, too */
+        if (i->silence_memblock) {
+            pa_memblock_unref(i->silence_memblock);
+            i->silence_memblock = NULL;
+        }
+    }
+
+    /* Dump already resampled data */
+    if (i->resampled_chunk.memblock) {
+        pa_memblock_unref(i->resampled_chunk.memblock);
+        i->resampled_chunk.memblock = NULL;
+        i->resampled_chunk.index = i->resampled_chunk.length = 0;
+    }
+    
+    /* Notify everyone */
+    pa_subscription_post(i->sink->core, PA_SUBSCRIPTION_EVENT_SINK_INPUT|PA_SUBSCRIPTION_EVENT_CHANGE, i->index);
+    pa_sink_notify(i->sink);
+
+    /* Ok, no let's feed the precomputed buffer to the old sink */
+    if (buffer)
+        pa_play_memblockq(origin, "Ghost Stream", &origin->sample_spec, &origin->channel_map, buffer, NULL);
+
+    return 0;
+}

Modified: trunk/src/pulsecore/sink-input.h
URL: http://0pointer.de/cgi-bin/viewcvs.cgi/trunk/src/pulsecore/sink-input.h?rev=1168&root=pulseaudio&r1=1167&r2=1168&view=diff
==============================================================================
--- trunk/src/pulsecore/sink-input.h (original)
+++ trunk/src/pulsecore/sink-input.h Sat Jul 29 17:06:49 2006
@@ -55,6 +55,11 @@
     pa_channel_map channel_map;
 
     pa_cvolume volume;
+
+    /* Some silence to play before the actual data. This is used to
+     * compensate for latency differences when moving a sink input
+     * "hot" between sinks. */
+    size_t move_silence;
     
     int (*peek) (pa_sink_input *i, pa_memchunk *chunk);
     void (*drop) (pa_sink_input *i, const pa_memchunk *chunk, size_t length);
@@ -66,6 +71,11 @@
 
     pa_memchunk resampled_chunk;
     pa_resampler *resampler;
+
+    int variable_rate;
+    pa_resample_method_t resample_method;
+
+    pa_memblock *silence_memblock;
 };
 
 pa_sink_input* pa_sink_input_new(
@@ -76,7 +86,7 @@
     const pa_channel_map *map,
     const pa_cvolume *volume,
     int variable_rate,
-    int resample_method);
+    pa_resample_method_t resample_method);
 
 void pa_sink_input_unref(pa_sink_input* i);
 pa_sink_input* pa_sink_input_ref(pa_sink_input* i);
@@ -103,4 +113,6 @@
 
 pa_resample_method_t pa_sink_input_get_resample_method(pa_sink_input *i);
 
+int pa_sink_input_move_to(pa_sink_input *i, pa_sink *dest, int immediately);
+
 #endif




More information about the pulseaudio-commits mailing list