Changes in kernel/generic/src/sysinfo/stats.c [169815e:07d4271] in mainline

File:

• kernel/generic/src/sysinfo/stats.c (modified) (7 diffs)

kernel/generic/src/sysinfo/stats.c

@@ -221 +221 @@
         stats_task->virtmem = get_task_virtmem(task->as);
         stats_task->resmem = get_task_resmem(task->as);
-        stats_task->threads = atomic_load(&task->refcount);
+        stats_task->threads = atomic_load(&task->lifecount);
         task_get_accounting(task, &(stats_task->ucycles),
             &(stats_task->kcycles));
@@ -299 +299 @@
 {
         assert(interrupts_disabled());
-        assert(irq_spinlock_locked(&thread->lock));
 
         stats_thread->thread_id = thread->tid;
         stats_thread->task_id = thread->task->taskid;
-        stats_thread->state = thread->state;
-        stats_thread->priority = thread->priority;
-        stats_thread->ucycles = thread->ucycles;
-        stats_thread->kcycles = thread->kcycles;
-
-        if (thread->cpu != NULL) {
+        stats_thread->state = atomic_get_unordered(&thread->state);
+        stats_thread->priority = atomic_get_unordered(&thread->priority);
+        stats_thread->ucycles = atomic_time_read(&thread->ucycles);
+        stats_thread->kcycles = atomic_time_read(&thread->kcycles);
+
+        cpu_t *cpu = atomic_get_unordered(&thread->cpu);
+
+        if (cpu != NULL) {
                 stats_thread->on_cpu = true;
-                stats_thread->cpu = thread->cpu->id;
+                stats_thread->cpu = cpu->id;
         } else
                 stats_thread->on_cpu = false;
@@ -361 +362 @@
         thread_t *thread = thread_first();
         while (thread != NULL) {
-                /* Interrupts are already disabled */
-                irq_spinlock_lock(&thread->lock, false);
-
                 /* Record the statistics and increment the index */
                 produce_stats_thread(thread, &stats_threads[i]);
                 i++;
-
-                irq_spinlock_unlock(&thread->lock, false);
 
                 thread = thread_next(thread);
@@ -515 +511 @@
 {
         /* Initially no return value */
-        sysinfo_return_t ret;
-        ret.tag = SYSINFO_VAL_UNDEFINED;
+        sysinfo_return_t ret = {
+                .tag = SYSINFO_VAL_UNDEFINED,
+        };
 
         /* Parse the task ID */
@@ -523 +520 @@
                 return ret;
 
-        /* Messing with task structures, avoid deadlock */
-        irq_spinlock_lock(&tasks_lock, true);
-
         task_t *task = task_find_by_id(task_id);
-        if (task == NULL) {
-                /* No task with this ID */
-                irq_spinlock_unlock(&tasks_lock, true);
+        if (!task)
                 return ret;
-        }
 
         if (dry_run) {
@@ -537 +528 @@
                 ret.data.data = NULL;
                 ret.data.size = sizeof(stats_task_t);
-
-                irq_spinlock_unlock(&tasks_lock, true);
         } else {
                 /* Allocate stats_task_t structure */
-                stats_task_t *stats_task =
-                    (stats_task_t *) malloc(sizeof(stats_task_t));
-                if (stats_task == NULL) {
-                        irq_spinlock_unlock(&tasks_lock, true);
-                        return ret;
+                stats_task_t *stats_task = malloc(sizeof(stats_task_t));
+
+                if (stats_task != NULL) {
+                        /* Correct return value */
+                        ret.tag = SYSINFO_VAL_FUNCTION_DATA;
+                        ret.data.data = stats_task;
+                        ret.data.size = sizeof(stats_task_t);
+
+                        irq_spinlock_lock(&task->lock, true);
+                        produce_stats_task(task, stats_task);
+                        irq_spinlock_unlock(&task->lock, true);
                 }
-
-                /* Correct return value */
-                ret.tag = SYSINFO_VAL_FUNCTION_DATA;
-                ret.data.data = (void *) stats_task;
-                ret.data.size = sizeof(stats_task_t);
-
-                /* Hand-over-hand locking */
-                irq_spinlock_exchange(&tasks_lock, &task->lock);
-
-                produce_stats_task(task, stats_task);
-
-                irq_spinlock_unlock(&task->lock, true);
-        }
-
+        }
+
+        task_release(task);
         return ret;
 }
@@ -624 +608 @@
                 ret.data.size = sizeof(stats_thread_t);
 
-                /*
-                 * Replaced hand-over-hand locking with regular nested sections
-                 * to avoid weak reference leak issues.
-                 */
-                irq_spinlock_lock(&thread->lock, false);
                 produce_stats_thread(thread, stats_thread);
-                irq_spinlock_unlock(&thread->lock, false);
 
                 irq_spinlock_unlock(&threads_lock, true);