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

Timestamp:

2019-08-03T08:28:26Z (5 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/configuration.c (modified) (8 diffs)

uspace/srv/sysman/configuration.c

@@ -39 +39 @@
 
 static hash_table_t units;
-static fibril_rwlock_t units_rwl;
 
 /* Hash table functions */
@@ -81 +80 @@
 {
         hash_table_create(&units, 0, 0, &units_ht_ops);
-        fibril_rwlock_initialize(&units_rwl);
 }
 
@@ -89 +87 @@
         assert(unit->state == STATE_EMBRYO);
         assert(unit->name != NULL);
-        assert(fibril_rwlock_is_write_locked(&units_rwl));
         sysman_log(LVL_DEBUG2, "%s('%s')", __func__, unit_name(unit));
 
@@ -100 +97 @@
 
 void configuration_start_update(void) {
-        assert(!fibril_rwlock_is_write_locked(&units_rwl));
-        sysman_log(LVL_DEBUG2, "%s", __func__);
-        fibril_rwlock_write_lock(&units_rwl);
+        sysman_log(LVL_DEBUG2, "%s", __func__);
 }
 
@@ -124 +119 @@
 void configuration_commit(void)
 {
-        assert(fibril_rwlock_is_write_locked(&units_rwl));
         sysman_log(LVL_DEBUG2, "%s", __func__);
 
@@ -132 +126 @@
          */
         hash_table_apply(&units, &configuration_commit_unit, NULL);
-        fibril_rwlock_write_unlock(&units_rwl);
 }
 
@@ -157 +150 @@
 void configuration_rollback(void)
 {
-        assert(fibril_rwlock_is_write_locked(&units_rwl));
         sysman_log(LVL_DEBUG2, "%s", __func__);
 
         hash_table_apply(&units, &configuration_rollback_unit, NULL);
-        fibril_rwlock_write_unlock(&units_rwl);
 }
 
@@ -199 +190 @@
 int configuration_resolve_dependecies(void)
 {
-        assert(fibril_rwlock_is_write_locked(&units_rwl));
         sysman_log(LVL_DEBUG2, "%s", __func__);