Changeset 4760793 in mainline for kernel/generic/src/time/clock.c

Timestamp:

2024-01-14T18:23:40Z (4 months ago)

Author:

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

Branches:

master

Children:

5663872, c7ceacf

Parents:

3b68542

Message:

Add CPU_LOCAL alongside CPU and segregate fields that are only used locally

This makes it more clear which fields can be used without synchronization
and which need more care.

File:

• kernel/generic/src/time/clock.c (modified) (3 diffs)

kernel/generic/src/time/clock.c

@@ -124 +124 @@
 {
         uint64_t now = get_cycle();
-        atomic_time_increment(&CPU->busy_cycles, now - CPU->last_cycle);
-        CPU->last_cycle = now;
+        atomic_time_increment(&CPU->busy_cycles, now - CPU_LOCAL->last_cycle);
+        CPU_LOCAL->last_cycle = now;
 }
 
@@ -137 +137 @@
 void clock(void)
 {
-        size_t missed_clock_ticks = CPU->missed_clock_ticks;
-        CPU->missed_clock_ticks = 0;
-
-        CPU->current_clock_tick += missed_clock_ticks + 1;
-        uint64_t current_clock_tick = CPU->current_clock_tick;
+        size_t missed_clock_ticks = CPU_LOCAL->missed_clock_ticks;
+        CPU_LOCAL->missed_clock_ticks = 0;
+
+        CPU_LOCAL->current_clock_tick += missed_clock_ticks + 1;
+        uint64_t current_clock_tick = CPU_LOCAL->current_clock_tick;
         clock_update_counters(current_clock_tick);
 
@@ -186 +186 @@
 
         if (THREAD) {
-                if (current_clock_tick >= CPU->preempt_deadline && PREEMPTION_ENABLED) {
+                if (current_clock_tick >= CPU_LOCAL->preempt_deadline && PREEMPTION_ENABLED) {
                         scheduler();
 #ifdef CONFIG_UDEBUG