Changeset 3f7e1f24 in mainline for uspace/srv/sysman/units/unit_cfg.c

Timestamp:

2019-08-03T08:28:26Z (6 years ago)

Author:

Matthieu Riolo <matthieu.riolo@…>

Children:

095d03c

Parents:

d7c5fc0

git-author:

Michal Koutný <xm.koutny+hos@…> (2015-04-22 17:54:08)

git-committer:

Matthieu Riolo <matthieu.riolo@…> (2019-08-03 08:28:26)

Message:

sysman: Refactored job manipulation (event loop + one main fibril)

File:

• uspace/srv/sysman/units/unit_cfg.c (modified) (3 diffs)

uspace/srv/sysman/units/unit_cfg.c

@@ -176 +176 @@
                 }
 
-                assert(unit->state = STATE_EMBRYO);
+                assert(unit->state == STATE_EMBRYO);
                 configuration_add_unit(unit);
         }
@@ -232 +232 @@
         assert(u_cfg);
 
-        /*
-         * Skip starting state and hold state lock during whole configuration
-         * load.
-         */
-        fibril_mutex_lock(&unit->state_mtx);
         int rc = cfg_load_configuration(u_cfg->path);
         
@@ -244 +239 @@
                 unit->state = STATE_FAILED;
         }
-        fibril_condvar_broadcast(&unit->state_cv);
-        fibril_mutex_unlock(&unit->state_mtx);
 
         return rc;