mirror of
https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git
synced 2025-11-03 09:01:50 -05:00
rescure-stream: handle failed moves as well as dying sinks/sources
This commit is contained in:
Lennart Poettering
parent
e4db56bf07
commit
fd1266c666
1 changed files with 138 additions and 39 deletions
|
|
@ -45,13 +45,43 @@ static const char* const valid_modargs[] = {
|
|||
};
|
||||
|
||||
struct userdata {
|
||||
pa_hook_slot *sink_slot, *source_slot;
|
||||
pa_hook_slot
|
||||
*sink_unlink_slot,
|
||||
*source_unlink_slot,
|
||||
*sink_input_move_fail_slot,
|
||||
*source_output_move_fail_slot;
|
||||
};
|
||||
|
||||
static pa_hook_result_t sink_hook_callback(pa_core *c, pa_sink *sink, void* userdata) {
|
||||
static pa_sink* find_evacuation_sink(pa_core *c, pa_sink_input *i, pa_sink *skip) {
|
||||
pa_sink *target, *def;
|
||||
uint32_t idx;
|
||||
|
||||
pa_assert(c);
|
||||
pa_assert(i);
|
||||
|
||||
def = pa_namereg_get_default_sink(c);
|
||||
|
||||
if (def && def != skip && pa_sink_input_may_move_to(i, def))
|
||||
return def;
|
||||
|
||||
PA_IDXSET_FOREACH(target, c->sinks, idx) {
|
||||
if (target == def)
|
||||
continue;
|
||||
|
||||
if (target == skip)
|
||||
continue;
|
||||
|
||||
if (pa_sink_input_may_move_to(i, target))
|
||||
return target;
|
||||
}
|
||||
|
||||
pa_log_debug("No evacuation sink found.");
|
||||
return NULL;
|
||||
}
|
||||
|
||||
static pa_hook_result_t sink_unlink_hook_callback(pa_core *c, pa_sink *sink, void* userdata) {
|
||||
pa_sink_input *i;
|
||||
uint32_t idx;
|
||||
pa_sink *target;
|
||||
|
||||
pa_assert(c);
|
||||
pa_assert(sink);
|
||||
|
|
@ -65,21 +95,12 @@ static pa_hook_result_t sink_hook_callback(pa_core *c, pa_sink *sink, void* user
|
|||
return PA_HOOK_OK;
|
||||
}
|
||||
|
||||
if (!(target = pa_namereg_get_default_sink(c)) || target == sink) {
|
||||
|
||||
PA_IDXSET_FOREACH(target, c->sinks, idx)
|
||||
if (target != sink)
|
||||
break;
|
||||
|
||||
if (!target) {
|
||||
pa_log_debug("No evacuation sink found.");
|
||||
return PA_HOOK_OK;
|
||||
}
|
||||
}
|
||||
|
||||
pa_assert(target != sink);
|
||||
|
||||
PA_IDXSET_FOREACH(i, sink->inputs, idx) {
|
||||
pa_sink *target;
|
||||
|
||||
if (!(target = find_evacuation_sink(c, i, sink)))
|
||||
continue;
|
||||
|
||||
if (pa_sink_input_move_to(i, target, FALSE) < 0)
|
||||
pa_log_info("Failed to move sink input %u \"%s\" to %s.", i->index,
|
||||
pa_strnull(pa_proplist_gets(i->proplist, PA_PROP_APPLICATION_NAME)), target->name);
|
||||
|
|
@ -91,9 +112,63 @@ static pa_hook_result_t sink_hook_callback(pa_core *c, pa_sink *sink, void* user
|
|||
return PA_HOOK_OK;
|
||||
}
|
||||
|
||||
static pa_hook_result_t source_hook_callback(pa_core *c, pa_source *source, void* userdata) {
|
||||
static pa_hook_result_t sink_input_move_fail_hook_callback(pa_core *c, pa_sink_input *i, void *userdata) {
|
||||
pa_sink *target;
|
||||
|
||||
pa_assert(c);
|
||||
pa_assert(i);
|
||||
|
||||
/* There's no point in doing anything if the core is shut down anyway */
|
||||
if (c->state == PA_CORE_SHUTDOWN)
|
||||
return PA_HOOK_OK;
|
||||
|
||||
if (!(target = find_evacuation_sink(c, i, NULL)))
|
||||
return PA_HOOK_OK;
|
||||
|
||||
if (pa_sink_input_finish_move(i, target, FALSE) < 0) {
|
||||
pa_log_info("Failed to move sink input %u \"%s\" to %s.", i->index,
|
||||
pa_strnull(pa_proplist_gets(i->proplist, PA_PROP_APPLICATION_NAME)), target->name);
|
||||
return PA_HOOK_OK;
|
||||
|
||||
} else {
|
||||
pa_log_info("Sucessfully moved sink input %u \"%s\" to %s.", i->index,
|
||||
pa_strnull(pa_proplist_gets(i->proplist, PA_PROP_APPLICATION_NAME)), target->name);
|
||||
return PA_HOOK_STOP;
|
||||
}
|
||||
}
|
||||
|
||||
static pa_source* find_evacuation_source(pa_core *c, pa_source_output *o, pa_source *skip) {
|
||||
pa_source *target, *def;
|
||||
uint32_t idx;
|
||||
|
||||
pa_assert(c);
|
||||
pa_assert(o);
|
||||
|
||||
def = pa_namereg_get_default_source(c);
|
||||
|
||||
if (def && def != skip && pa_source_output_may_move_to(o, def))
|
||||
return def;
|
||||
|
||||
PA_IDXSET_FOREACH(target, c->sources, idx) {
|
||||
if (target == def)
|
||||
continue;
|
||||
|
||||
if (target == skip)
|
||||
continue;
|
||||
|
||||
if (!target->monitor_of != !skip->monitor_of)
|
||||
continue;
|
||||
|
||||
if (pa_source_output_may_move_to(o, target))
|
||||
return target;
|
||||
}
|
||||
|
||||
pa_log_debug("No evacuation source found.");
|
||||
return NULL;
|
||||
}
|
||||
|
||||
static pa_hook_result_t source_unlink_hook_callback(pa_core *c, pa_source *source, void* userdata) {
|
||||
pa_source_output *o;
|
||||
pa_source *target;
|
||||
uint32_t idx;
|
||||
|
||||
pa_assert(c);
|
||||
|
|
@ -108,21 +183,12 @@ static pa_hook_result_t source_hook_callback(pa_core *c, pa_source *source, void
|
|||
return PA_HOOK_OK;
|
||||
}
|
||||
|
||||
if (!(target = pa_namereg_get_default_source(c)) || target == source) {
|
||||
|
||||
PA_IDXSET_FOREACH(target, c->sources, idx)
|
||||
if (target != source && !target->monitor_of == !source->monitor_of)
|
||||
break;
|
||||
|
||||
if (!target) {
|
||||
pa_log_info("No evacuation source found.");
|
||||
return PA_HOOK_OK;
|
||||
}
|
||||
}
|
||||
|
||||
pa_assert(target != source);
|
||||
|
||||
PA_IDXSET_FOREACH(o, source->outputs, idx) {
|
||||
pa_source *target;
|
||||
|
||||
if (!(target = find_evacuation_source(c, o, source)))
|
||||
continue;
|
||||
|
||||
if (pa_source_output_move_to(o, target, FALSE) < 0)
|
||||
pa_log_info("Failed to move source output %u \"%s\" to %s.", o->index,
|
||||
pa_strnull(pa_proplist_gets(o->proplist, PA_PROP_APPLICATION_NAME)), target->name);
|
||||
|
|
@ -134,6 +200,31 @@ static pa_hook_result_t source_hook_callback(pa_core *c, pa_source *source, void
|
|||
return PA_HOOK_OK;
|
||||
}
|
||||
|
||||
static pa_hook_result_t source_output_move_fail_hook_callback(pa_core *c, pa_source_output *i, void *userdata) {
|
||||
pa_source *target;
|
||||
|
||||
pa_assert(c);
|
||||
pa_assert(i);
|
||||
|
||||
/* There's no point in doing anything if the core is shut down anyway */
|
||||
if (c->state == PA_CORE_SHUTDOWN)
|
||||
return PA_HOOK_OK;
|
||||
|
||||
if (!(target = find_evacuation_source(c, i, NULL)))
|
||||
return PA_HOOK_OK;
|
||||
|
||||
if (pa_source_output_finish_move(i, target, FALSE) < 0) {
|
||||
pa_log_info("Failed to move source input %u \"%s\" to %s.", i->index,
|
||||
pa_strnull(pa_proplist_gets(i->proplist, PA_PROP_APPLICATION_NAME)), target->name);
|
||||
return PA_HOOK_OK;
|
||||
|
||||
} else {
|
||||
pa_log_info("Sucessfully moved source input %u \"%s\" to %s.", i->index,
|
||||
pa_strnull(pa_proplist_gets(i->proplist, PA_PROP_APPLICATION_NAME)), target->name);
|
||||
return PA_HOOK_STOP;
|
||||
}
|
||||
}
|
||||
|
||||
int pa__init(pa_module*m) {
|
||||
pa_modargs *ma;
|
||||
struct userdata *u;
|
||||
|
|
@ -148,8 +239,11 @@ int pa__init(pa_module*m) {
|
|||
m->userdata = u = pa_xnew(struct userdata, 1);
|
||||
|
||||
/* A little bit later than module-stream-restore, module-intended-roles... */
|
||||
u->sink_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SINK_UNLINK], PA_HOOK_LATE+20, (pa_hook_cb_t) sink_hook_callback, u);
|
||||
u->source_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SOURCE_UNLINK], PA_HOOK_LATE+20, (pa_hook_cb_t) source_hook_callback, u);
|
||||
u->sink_unlink_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SINK_UNLINK], PA_HOOK_LATE+20, (pa_hook_cb_t) sink_unlink_hook_callback, u);
|
||||
u->source_unlink_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SOURCE_UNLINK], PA_HOOK_LATE+20, (pa_hook_cb_t) source_unlink_hook_callback, u);
|
||||
|
||||
u->sink_input_move_fail_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SINK_INPUT_MOVE_FAIL], PA_HOOK_LATE+20, (pa_hook_cb_t) sink_input_move_fail_hook_callback, u);
|
||||
u->source_output_move_fail_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SOURCE_OUTPUT_MOVE_FAIL], PA_HOOK_LATE+20, (pa_hook_cb_t) source_output_move_fail_hook_callback, u);
|
||||
|
||||
pa_modargs_free(ma);
|
||||
return 0;
|
||||
|
|
@ -163,10 +257,15 @@ void pa__done(pa_module*m) {
|
|||
if (!(u = m->userdata))
|
||||
return;
|
||||
|
||||
if (u->sink_slot)
|
||||
pa_hook_slot_free(u->sink_slot);
|
||||
if (u->source_slot)
|
||||
pa_hook_slot_free(u->source_slot);
|
||||
if (u->sink_unlink_slot)
|
||||
pa_hook_slot_free(u->sink_unlink_slot);
|
||||
if (u->source_unlink_slot)
|
||||
pa_hook_slot_free(u->source_unlink_slot);
|
||||
|
||||
if (u->sink_input_move_fail_slot)
|
||||
pa_hook_slot_free(u->sink_input_move_fail_slot);
|
||||
if (u->source_output_move_fail_slot)
|
||||
pa_hook_slot_free(u->source_output_move_fail_slot);
|
||||
|
||||
pa_xfree(u);
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue