X-Git-Url: https://code.delx.au/pulseaudio/blobdiff_plain/ded07a5898a0484783c29912d259ebda7392f5f4..70441d40fb07b13cca85d08fe09c9ddb81eadaf0:/src/pulsecore/source.c diff --git a/src/pulsecore/source.c b/src/pulsecore/source.c index 1ec80271..da0dc685 100644 --- a/src/pulsecore/source.c +++ b/src/pulsecore/source.c @@ -27,6 +27,7 @@ #include #include +#include #include #include #include @@ -39,7 +40,7 @@ #include #include #include -#include +#include #include #include "source.h" @@ -72,6 +73,7 @@ pa_source_new_data* pa_source_new_data_init(pa_source_new_data *data) { pa_zero(*data); data->proplist = pa_proplist_new(); + data->ports = pa_hashmap_new_full(pa_idxset_string_hash_func, pa_idxset_string_compare_func, NULL, (pa_free_cb_t) pa_device_port_unref); return data; } @@ -97,6 +99,13 @@ void pa_source_new_data_set_channel_map(pa_source_new_data *data, const pa_chann data->channel_map = *map; } +void pa_source_new_data_set_alternate_sample_rate(pa_source_new_data *data, const uint32_t alternate_sample_rate) { + pa_assert(data); + + data->alternate_sample_rate_is_set = true; + data->alternate_sample_rate = alternate_sample_rate; +} + void pa_source_new_data_set_volume(pa_source_new_data *data, const pa_cvolume *volume) { pa_assert(data); @@ -104,10 +113,10 @@ void pa_source_new_data_set_volume(pa_source_new_data *data, const pa_cvolume *v data->volume = *volume; } -void pa_source_new_data_set_muted(pa_source_new_data *data, pa_bool_t mute) { +void pa_source_new_data_set_muted(pa_source_new_data *data, bool mute) { pa_assert(data); - data->muted_is_set = TRUE; + data->muted_is_set = true; data->muted = !!mute; } @@ -123,14 +132,8 @@ void pa_source_new_data_done(pa_source_new_data *data) { pa_proplist_free(data->proplist); - if (data->ports) { - pa_device_port *p; - - while ((p = pa_hashmap_steal_first(data->ports))) - pa_device_port_free(p); - - pa_hashmap_free(data->ports, NULL, NULL); - } + if (data->ports) + pa_hashmap_free(data->ports); pa_xfree(data->name); pa_xfree(data->active_port); @@ -143,11 +146,13 @@ static void reset_callbacks(pa_source *s) { s->set_state = NULL; s->get_volume = NULL; s->set_volume = NULL; + s->write_volume = NULL; s->get_mute = NULL; s->set_mute = NULL; s->update_requested_latency = NULL; s->set_port = NULL; s->get_formats = NULL; + s->update_rate = NULL; } /* Called from main context */ @@ -201,22 +206,28 @@ pa_source* pa_source_new( if (!data->volume_is_set) { pa_cvolume_reset(&data->volume, data->sample_spec.channels); - data->save_volume = FALSE; + data->save_volume = false; } pa_return_null_if_fail(pa_cvolume_valid(&data->volume)); pa_return_null_if_fail(pa_cvolume_compatible(&data->volume, &data->sample_spec)); if (!data->muted_is_set) - data->muted = FALSE; + data->muted = false; if (data->card) pa_proplist_update(data->proplist, PA_UPDATE_MERGE, data->card->proplist); - pa_device_init_description(data->proplist); - pa_device_init_icon(data->proplist, FALSE); + pa_device_init_description(data->proplist, data->card); + pa_device_init_icon(data->proplist, false); pa_device_init_intended_roles(data->proplist); + if (!data->active_port) { + pa_device_port *p = pa_device_port_find_best(data->ports); + if (p) + pa_source_new_data_set_port(data, p->name); + } + if (pa_hook_fire(&core->hooks[PA_CORE_HOOK_SOURCE_FIXATE], data) < 0) { pa_xfree(s); pa_namereg_unregister(core, name); @@ -230,7 +241,8 @@ pa_source* pa_source_new( s->state = PA_SOURCE_INIT; s->flags = flags; s->priority = 0; - s->suspend_cause = 0; + s->suspend_cause = data->suspend_cause; + pa_source_set_mixer_dirty(s, false); s->name = pa_xstrdup(name); s->proplist = pa_proplist_copy(data->proplist); s->driver = pa_xstrdup(pa_path_get_filename(data->driver)); @@ -241,6 +253,17 @@ pa_source* pa_source_new( s->sample_spec = data->sample_spec; s->channel_map = data->channel_map; + s->default_sample_rate = s->sample_spec.rate; + + if (data->alternate_sample_rate_is_set) + s->alternate_sample_rate = data->alternate_sample_rate; + else + s->alternate_sample_rate = s->core->alternate_sample_rate; + + if (s->sample_spec.rate == s->alternate_sample_rate) { + pa_log_warn("Default and alternate sample rates are the same."); + s->alternate_sample_rate = 0; + } s->outputs = pa_idxset_new(NULL, NULL); s->n_corked = 0; @@ -252,7 +275,7 @@ pa_source* pa_source_new( s->base_volume = PA_VOLUME_NORM; s->n_volume_steps = PA_VOLUME_NORM+1; s->muted = data->muted; - s->refresh_volume = s->refresh_muted = FALSE; + s->refresh_volume = s->refresh_muted = false; reset_callbacks(s); s->userdata = NULL; @@ -265,20 +288,21 @@ pa_source* pa_source_new( data->ports = NULL; s->active_port = NULL; - s->save_port = FALSE; + s->save_port = false; - if (data->active_port && s->ports) + if (data->active_port) if ((s->active_port = pa_hashmap_get(s->ports, data->active_port))) s->save_port = data->save_port; - if (!s->active_port && s->ports) { - void *state; - pa_device_port *p; + /* Hopefully the active port has already been assigned in the previous call + to pa_device_port_find_best, but better safe than sorry */ + if (!s->active_port) + s->active_port = pa_device_port_find_best(s->ports); - PA_HASHMAP_FOREACH(p, s->ports, state) - if (!s->active_port || p->priority > s->active_port->priority) - s->active_port = p; - } + if (s->active_port) + s->latency_offset = s->active_port->latency_offset; + else + s->latency_offset = 0; s->save_volume = data->save_volume; s->save_muted = data->save_muted; @@ -291,12 +315,13 @@ pa_source* pa_source_new( 0); s->thread_info.rtpoll = NULL; - s->thread_info.outputs = pa_hashmap_new(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func); + s->thread_info.outputs = pa_hashmap_new_full(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func, NULL, + (pa_free_cb_t) pa_source_output_unref); s->thread_info.soft_volume = s->soft_volume; s->thread_info.soft_muted = s->muted; s->thread_info.state = s->state; s->thread_info.max_rewind = 0; - s->thread_info.requested_latency_valid = FALSE; + s->thread_info.requested_latency_valid = false; s->thread_info.requested_latency = 0; s->thread_info.min_latency = ABSOLUTE_MIN_LATENCY; s->thread_info.max_latency = ABSOLUTE_MAX_LATENCY; @@ -305,8 +330,9 @@ pa_source* pa_source_new( PA_LLIST_HEAD_INIT(pa_source_volume_change, s->thread_info.volume_changes); s->thread_info.volume_changes_tail = NULL; pa_sw_cvolume_multiply(&s->thread_info.current_hw_volume, &s->soft_volume, &s->real_volume); - s->thread_info.volume_change_safety_margin = core->sync_volume_safety_margin_usec; - s->thread_info.volume_change_extra_delay = core->sync_volume_extra_delay_usec; + s->thread_info.volume_change_safety_margin = core->deferred_volume_safety_margin_usec; + s->thread_info.volume_change_extra_delay = core->deferred_volume_extra_delay_usec; + s->thread_info.latency_offset = s->latency_offset; /* FIXME: This should probably be moved to pa_source_put() */ pa_assert_se(pa_idxset_put(core->sources, s, &s->index) >= 0); @@ -329,7 +355,7 @@ pa_source* pa_source_new( /* Called from main context */ static int source_set_state(pa_source *s, pa_source_state_t state) { int ret; - pa_bool_t suspend_change; + bool suspend_change; pa_source_state_t original_state; pa_assert(s); @@ -359,7 +385,7 @@ static int source_set_state(pa_source *s, pa_source_state_t state) { s->state = state; - if (state != PA_SOURCE_UNLINKED) { /* if we enter UNLINKED state pa_source_unlink() will fire the apropriate events */ + if (state != PA_SOURCE_UNLINKED) { /* if we enter UNLINKED state pa_source_unlink() will fire the appropriate events */ pa_hook_fire(&s->core->hooks[PA_CORE_HOOK_SOURCE_STATE_CHANGED], s); pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE | PA_SUBSCRIPTION_EVENT_CHANGE, s->index); } @@ -388,15 +414,49 @@ void pa_source_set_get_volume_callback(pa_source *s, pa_source_cb_t cb) { } void pa_source_set_set_volume_callback(pa_source *s, pa_source_cb_t cb) { + pa_source_flags_t flags; + pa_assert(s); + pa_assert(!s->write_volume || cb); s->set_volume = cb; + + /* Save the current flags so we can tell if they've changed */ + flags = s->flags; + + if (cb) { + /* The source implementor is responsible for setting decibel volume support */ + s->flags |= PA_SOURCE_HW_VOLUME_CTRL; + } else { + s->flags &= ~PA_SOURCE_HW_VOLUME_CTRL; + /* See note below in pa_source_put() about volume sharing and decibel volumes */ + pa_source_enable_decibel_volume(s, !(s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)); + } + + /* If the flags have changed after init, let any clients know via a change event */ + if (s->state != PA_SOURCE_INIT && flags != s->flags) + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); } void pa_source_set_write_volume_callback(pa_source *s, pa_source_cb_t cb) { + pa_source_flags_t flags; + pa_assert(s); + pa_assert(!cb || s->set_volume); s->write_volume = cb; + + /* Save the current flags so we can tell if they've changed */ + flags = s->flags; + + if (cb) + s->flags |= PA_SOURCE_DEFERRED_VOLUME; + else + s->flags &= ~PA_SOURCE_DEFERRED_VOLUME; + + /* If the flags have changed after init, let any clients know via a change event */ + if (s->state != PA_SOURCE_INIT && flags != s->flags) + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); } void pa_source_set_get_mute_callback(pa_source *s, pa_source_cb_t cb) { @@ -406,9 +466,65 @@ void pa_source_set_get_mute_callback(pa_source *s, pa_source_cb_t cb) { } void pa_source_set_set_mute_callback(pa_source *s, pa_source_cb_t cb) { + pa_source_flags_t flags; + pa_assert(s); s->set_mute = cb; + + /* Save the current flags so we can tell if they've changed */ + flags = s->flags; + + if (cb) + s->flags |= PA_SOURCE_HW_MUTE_CTRL; + else + s->flags &= ~PA_SOURCE_HW_MUTE_CTRL; + + /* If the flags have changed after init, let any clients know via a change event */ + if (s->state != PA_SOURCE_INIT && flags != s->flags) + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); +} + +static void enable_flat_volume(pa_source *s, bool enable) { + pa_source_flags_t flags; + + pa_assert(s); + + /* Always follow the overall user preference here */ + enable = enable && s->core->flat_volumes; + + /* Save the current flags so we can tell if they've changed */ + flags = s->flags; + + if (enable) + s->flags |= PA_SOURCE_FLAT_VOLUME; + else + s->flags &= ~PA_SOURCE_FLAT_VOLUME; + + /* If the flags have changed after init, let any clients know via a change event */ + if (s->state != PA_SOURCE_INIT && flags != s->flags) + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); +} + +void pa_source_enable_decibel_volume(pa_source *s, bool enable) { + pa_source_flags_t flags; + + pa_assert(s); + + /* Save the current flags so we can tell if they've changed */ + flags = s->flags; + + if (enable) { + s->flags |= PA_SOURCE_DECIBEL_VOLUME; + enable_flat_volume(s, true); + } else { + s->flags &= ~PA_SOURCE_DECIBEL_VOLUME; + enable_flat_volume(s, false); + } + + /* If the flags have changed after init, let any clients know via a change event */ + if (s->state != PA_SOURCE_INIT && flags != s->flags) + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); } /* Called from main context */ @@ -424,8 +540,18 @@ void pa_source_put(pa_source *s) { pa_assert(s->thread_info.min_latency <= s->thread_info.max_latency); /* Generally, flags should be initialized via pa_source_new(). As a - * special exception we allow volume related flags to be set - * between _new() and _put(). */ + * special exception we allow some volume related flags to be set + * between _new() and _put() by the callback setter functions above. + * + * Thus we implement a couple safeguards here which ensure the above + * setters were used (or at least the implementor made manual changes + * in a compatible way). + * + * Note: All of these flags set here can change over the life time + * of the source. */ + pa_assert(!(s->flags & PA_SOURCE_HW_VOLUME_CTRL) || s->set_volume); + pa_assert(!(s->flags & PA_SOURCE_DEFERRED_VOLUME) || s->write_volume); + pa_assert(!(s->flags & PA_SOURCE_HW_MUTE_CTRL) || s->set_mute); /* XXX: Currently decibel volume is disabled for all sources that use volume * sharing. When the master source supports decibel volume, it would be good @@ -434,17 +560,22 @@ void pa_source_put(pa_source *s) { * a master source to another. One solution for this problem would be to * remove user-visible volume altogether from filter sources when volume * sharing is used, but the current approach was easier to implement... */ + /* We always support decibel volumes in software, otherwise we leave it to + * the source implementor to set this flag as needed. + * + * Note: This flag can also change over the life time of the source. */ if (!(s->flags & PA_SOURCE_HW_VOLUME_CTRL) && !(s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) - s->flags |= PA_SOURCE_DECIBEL_VOLUME; + pa_source_enable_decibel_volume(s, true); - if ((s->flags & PA_SOURCE_DECIBEL_VOLUME) && s->core->flat_volumes) - s->flags |= PA_SOURCE_FLAT_VOLUME; + /* If the source implementor support DB volumes by itself, we should always + * try and enable flat volumes too */ + if ((s->flags & PA_SOURCE_DECIBEL_VOLUME)) + enable_flat_volume(s, true); if (s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER) { - pa_source *root_source = s->output_from_master->source; + pa_source *root_source = pa_source_get_master(s); - while (root_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER) - root_source = root_source->output_from_master->source; + pa_assert(PA_LIKELY(root_source)); s->reference_volume = root_source->reference_volume; pa_cvolume_remap(&s->reference_volume, &root_source->channel_map, &s->channel_map); @@ -466,12 +597,11 @@ void pa_source_put(pa_source *s) { && ((s->flags & PA_SOURCE_DECIBEL_VOLUME || (s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER))))); pa_assert(!(s->flags & PA_SOURCE_DECIBEL_VOLUME) || s->n_volume_steps == PA_VOLUME_NORM+1); pa_assert(!(s->flags & PA_SOURCE_DYNAMIC_LATENCY) == (s->thread_info.fixed_latency != 0)); - pa_assert(!(s->flags & PA_SOURCE_HW_VOLUME_CTRL) || s->set_volume); - pa_assert(!(s->flags & PA_SOURCE_SYNC_VOLUME) || (s->flags & PA_SOURCE_HW_VOLUME_CTRL)); - pa_assert(!(s->flags & PA_SOURCE_SYNC_VOLUME) || s->write_volume); - pa_assert(!(s->flags & PA_SOURCE_HW_MUTE_CTRL) || s->set_mute); - pa_assert_se(source_set_state(s, PA_SOURCE_IDLE) == 0); + if (s->suspend_cause) + pa_assert_se(source_set_state(s, PA_SOURCE_SUSPENDED) == 0); + else + pa_assert_se(source_set_state(s, PA_SOURCE_IDLE) == 0); pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE | PA_SUBSCRIPTION_EVENT_NEW, s->index); pa_hook_fire(&s->core->hooks[PA_CORE_HOOK_SOURCE_PUT], s); @@ -479,7 +609,7 @@ void pa_source_put(pa_source *s) { /* Called from main context */ void pa_source_unlink(pa_source *s) { - pa_bool_t linked; + bool linked; pa_source_output *o, *j = NULL; pa_assert(s); @@ -521,7 +651,6 @@ void pa_source_unlink(pa_source *s) { /* Called from main context */ static void source_free(pa_object *o) { - pa_source_output *so; pa_source *s = PA_SOURCE(o); pa_assert(s); @@ -533,12 +662,8 @@ static void source_free(pa_object *o) { pa_log_info("Freeing source %u \"%s\"", s->index, s->name); - pa_idxset_free(s->outputs, NULL, NULL); - - while ((so = pa_hashmap_steal_first(s->thread_info.outputs))) - pa_source_output_unref(so); - - pa_hashmap_free(s->thread_info.outputs, NULL, NULL); + pa_idxset_free(s->outputs, NULL); + pa_hashmap_free(s->thread_info.outputs); if (s->silence.memblock) pa_memblock_unref(s->silence.memblock); @@ -549,14 +674,8 @@ static void source_free(pa_object *o) { if (s->proplist) pa_proplist_free(s->proplist); - if (s->ports) { - pa_device_port *p; - - while ((p = pa_hashmap_steal_first(s->ports))) - pa_device_port_free(p); - - pa_hashmap_free(s->ports, NULL, NULL); - } + if (s->ports) + pa_hashmap_free(s->ports); pa_xfree(s); } @@ -571,16 +690,36 @@ void pa_source_set_asyncmsgq(pa_source *s, pa_asyncmsgq *q) { /* Called from main context, and not while the IO thread is active, please */ void pa_source_update_flags(pa_source *s, pa_source_flags_t mask, pa_source_flags_t value) { + pa_source_flags_t old_flags; + pa_source_output *output; + uint32_t idx; + pa_source_assert_ref(s); pa_assert_ctl_context(); - if (mask == 0) - return; - /* For now, allow only a minimal set of flags to be changed. */ pa_assert((mask & ~(PA_SOURCE_DYNAMIC_LATENCY|PA_SOURCE_LATENCY)) == 0); + old_flags = s->flags; s->flags = (s->flags & ~mask) | (value & mask); + + if (s->flags == old_flags) + return; + + if ((s->flags & PA_SOURCE_LATENCY) != (old_flags & PA_SOURCE_LATENCY)) + pa_log_debug("Source %s: LATENCY flag %s.", s->name, (s->flags & PA_SOURCE_LATENCY) ? "enabled" : "disabled"); + + if ((s->flags & PA_SOURCE_DYNAMIC_LATENCY) != (old_flags & PA_SOURCE_DYNAMIC_LATENCY)) + pa_log_debug("Source %s: DYNAMIC_LATENCY flag %s.", + s->name, (s->flags & PA_SOURCE_DYNAMIC_LATENCY) ? "enabled" : "disabled"); + + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE | PA_SUBSCRIPTION_EVENT_CHANGE, s->index); + pa_hook_fire(&s->core->hooks[PA_CORE_HOOK_SOURCE_FLAGS_CHANGED], s); + + PA_IDXSET_FOREACH(output, s->outputs, idx) { + if (output->destination_source) + pa_source_update_flags(output->destination_source, mask, value); + } } /* Called from IO context, or before _put() from main context */ @@ -603,8 +742,13 @@ int pa_source_update_status(pa_source*s) { return source_set_state(s, pa_source_used_by(s) ? PA_SOURCE_RUNNING : PA_SOURCE_IDLE); } +/* Called from any context - must be threadsafe */ +void pa_source_set_mixer_dirty(pa_source *s, bool is_dirty) { + pa_atomic_store(&s->mixer_dirty, is_dirty ? 1 : 0); +} + /* Called from main context */ -int pa_source_suspend(pa_source *s, pa_bool_t suspend, pa_suspend_cause_t cause) { +int pa_source_suspend(pa_source *s, bool suspend, pa_suspend_cause_t cause) { pa_source_assert_ref(s); pa_assert_ctl_context(); pa_assert(PA_SOURCE_IS_LINKED(s->state)); @@ -618,6 +762,27 @@ int pa_source_suspend(pa_source *s, pa_bool_t suspend, pa_suspend_cause_t cause) else s->suspend_cause &= ~cause; + if (!(s->suspend_cause & PA_SUSPEND_SESSION) && (pa_atomic_load(&s->mixer_dirty) != 0)) { + /* This might look racy but isn't: If somebody sets mixer_dirty exactly here, + it'll be handled just fine. */ + pa_source_set_mixer_dirty(s, false); + pa_log_debug("Mixer is now accessible. Updating alsa mixer settings."); + if (s->active_port && s->set_port) { + if (s->flags & PA_SOURCE_DEFERRED_VOLUME) { + struct source_message_set_port msg = { .port = s->active_port, .ret = 0 }; + pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_SET_PORT, &msg, 0, NULL) == 0); + } + else + s->set_port(s, s->active_port); + } + else { + if (s->set_mute) + s->set_mute(s); + if (s->set_volume) + s->set_volume(s); + } + } + if ((pa_source_get_state(s) == PA_SOURCE_SUSPENDED) == !!s->suspend_cause) return 0; @@ -675,7 +840,7 @@ pa_queue *pa_source_move_all_start(pa_source *s, pa_queue *q) { } /* Called from main context */ -void pa_source_move_all_finish(pa_source *s, pa_queue *q, pa_bool_t save) { +void pa_source_move_all_finish(pa_source *s, pa_queue *q, bool save) { pa_source_output *o; pa_source_assert_ref(s); @@ -690,7 +855,7 @@ void pa_source_move_all_finish(pa_source *s, pa_queue *q, pa_bool_t save) { pa_source_output_unref(o); } - pa_queue_free(q, NULL, NULL); + pa_queue_free(q, NULL); } /* Called from main context */ @@ -705,7 +870,7 @@ void pa_source_move_all_fail(pa_queue *q) { pa_source_output_unref(o); } - pa_queue_free(q, NULL, NULL); + pa_queue_free(q, NULL); } /* Called from IO thread context */ @@ -804,6 +969,119 @@ void pa_source_post_direct(pa_source*s, pa_source_output *o, const pa_memchunk * pa_source_output_push(o, chunk); } +/* Called from main thread */ +int pa_source_update_rate(pa_source *s, uint32_t rate, bool passthrough) { + int ret; + uint32_t desired_rate = rate; + uint32_t default_rate = s->default_sample_rate; + uint32_t alternate_rate = s->alternate_sample_rate; + bool use_alternate = false; + + if (rate == s->sample_spec.rate) + return 0; + + if (!s->update_rate && !s->monitor_of) + return -1; + + if (PA_UNLIKELY(default_rate == alternate_rate && !passthrough)) { + pa_log_debug("Default and alternate sample rates are the same."); + return -1; + } + + if (PA_SOURCE_IS_RUNNING(s->state)) { + pa_log_info("Cannot update rate, SOURCE_IS_RUNNING, will keep using %u Hz", + s->sample_spec.rate); + return -1; + } + + if (s->monitor_of) { + if (PA_SINK_IS_RUNNING(s->monitor_of->state)) { + pa_log_info("Cannot update rate, this is a monitor source and the sink is running."); + return -1; + } + } + + if (PA_UNLIKELY(!pa_sample_rate_valid(desired_rate))) + return -1; + + if (!passthrough) { + pa_assert((default_rate % 4000 == 0) || (default_rate % 11025 == 0)); + pa_assert((alternate_rate % 4000 == 0) || (alternate_rate % 11025 == 0)); + + if (default_rate % 11025 == 0) { + if ((alternate_rate % 4000 == 0) && (desired_rate % 4000 == 0)) + use_alternate=true; + } else { + /* default is 4000 multiple */ + if ((alternate_rate % 11025 == 0) && (desired_rate % 11025 == 0)) + use_alternate=true; + } + + if (use_alternate) + desired_rate = alternate_rate; + else + desired_rate = default_rate; + } else { + desired_rate = rate; /* use stream sampling rate, discard default/alternate settings */ + } + + if (desired_rate == s->sample_spec.rate) + return -1; + + if (!passthrough && pa_source_used_by(s) > 0) + return -1; + + pa_log_debug("Suspending source %s due to changing the sample rate.", s->name); + pa_source_suspend(s, true, PA_SUSPEND_INTERNAL); + + if (s->update_rate) + ret = s->update_rate(s, desired_rate); + else { + /* This is a monitor source. */ + + /* XXX: This code is written with non-passthrough streams in mind. I + * have no idea whether the behaviour with passthrough streams is + * sensible. */ + if (!passthrough) { + uint32_t old_rate = s->sample_spec.rate; + + s->sample_spec.rate = desired_rate; + ret = pa_sink_update_rate(s->monitor_of, desired_rate, false); + + if (ret < 0) { + /* Changing the sink rate failed, roll back the old rate for + * the monitor source. Why did we set the source rate before + * calling pa_sink_update_rate(), you may ask. The reason is + * that pa_sink_update_rate() tries to update the monitor + * source rate, but we are already in the process of updating + * the monitor source rate, so there's a risk of entering an + * infinite loop. Setting the source rate before calling + * pa_sink_update_rate() makes the rate == s->sample_spec.rate + * check in the beginning of this function return early, so we + * avoid looping. */ + s->sample_spec.rate = old_rate; + } + } else + ret = -1; + } + + if (ret >= 0) { + uint32_t idx; + pa_source_output *o; + + PA_IDXSET_FOREACH(o, s->outputs, idx) { + if (o->state == PA_SOURCE_OUTPUT_CORKED) + pa_source_output_update_rate(o); + } + + pa_log_info("Changed sampling rate successfully"); + } + + pa_source_suspend(s, false, PA_SUSPEND_INTERNAL); + + return ret; +} + /* Called from main thread */ pa_usec_t pa_source_get_latency(pa_source *s) { pa_usec_t usec; @@ -820,6 +1098,13 @@ pa_usec_t pa_source_get_latency(pa_source *s) { pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_GET_LATENCY, &usec, 0, NULL) == 0); + /* usec is unsigned, so check that the offset can be added to usec without + * underflowing. */ + if (-s->latency_offset <= (int64_t) usec) + usec += s->latency_offset; + else + usec = 0; + return usec; } @@ -847,6 +1132,13 @@ pa_usec_t pa_source_get_latency_within_thread(pa_source *s) { if (o->process_msg(o, PA_SOURCE_MESSAGE_GET_LATENCY, &usec, 0, NULL) < 0) return -1; + /* usec is unsigned, so check that the offset can be added to usec without + * underflowing. */ + if (-s->thread_info.latency_offset <= (int64_t) usec) + usec += s->thread_info.latency_offset; + else + usec = 0; + return usec; } @@ -856,17 +1148,34 @@ pa_usec_t pa_source_get_latency_within_thread(pa_source *s) { * When a source uses volume sharing, it never has the PA_SOURCE_FLAT_VOLUME flag * set. Instead, flat volume mode is detected by checking whether the root source * has the flag set. */ -pa_bool_t pa_source_flat_volume_enabled(pa_source *s) { +bool pa_source_flat_volume_enabled(pa_source *s) { pa_source_assert_ref(s); - while (s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER) + s = pa_source_get_master(s); + + if (PA_LIKELY(s)) + return (s->flags & PA_SOURCE_FLAT_VOLUME); + else + return false; +} + +/* Called from the main thread (and also from the IO thread while the main + * thread is waiting). */ +pa_source *pa_source_get_master(pa_source *s) { + pa_source_assert_ref(s); + + while (s && (s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) { + if (PA_UNLIKELY(!s->output_from_master)) + return NULL; + s = s->output_from_master->source; + } - return (s->flags & PA_SOURCE_FLAT_VOLUME); + return s; } /* Called from main context */ -pa_bool_t pa_source_is_passthrough(pa_source *s) { +bool pa_source_is_passthrough(pa_source *s) { pa_source_assert_ref(s); @@ -874,6 +1183,27 @@ pa_bool_t pa_source_is_passthrough(pa_source *s) { return (s->monitor_of && pa_sink_is_passthrough(s->monitor_of)); } +/* Called from main context */ +void pa_source_enter_passthrough(pa_source *s) { + pa_cvolume volume; + + /* set the volume to NORM */ + s->saved_volume = *pa_source_get_volume(s, true); + s->saved_save_volume = s->save_volume; + + pa_cvolume_set(&volume, s->sample_spec.channels, PA_MIN(s->base_volume, PA_VOLUME_NORM)); + pa_source_set_volume(s, &volume, true, false); +} + +/* Called from main context */ +void pa_source_leave_passthrough(pa_source *s) { + /* Restore source volume to what it was before we entered passthrough mode */ + pa_source_set_volume(s, &s->saved_volume, true, s->saved_save_volume); + + pa_cvolume_init(&s->saved_volume); + s->saved_save_volume = false; +} + /* Called from main context. */ static void compute_reference_ratio(pa_source_output *o) { unsigned c = 0; @@ -1056,7 +1386,7 @@ static void get_maximum_output_volume(pa_source *s, pa_cvolume *max_volume, cons /* Ignore this output. The origin source uses volume sharing, so this * output's volume will be set to be equal to the root source's real - * volume. Obviously this outputs's current volume must not then + * volume. Obviously this output's current volume must not then * affect what the root source's real volume will be. */ continue; } @@ -1069,7 +1399,7 @@ static void get_maximum_output_volume(pa_source *s, pa_cvolume *max_volume, cons /* Called from main thread. Only called for the root source in volume sharing * cases, except for internal recursive calls. */ -static pa_bool_t has_outputs(pa_source *s) { +static bool has_outputs(pa_source *s) { pa_source_output *o; uint32_t idx; @@ -1077,10 +1407,10 @@ static pa_bool_t has_outputs(pa_source *s) { PA_IDXSET_FOREACH(o, s->outputs, idx) { if (!o->destination_source || !(o->destination_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER) || has_outputs(o->destination_source)) - return TRUE; + return true; } - return FALSE; + return false; } /* Called from main thread. Only called for the root source in volume sharing @@ -1099,20 +1429,13 @@ static void update_real_volume(pa_source *s, const pa_cvolume *new_volume, pa_ch PA_IDXSET_FOREACH(o, s->outputs, idx) { if (o->destination_source && (o->destination_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) { if (pa_source_flat_volume_enabled(s)) { - pa_cvolume old_volume = o->volume; + pa_cvolume new_output_volume; /* Follow the root source's real volume. */ - o->volume = *new_volume; - pa_cvolume_remap(&o->volume, channel_map, &o->channel_map); + new_output_volume = *new_volume; + pa_cvolume_remap(&new_output_volume, channel_map, &o->channel_map); + pa_source_output_set_volume_direct(o, &new_output_volume); compute_reference_ratio(o); - - /* The volume changed, let's tell people so */ - if (!pa_cvolume_equal(&old_volume, &o->volume)) { - if (o->volume_changed) - o->volume_changed(o); - - pa_subscription_post(o->core, PA_SUBSCRIPTION_EVENT_SOURCE_OUTPUT|PA_SUBSCRIPTION_EVENT_CHANGE, o->index); - } } update_real_volume(o->destination_source, new_volume, channel_map); @@ -1167,7 +1490,7 @@ static void propagate_reference_volume(pa_source *s) { * source output volumes accordingly */ PA_IDXSET_FOREACH(o, s->outputs, idx) { - pa_cvolume old_volume; + pa_cvolume new_volume; if (o->destination_source && (o->destination_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) { propagate_reference_volume(o->destination_source); @@ -1178,33 +1501,23 @@ static void propagate_reference_volume(pa_source *s) { continue; } - old_volume = o->volume; - /* This basically calculates: * * o->volume := o->reference_volume * o->reference_ratio */ - o->volume = s->reference_volume; - pa_cvolume_remap(&o->volume, &s->channel_map, &o->channel_map); - pa_sw_cvolume_multiply(&o->volume, &o->volume, &o->reference_ratio); - - /* The volume changed, let's tell people so */ - if (!pa_cvolume_equal(&old_volume, &o->volume)) { - - if (o->volume_changed) - o->volume_changed(o); - - pa_subscription_post(o->core, PA_SUBSCRIPTION_EVENT_SOURCE_OUTPUT|PA_SUBSCRIPTION_EVENT_CHANGE, o->index); - } + new_volume = s->reference_volume; + pa_cvolume_remap(&new_volume, &s->channel_map, &o->channel_map); + pa_sw_cvolume_multiply(&new_volume, &new_volume, &o->reference_ratio); + pa_source_output_set_volume_direct(o, &new_volume); } } /* Called from main thread. Only called for the root source in volume sharing * cases, except for internal recursive calls. The return value indicates * whether any reference volume actually changed. */ -static pa_bool_t update_reference_volume(pa_source *s, const pa_cvolume *v, const pa_channel_map *channel_map, pa_bool_t save) { +static bool update_reference_volume(pa_source *s, const pa_cvolume *v, const pa_channel_map *channel_map, bool save) { pa_cvolume volume; - pa_bool_t reference_volume_changed; + bool reference_volume_changed; pa_source_output *o; uint32_t idx; @@ -1218,13 +1531,11 @@ static pa_bool_t update_reference_volume(pa_source *s, const pa_cvolume *v, cons pa_cvolume_remap(&volume, channel_map, &s->channel_map); reference_volume_changed = !pa_cvolume_equal(&volume, &s->reference_volume); - s->reference_volume = volume; + pa_source_set_reference_volume_direct(s, &volume); s->save_volume = (!reference_volume_changed && s->save_volume) || save; - if (reference_volume_changed) - pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); - else if (!(s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) + if (!reference_volume_changed && !(s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) /* If the root source's volume doesn't change, then there can't be any * changes in the other source in the source tree either. * @@ -1233,29 +1544,29 @@ static pa_bool_t update_reference_volume(pa_source *s, const pa_cvolume *v, cons * due to rounding errors. If that happens, we still want to propagate * the changed root source volume to the sources connected to the * intermediate source that didn't change its volume. This theoretical - * possiblity is the reason why we have that !(s->flags & + * possibility is the reason why we have that !(s->flags & * PA_SOURCE_SHARE_VOLUME_WITH_MASTER) condition. Probably nobody would - * notice even if we returned here FALSE always if - * reference_volume_changed is FALSE. */ - return FALSE; + * notice even if we returned here false always if + * reference_volume_changed is false. */ + return false; PA_IDXSET_FOREACH(o, s->outputs, idx) { if (o->destination_source && (o->destination_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) - update_reference_volume(o->destination_source, v, channel_map, FALSE); + update_reference_volume(o->destination_source, v, channel_map, false); } - return TRUE; + return true; } /* Called from main thread */ void pa_source_set_volume( pa_source *s, const pa_cvolume *volume, - pa_bool_t send_msg, - pa_bool_t save) { + bool send_msg, + bool save) { pa_cvolume new_reference_volume; - pa_source *root_source = s; + pa_source *root_source; pa_source_assert_ref(s); pa_assert_ctl_context(); @@ -1264,17 +1575,19 @@ void pa_source_set_volume( pa_assert(volume || pa_source_flat_volume_enabled(s)); pa_assert(!volume || volume->channels == 1 || pa_cvolume_compatible(volume, &s->sample_spec)); - /* make sure we don't change the volume when a PASSTHROUGH output is connected */ - if (pa_source_is_passthrough(s)) { - /* FIXME: Need to notify client that volume control is disabled */ - pa_log_warn("Cannot change volume, Source is monitor of a PASSTHROUGH sink"); + /* make sure we don't change the volume in PASSTHROUGH mode ... + * ... *except* if we're being invoked to reset the volume to ensure 0 dB gain */ + if (pa_source_is_passthrough(s) && (!volume || !pa_cvolume_is_norm(volume))) { + pa_log_warn("Cannot change volume, source is monitor of a PASSTHROUGH sink"); return; } /* In case of volume sharing, the volume is set for the root source first, * from which it's then propagated to the sharing sources. */ - while (root_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER) - root_source = root_source->output_from_master->source; + root_source = pa_source_get_master(s); + + if (PA_UNLIKELY(!root_source)) + return; /* As a special exception we accept mono volumes on all sources -- * even on those with more complex channel maps */ @@ -1288,13 +1601,7 @@ void pa_source_set_volume( } pa_cvolume_remap(&new_reference_volume, &s->channel_map, &root_source->channel_map); - } - - /* If volume is NULL we synchronize the source's real and reference - * volumes with the stream volumes. If it is not NULL we update - * the reference_volume with it. */ - if (volume) { if (update_reference_volume(root_source, &new_reference_volume, &root_source->channel_map, save)) { if (pa_source_flat_volume_enabled(root_source)) { /* OK, propagate this volume change back to the outputs */ @@ -1307,6 +1614,9 @@ void pa_source_set_volume( } } else { + /* If volume is NULL we synchronize the source's real and + * reference volumes with the stream volumes. */ + pa_assert(pa_source_flat_volume_enabled(root_source)); /* Ok, let's determine the new real volume */ @@ -1314,6 +1624,9 @@ void pa_source_set_volume( /* Let's 'push' the reference volume if necessary */ pa_cvolume_merge(&new_reference_volume, &s->reference_volume, &root_source->real_volume); + /* If the source and its root don't have the same number of channels, we need to remap */ + if (s != root_source && !pa_channel_map_equal(&s->channel_map, &root_source->channel_map)) + pa_cvolume_remap(&new_reference_volume, &s->channel_map, &root_source->channel_map); update_reference_volume(root_source, &new_reference_volume, &root_source->channel_map, save); /* Now that the reference volume is updated, we can update the streams' @@ -1327,7 +1640,7 @@ void pa_source_set_volume( * apply one to root_source->soft_volume */ pa_cvolume_reset(&root_source->soft_volume, root_source->sample_spec.channels); - if (!(root_source->flags & PA_SOURCE_SYNC_VOLUME)) + if (!(root_source->flags & PA_SOURCE_DEFERRED_VOLUME)) root_source->set_volume(root_source); } else @@ -1347,7 +1660,7 @@ void pa_source_set_soft_volume(pa_source *s, const pa_cvolume *volume) { pa_source_assert_ref(s); pa_assert(!(s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)); - if (s->flags & PA_SOURCE_SYNC_VOLUME) + if (s->flags & PA_SOURCE_DEFERRED_VOLUME) pa_source_assert_io_context(s); else pa_assert_ctl_context(); @@ -1357,7 +1670,7 @@ void pa_source_set_soft_volume(pa_source *s, const pa_cvolume *volume) { else s->soft_volume = *volume; - if (PA_SOURCE_IS_LINKED(s->state) && !(s->flags & PA_SOURCE_SYNC_VOLUME)) + if (PA_SOURCE_IS_LINKED(s->state) && !(s->flags & PA_SOURCE_DEFERRED_VOLUME)) pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_SET_VOLUME, NULL, 0, NULL) == 0); else s->thread_info.soft_volume = s->soft_volume; @@ -1384,13 +1697,12 @@ static void propagate_real_volume(pa_source *s, const pa_cvolume *old_real_volum return; /* 1. Make the real volume the reference volume */ - update_reference_volume(s, &s->real_volume, &s->channel_map, TRUE); + update_reference_volume(s, &s->real_volume, &s->channel_map, true); } if (pa_source_flat_volume_enabled(s)) { - PA_IDXSET_FOREACH(o, s->outputs, idx) { - pa_cvolume old_volume = o->volume; + pa_cvolume new_volume; /* 2. Since the source's reference and real volumes are equal * now our ratios should be too. */ @@ -1404,18 +1716,10 @@ static void propagate_real_volume(pa_source *s, const pa_cvolume *old_real_volum * o->volume = s->reference_volume * o->reference_ratio * * This is identical to propagate_reference_volume() */ - o->volume = s->reference_volume; - pa_cvolume_remap(&o->volume, &s->channel_map, &o->channel_map); - pa_sw_cvolume_multiply(&o->volume, &o->volume, &o->reference_ratio); - - /* Notify if something changed */ - if (!pa_cvolume_equal(&old_volume, &o->volume)) { - - if (o->volume_changed) - o->volume_changed(o); - - pa_subscription_post(o->core, PA_SUBSCRIPTION_EVENT_SOURCE_OUTPUT|PA_SUBSCRIPTION_EVENT_CHANGE, o->index); - } + new_volume = s->reference_volume; + pa_cvolume_remap(&new_volume, &s->channel_map, &o->channel_map); + pa_sw_cvolume_multiply(&new_volume, &new_volume, &o->reference_ratio); + pa_source_output_set_volume_direct(o, &new_volume); if (o->destination_source && (o->destination_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) propagate_real_volume(o->destination_source, old_real_volume); @@ -1426,7 +1730,7 @@ static void propagate_real_volume(pa_source *s, const pa_cvolume *old_real_volum * to save changed hw settings given that hw volume changes not * triggered by PA are almost certainly done by the user. */ if (!(s->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER)) - s->save_volume = TRUE; + s->save_volume = true; } /* Called from io thread */ @@ -1438,7 +1742,7 @@ void pa_source_update_volume_and_mute(pa_source *s) { } /* Called from main thread */ -const pa_cvolume *pa_source_get_volume(pa_source *s, pa_bool_t force_refresh) { +const pa_cvolume *pa_source_get_volume(pa_source *s, bool force_refresh) { pa_source_assert_ref(s); pa_assert_ctl_context(); pa_assert(PA_SOURCE_IS_LINKED(s->state)); @@ -1450,7 +1754,7 @@ const pa_cvolume *pa_source_get_volume(pa_source *s, pa_bool_t force_refresh) { old_real_volume = s->real_volume; - if (!(s->flags & PA_SOURCE_SYNC_VOLUME) && s->get_volume) + if (!(s->flags & PA_SOURCE_DEFERRED_VOLUME) && s->get_volume) s->get_volume(s); pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_GET_VOLUME, NULL, 0, NULL) == 0); @@ -1480,43 +1784,48 @@ void pa_source_volume_changed(pa_source *s, const pa_cvolume *new_real_volume) { } /* Called from main thread */ -void pa_source_set_mute(pa_source *s, pa_bool_t mute, pa_bool_t save) { - pa_bool_t old_muted; +void pa_source_set_mute(pa_source *s, bool mute, bool save) { + bool old_muted; pa_source_assert_ref(s); pa_assert_ctl_context(); pa_assert(PA_SOURCE_IS_LINKED(s->state)); old_muted = s->muted; + + if (mute == old_muted) { + s->save_muted |= save; + return; + } + s->muted = mute; - s->save_muted = (old_muted == s->muted && s->save_muted) || save; + s->save_muted = save; - if (!(s->flags & PA_SOURCE_SYNC_VOLUME) && s->set_mute) + if (!(s->flags & PA_SOURCE_DEFERRED_VOLUME) && s->set_mute) s->set_mute(s); + pa_log_debug("The mute of source %s changed from %s to %s.", s->name, pa_yes_no(old_muted), pa_yes_no(mute)); pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_SET_MUTE, NULL, 0, NULL) == 0); - - if (old_muted != s->muted) - pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); } /* Called from main thread */ -pa_bool_t pa_source_get_mute(pa_source *s, pa_bool_t force_refresh) { +bool pa_source_get_mute(pa_source *s, bool force_refresh) { pa_source_assert_ref(s); pa_assert_ctl_context(); pa_assert(PA_SOURCE_IS_LINKED(s->state)); if (s->refresh_muted || force_refresh) { - pa_bool_t old_muted = s->muted; + bool old_muted = s->muted; - if (!(s->flags & PA_SOURCE_SYNC_VOLUME) && s->get_mute) + if (!(s->flags & PA_SOURCE_DEFERRED_VOLUME) && s->get_mute) s->get_mute(s); pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_GET_MUTE, NULL, 0, NULL) == 0); if (old_muted != s->muted) { - s->save_muted = TRUE; + s->save_muted = true; pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); @@ -1529,7 +1838,7 @@ pa_bool_t pa_source_get_mute(pa_source *s, pa_bool_t force_refresh) { } /* Called from main thread */ -void pa_source_mute_changed(pa_source *s, pa_bool_t new_muted) { +void pa_source_mute_changed(pa_source *s, bool new_muted) { pa_source_assert_ref(s); pa_assert_ctl_context(); pa_assert(PA_SOURCE_IS_LINKED(s->state)); @@ -1540,13 +1849,13 @@ void pa_source_mute_changed(pa_source *s, pa_bool_t new_muted) { return; s->muted = new_muted; - s->save_muted = TRUE; + s->save_muted = true; pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); } /* Called from main thread */ -pa_bool_t pa_source_update_proplist(pa_source *s, pa_update_mode_t mode, pa_proplist *p) { +bool pa_source_update_proplist(pa_source *s, pa_update_mode_t mode, pa_proplist *p) { pa_source_assert_ref(s); pa_assert_ctl_context(); @@ -1558,7 +1867,7 @@ pa_bool_t pa_source_update_proplist(pa_source *s, pa_update_mode_t mode, pa_prop pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); } - return TRUE; + return true; } /* Called from main thread */ @@ -1632,7 +1941,7 @@ unsigned pa_source_check_suspend(pa_source *s) { /* We do not assert here. It is perfectly valid for a source output to * be in the INIT state (i.e. created, marked done but not yet put) * and we should not care if it's unlinked as it won't contribute - * towarards our busy status. + * towards our busy status. */ if (!PA_SOURCE_OUTPUT_IS_LINKED(st)) continue; @@ -1662,7 +1971,7 @@ static void sync_output_volumes_within_thread(pa_source *s) { continue; o->thread_info.soft_volume = o->soft_volume; - //pa_source_output_request_rewind(o, 0, TRUE, FALSE, FALSE); + //pa_source_output_request_rewind(o, 0, true, false, false); } } @@ -1700,7 +2009,7 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ } pa_assert(!o->thread_info.attached); - o->thread_info.attached = TRUE; + o->thread_info.attached = true; if (o->attach) o->attach(o); @@ -1731,17 +2040,15 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ o->detach(o); pa_assert(o->thread_info.attached); - o->thread_info.attached = FALSE; + o->thread_info.attached = false; if (o->thread_info.direct_on_input) { pa_hashmap_remove(o->thread_info.direct_on_input->thread_info.direct_outputs, PA_UINT32_TO_PTR(o->index)); o->thread_info.direct_on_input = NULL; } - if (pa_hashmap_remove(s->thread_info.outputs, PA_UINT32_TO_PTR(o->index))) - pa_source_output_unref(o); - - pa_source_invalidate_requested_latency(s, TRUE); + pa_hashmap_remove_and_free(s->thread_info.outputs, PA_UINT32_TO_PTR(o->index)); + pa_source_invalidate_requested_latency(s, true); /* In flat volume mode we need to update the volume as * well */ @@ -1749,18 +2056,17 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ } case PA_SOURCE_MESSAGE_SET_SHARED_VOLUME: { - pa_source *root_source = s; + pa_source *root_source = pa_source_get_master(s); - while (root_source->flags & PA_SOURCE_SHARE_VOLUME_WITH_MASTER) - root_source = root_source->output_from_master->source; + if (PA_LIKELY(root_source)) + set_shared_volume_within_thread(root_source); - set_shared_volume_within_thread(root_source); return 0; } case PA_SOURCE_MESSAGE_SET_VOLUME_SYNCED: - if (s->flags & PA_SOURCE_SYNC_VOLUME) { + if (s->flags & PA_SOURCE_DEFERRED_VOLUME) { s->set_volume(s); pa_source_volume_change_push(s); } @@ -1780,7 +2086,7 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ case PA_SOURCE_MESSAGE_GET_VOLUME: - if ((s->flags & PA_SOURCE_SYNC_VOLUME) && s->get_volume) { + if ((s->flags & PA_SOURCE_DEFERRED_VOLUME) && s->get_volume) { s->get_volume(s); pa_source_volume_change_flush(s); pa_sw_cvolume_divide(&s->thread_info.current_hw_volume, &s->real_volume, &s->soft_volume); @@ -1799,21 +2105,21 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ s->thread_info.soft_muted = s->muted; } - if (s->flags & PA_SOURCE_SYNC_VOLUME && s->set_mute) + if (s->flags & PA_SOURCE_DEFERRED_VOLUME && s->set_mute) s->set_mute(s); return 0; case PA_SOURCE_MESSAGE_GET_MUTE: - if (s->flags & PA_SOURCE_SYNC_VOLUME && s->get_mute) + if (s->flags & PA_SOURCE_DEFERRED_VOLUME && s->get_mute) s->get_mute(s); return 0; case PA_SOURCE_MESSAGE_SET_STATE: { - pa_bool_t suspend_change = + bool suspend_change = (s->thread_info.state == PA_SOURCE_SUSPENDED && PA_SOURCE_IS_OPENED(PA_PTR_TO_UINT(userdata))) || (PA_SOURCE_IS_OPENED(s->thread_info.state) && PA_PTR_TO_UINT(userdata) == PA_SOURCE_SUSPENDED); @@ -1831,18 +2137,6 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ return 0; } - case PA_SOURCE_MESSAGE_DETACH: - - /* Detach all streams */ - pa_source_detach_within_thread(s); - return 0; - - case PA_SOURCE_MESSAGE_ATTACH: - - /* Reattach all streams */ - pa_source_attach_within_thread(s); - return 0; - case PA_SOURCE_MESSAGE_GET_REQUESTED_LATENCY: { pa_usec_t *usec = userdata; @@ -1917,8 +2211,16 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ /* This message is sent from IO-thread and handled in main thread. */ pa_assert_ctl_context(); - pa_source_get_volume(s, TRUE); - pa_source_get_mute(s, TRUE); + /* Make sure we're not messing with main thread when no longer linked */ + if (!PA_SOURCE_IS_LINKED(s->state)) + return 0; + + pa_source_get_volume(s, true); + pa_source_get_mute(s, true); + return 0; + + case PA_SOURCE_MESSAGE_SET_LATENCY_OFFSET: + s->thread_info.latency_offset = offset; return 0; case PA_SOURCE_MESSAGE_MAX: @@ -1929,7 +2231,7 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_ } /* Called from main thread */ -int pa_source_suspend_all(pa_core *c, pa_bool_t suspend, pa_suspend_cause_t cause) { +int pa_source_suspend_all(pa_core *c, bool suspend, pa_suspend_cause_t cause) { pa_source *source; uint32_t idx; int ret = 0; @@ -1951,24 +2253,6 @@ int pa_source_suspend_all(pa_core *c, pa_bool_t suspend, pa_suspend_cause_t caus return ret; } -/* Called from main thread */ -void pa_source_detach(pa_source *s) { - pa_source_assert_ref(s); - pa_assert_ctl_context(); - pa_assert(PA_SOURCE_IS_LINKED(s->state)); - - pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_DETACH, NULL, 0, NULL) == 0); -} - -/* Called from main thread */ -void pa_source_attach(pa_source *s) { - pa_source_assert_ref(s); - pa_assert_ctl_context(); - pa_assert(PA_SOURCE_IS_LINKED(s->state)); - - pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_ATTACH, NULL, 0, NULL) == 0); -} - /* Called from IO thread */ void pa_source_detach_within_thread(pa_source *s) { pa_source_output *o; @@ -2023,7 +2307,7 @@ pa_usec_t pa_source_get_requested_latency_within_thread(pa_source *s) { if (PA_SOURCE_IS_LINKED(s->thread_info.state)) { /* Only cache this if we are fully set up */ s->thread_info.requested_latency = result; - s->thread_info.requested_latency_valid = TRUE; + s->thread_info.requested_latency_valid = true; } return result; @@ -2075,7 +2359,7 @@ void pa_source_set_max_rewind(pa_source *s, size_t max_rewind) { } /* Called from IO thread */ -void pa_source_invalidate_requested_latency(pa_source *s, pa_bool_t dynamic) { +void pa_source_invalidate_requested_latency(pa_source *s, bool dynamic) { pa_source_output *o; void *state = NULL; @@ -2083,7 +2367,7 @@ void pa_source_invalidate_requested_latency(pa_source *s, pa_bool_t dynamic) { pa_source_assert_io_context(s); if ((s->flags & PA_SOURCE_DYNAMIC_LATENCY)) - s->thread_info.requested_latency_valid = FALSE; + s->thread_info.requested_latency_valid = false; else if (dynamic) return; @@ -2187,7 +2471,7 @@ void pa_source_set_latency_range_within_thread(pa_source *s, pa_usec_t min_laten o->update_source_latency_range(o); } - pa_source_invalidate_requested_latency(s, FALSE); + pa_source_invalidate_requested_latency(s, false); } /* Called from main thread, before the source is put */ @@ -2237,6 +2521,8 @@ void pa_source_set_fixed_latency_within_thread(pa_source *s, pa_usec_t latency) if (s->flags & PA_SOURCE_DYNAMIC_LATENCY) { pa_assert(latency == 0); + s->thread_info.fixed_latency = 0; + return; } @@ -2257,7 +2543,19 @@ void pa_source_set_fixed_latency_within_thread(pa_source *s, pa_usec_t latency) o->update_source_fixed_latency(o); } - pa_source_invalidate_requested_latency(s, FALSE); + pa_source_invalidate_requested_latency(s, false); +} + +/* Called from main thread */ +void pa_source_set_latency_offset(pa_source *s, int64_t offset) { + pa_source_assert_ref(s); + + s->latency_offset = offset; + + if (PA_SOURCE_IS_LINKED(s->state)) + pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_SET_LATENCY_OFFSET, NULL, offset, NULL) == 0); + else + s->thread_info.latency_offset = offset; } /* Called from main thread */ @@ -2275,7 +2573,7 @@ size_t pa_source_get_max_rewind(pa_source *s) { } /* Called from main context */ -int pa_source_set_port(pa_source *s, const char *name, pa_bool_t save) { +int pa_source_set_port(pa_source *s, const char *name, bool save) { pa_device_port *port; int ret; @@ -2287,7 +2585,7 @@ int pa_source_set_port(pa_source *s, const char *name, pa_bool_t save) { return -PA_ERR_NOTIMPLEMENTED; } - if (!s->ports) + if (!name) return -PA_ERR_NOENTITY; if (!(port = pa_hashmap_get(s->ports, name))) @@ -2298,7 +2596,7 @@ int pa_source_set_port(pa_source *s, const char *name, pa_bool_t save) { return 0; } - if (s->flags & PA_SOURCE_SYNC_VOLUME) { + if (s->flags & PA_SOURCE_DEFERRED_VOLUME) { struct source_message_set_port msg = { .port = port, .ret = 0 }; pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_SET_PORT, &msg, 0, NULL) == 0); ret = msg.ret; @@ -2425,24 +2723,33 @@ static void pa_source_volume_change_flush(pa_source *s) { } /* Called from the IO thread. */ -pa_bool_t pa_source_volume_change_apply(pa_source *s, pa_usec_t *usec_to_next) { - pa_usec_t now = pa_rtclock_now(); - pa_bool_t ret = FALSE; +bool pa_source_volume_change_apply(pa_source *s, pa_usec_t *usec_to_next) { + pa_usec_t now; + bool ret = false; pa_assert(s); + + if (!s->thread_info.volume_changes || !PA_SOURCE_IS_LINKED(s->state)) { + if (usec_to_next) + *usec_to_next = 0; + return ret; + } + pa_assert(s->write_volume); + now = pa_rtclock_now(); + while (s->thread_info.volume_changes && now >= s->thread_info.volume_changes->at) { pa_source_volume_change *c = s->thread_info.volume_changes; PA_LLIST_REMOVE(pa_source_volume_change, s->thread_info.volume_changes, c); pa_log_debug("Volume change to %d at %llu was written %llu usec late", pa_cvolume_avg(&c->hw_volume), (long long unsigned) c->at, (long long unsigned) (now - c->at)); - ret = TRUE; + ret = true; s->thread_info.current_hw_volume = c->hw_volume; pa_source_volume_change_free(c); } - if (s->write_volume && ret) + if (ret) s->write_volume(s); if (s->thread_info.volume_changes) { @@ -2459,7 +2766,6 @@ pa_bool_t pa_source_volume_change_apply(pa_source *s, pa_usec_t *usec_to_next) { return ret; } - /* Called from the main thread */ /* Gets the list of formats supported by the source. The members and idxset must * be freed by the caller. */ @@ -2485,10 +2791,9 @@ pa_idxset* pa_source_get_formats(pa_source *s) { /* Called from the main thread */ /* Checks if the source can accept this format */ -pa_bool_t pa_source_check_format(pa_source *s, pa_format_info *f) -{ +bool pa_source_check_format(pa_source *s, pa_format_info *f) { pa_idxset *formats = NULL; - pa_bool_t ret = FALSE; + bool ret = false; pa_assert(s); pa_assert(f); @@ -2501,12 +2806,12 @@ pa_bool_t pa_source_check_format(pa_source *s, pa_format_info *f) PA_IDXSET_FOREACH(finfo_device, formats, i) { if (pa_format_info_is_compatible(finfo_device, f)) { - ret = TRUE; + ret = true; break; } } - pa_idxset_free(formats, (pa_free2_cb_t) pa_format_info_free2, NULL); + pa_idxset_free(formats, (pa_free_cb_t) pa_format_info_free); } return ret; @@ -2536,7 +2841,31 @@ pa_idxset* pa_source_check_formats(pa_source *s, pa_idxset *in_formats) { done: if (source_formats) - pa_idxset_free(source_formats, (pa_free2_cb_t) pa_format_info_free2, NULL); + pa_idxset_free(source_formats, (pa_free_cb_t) pa_format_info_free); return out_formats; } + +/* Called from the main thread. */ +void pa_source_set_reference_volume_direct(pa_source *s, const pa_cvolume *volume) { + pa_cvolume old_volume; + char old_volume_str[PA_CVOLUME_SNPRINT_VERBOSE_MAX]; + char new_volume_str[PA_CVOLUME_SNPRINT_VERBOSE_MAX]; + + pa_assert(s); + pa_assert(volume); + + old_volume = s->reference_volume; + + if (pa_cvolume_equal(volume, &old_volume)) + return; + + s->reference_volume = *volume; + pa_log_debug("The reference volume of source %s changed from %s to %s.", s->name, + pa_cvolume_snprint_verbose(old_volume_str, sizeof(old_volume_str), &old_volume, &s->channel_map, + s->flags & PA_SOURCE_DECIBEL_VOLUME), + pa_cvolume_snprint_verbose(new_volume_str, sizeof(new_volume_str), volume, &s->channel_map, + s->flags & PA_SOURCE_DECIBEL_VOLUME)); + + pa_subscription_post(s->core, PA_SUBSCRIPTION_EVENT_SOURCE|PA_SUBSCRIPTION_EVENT_CHANGE, s->index); +}