folkslinux
/
eudev
mirror of https://github.com/eudev-project/eudev.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

target: add implicit target/unit ordering deps only if both sides have been fully loaded

This commit is contained in:
Lennart Poettering 2010-09-14 01:51:30 +02:00
parent 83a95334c9
commit bba34eedc7
3 changed files with 37 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/target.c
View File

@ -53,8 +53,29 @@ static void target_set_state(Target *t, TargetState state) {
}
static int target_add_default_dependencies(Target *t) {
Iterator i;
Unit *other;
int r;
assert(t);
/* Imply ordering for requirement dependencies on target
* units. Note that when the user created a contradicting
* ordering manually we won't add anything in here to make
* sure we don't create a loop. */
SET_FOREACH(other, t->meta.dependencies[UNIT_REQUIRES], i)
if ((r = unit_add_default_target_dependency(other, UNIT(t))) < 0)
return r;
SET_FOREACH(other, t->meta.dependencies[UNIT_REQUIRES_OVERRIDABLE], i)
if ((r = unit_add_default_target_dependency(other, UNIT(t))) < 0)
return r;
SET_FOREACH(other, t->meta.dependencies[UNIT_WANTS], i)
if ((r = unit_add_default_target_dependency(other, UNIT(t))) < 0)
return r;
/* Make sure targets are unloaded on shutdown */
return unit_add_dependency_by_name(UNIT(t), UNIT_CONFLICTED_BY, SPECIAL_SHUTDOWN_TARGET, NULL, true);
}

22
src/unit.c
View File

@ -726,13 +726,19 @@ int unit_load_fragment_and_dropin_optional(Unit *u) {
return 0;
}
static int unit_add_one_default_dependency(Unit *u, Unit *target) {
int unit_add_default_target_dependency(Unit *u, Unit *target) {
assert(u);
assert(target);
if (target->meta.type != UNIT_TARGET)
return 0;
/* Only add the dependency if boths units are loaded, so that
* that loop check below is reliable */
if (u->meta.load_state != UNIT_LOADED ||
target->meta.load_state != UNIT_LOADED)
return 0;
/* Don't create loops */
if (set_get(target->meta.dependencies[UNIT_BEFORE], u))
return 0;
@ -741,22 +747,22 @@ static int unit_add_one_default_dependency(Unit *u, Unit *target) {
}
static int unit_add_default_dependencies(Unit *u) {
Unit *other;
Unit *target;
Iterator i;
int r;
assert(u);
SET_FOREACH(other, u->meta.dependencies[UNIT_REQUIRED_BY], i)
if ((r = unit_add_one_default_dependency(u, other)) < 0)
SET_FOREACH(target, u->meta.dependencies[UNIT_REQUIRED_BY], i)
if ((r = unit_add_default_target_dependency(u, target)) < 0)
return r;
SET_FOREACH(other, u->meta.dependencies[UNIT_REQUIRED_BY_OVERRIDABLE], i)
if ((r = unit_add_one_default_dependency(u, other)) < 0)
SET_FOREACH(target, u->meta.dependencies[UNIT_REQUIRED_BY_OVERRIDABLE], i)
if ((r = unit_add_default_target_dependency(u, target)) < 0)
return r;
SET_FOREACH(other, u->meta.dependencies[UNIT_WANTED_BY], i)
if ((r = unit_add_one_default_dependency(u, other)) < 0)
SET_FOREACH(target, u->meta.dependencies[UNIT_WANTED_BY], i)
if ((r = unit_add_default_target_dependency(u, target)) < 0)
return r;
return 0;

2
src/unit.h
View File

@ -501,6 +501,8 @@ Unit *unit_following(Unit *u);
bool unit_pending_inactive(Unit *u);
int unit_add_default_target_dependency(Unit *u, Unit *target);
const char *unit_load_state_to_string(UnitLoadState i);
UnitLoadState unit_load_state_from_string(const char *s);