mirror of
https://github.com/AuxXxilium/eudev.git
synced 2025-01-14 16:04:58 +07:00
unit: check for unneeded dependencies even when unit stop was expected
systemd did not stop units marked as "StopWhenUnneeded=yes" when the requiring unit was stopped on user's request. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=704197
This commit is contained in:
parent
f60c2665f9
commit
cd0504d0a1
12
src/unit.c
12
src/unit.c
@ -1105,6 +1105,14 @@ static void retroactively_stop_dependencies(Unit *u) {
|
|||||||
SET_FOREACH(other, u->meta.dependencies[UNIT_BOUND_BY], i)
|
SET_FOREACH(other, u->meta.dependencies[UNIT_BOUND_BY], i)
|
||||||
if (!UNIT_IS_INACTIVE_OR_DEACTIVATING(unit_active_state(other)))
|
if (!UNIT_IS_INACTIVE_OR_DEACTIVATING(unit_active_state(other)))
|
||||||
manager_add_job(u->meta.manager, JOB_STOP, other, JOB_REPLACE, true, NULL, NULL);
|
manager_add_job(u->meta.manager, JOB_STOP, other, JOB_REPLACE, true, NULL, NULL);
|
||||||
|
}
|
||||||
|
|
||||||
|
static void check_unneeded_dependencies(Unit *u) {
|
||||||
|
Iterator i;
|
||||||
|
Unit *other;
|
||||||
|
|
||||||
|
assert(u);
|
||||||
|
assert(UNIT_IS_INACTIVE_OR_DEACTIVATING(unit_active_state(u)));
|
||||||
|
|
||||||
/* Garbage collect services that might not be needed anymore, if enabled */
|
/* Garbage collect services that might not be needed anymore, if enabled */
|
||||||
SET_FOREACH(other, u->meta.dependencies[UNIT_REQUIRES], i)
|
SET_FOREACH(other, u->meta.dependencies[UNIT_REQUIRES], i)
|
||||||
@ -1263,6 +1271,10 @@ void unit_notify(Unit *u, UnitActiveState os, UnitActiveState ns, bool reload_su
|
|||||||
retroactively_stop_dependencies(u);
|
retroactively_stop_dependencies(u);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* stop unneeded units regardless if going down was expected or not */
|
||||||
|
if (UNIT_IS_ACTIVE_OR_ACTIVATING(os) && UNIT_IS_INACTIVE_OR_DEACTIVATING(ns))
|
||||||
|
check_unneeded_dependencies(u);
|
||||||
|
|
||||||
if (ns != os && ns == UNIT_FAILED) {
|
if (ns != os && ns == UNIT_FAILED) {
|
||||||
log_notice("Unit %s entered failed state.", u->meta.id);
|
log_notice("Unit %s entered failed state.", u->meta.id);
|
||||||
unit_trigger_on_failure(u);
|
unit_trigger_on_failure(u);
|
||||||
|
Loading…
Reference in New Issue
Block a user