Changeset dfa4be62 in mainline for kernel/generic/src/sysinfo/stats.c

Timestamp:

2024-01-21T16:23:19Z (4 months ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

master

Children:

d23712e

Parents:

a3d87b9

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2023-03-28 17:40:43)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2024-01-21 16:23:19)

Message:

Thread lock is no longer necessary

File:

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

kernel/generic/src/sysinfo/stats.c

@@ -362 +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);
@@ -625 +620 @@
                 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);