Changes in / [91d860d:48cc66f] in mainline

Location:

uspace/srv/logger

Files:

• initlvl.c (modified) (1 diff)
• logger.h (modified) (2 diffs)
• logs.c (modified) (7 diffs)

uspace/srv/logger/initlvl.c

@@ -67 +67 @@
 
         log->logged_level = level;
-        refcount_up(&log->ref_counter);
+        log->ref_counter++;
 
         log_unlock(log);

uspace/srv/logger/logger.h

@@ -41 +41 @@
 #include <async.h>
 #include <stdbool.h>
-#include <refcount.h>
 #include <fibril_synch.h>
 #include <stdio.h>
@@ -65 +64 @@
         link_t link;
 
-        atomic_refcount_t ref_counter;
+        size_t ref_counter;
 
         fibril_mutex_t guard;

uspace/srv/logger/logs.c

@@ -99 +99 @@
         fibril_mutex_initialize(&result->guard);
         link_initialize(&result->link);
-        refcount_init(&result->ref_counter);
-        refcount_up(&result->ref_counter);
         result->parent = parent;
 
@@ -127 +125 @@
                 list_append(&result->link, &log_list);
                 if (result->parent != NULL) {
-                        refcount_up(&result->parent->ref_counter);
+                        fibril_mutex_lock(&result->parent->guard);
+                        result->parent->ref_counter++;
+                        fibril_mutex_unlock(&result->parent->guard);
                 }
         }
@@ -209 +209 @@
 {
         assert(fibril_mutex_is_locked(&log->guard));
-
-        if (!refcount_down(&log->ref_counter)) {
-                /* We are definitely not the last ones. */
+        assert(log->ref_counter > 0);
+
+        /* We are definitely not the last ones. */
+        if (log->ref_counter > 1) {
+                log->ref_counter--;
                 fibril_mutex_unlock(&log->guard);
                 return;
         }
-
-
 
         /*
@@ -225 +225 @@
          *   LOCKED(list), wants to LOCK(log)
          */
-        refcount_up(&log->ref_counter);
         fibril_mutex_unlock(&log->guard);
 
@@ -235 +234 @@
          */
         fibril_mutex_lock(&log->guard);
-
-        if (!refcount_down(&log->ref_counter)) {
+        assert(log->ref_counter > 0);
+        log->ref_counter--;
+
+        if (log->ref_counter > 0) {
                 /*
                  * Meanwhile, someone else increased the ref counter.
@@ -254 +255 @@
          * - destroy dest (if top-level log)
          */
+        assert(log->ref_counter == 0);
+
         list_remove(&log->link);
         fibril_mutex_unlock(&log_list_guard);
@@ -311 +314 @@
 
         assert(fibril_mutex_is_locked(&new_log->guard));
-        refcount_up(&new_log->ref_counter);
+        new_log->ref_counter++;
 
         logs->logs[logs->logs_count] = new_log;