Changeset 4760793 in mainline for kernel/generic/src/proc/scheduler.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/proc/scheduler.c (modified) (5 diffs)

kernel/generic/src/proc/scheduler.c

@@ -216 +216 @@
 
                 /* This is safe because interrupts are disabled. */
-                CPU->preempt_deadline = CPU->current_clock_tick + us2ticks(time_to_run);
+                CPU_LOCAL->preempt_deadline =
+                    CPU_LOCAL->current_clock_tick + us2ticks(time_to_run);
 
                 /*
@@ -257 +258 @@
                  * This improves energy saving and hyperthreading.
                  */
-                CPU->idle = true;
+                CPU_LOCAL->idle = true;
 
                 /*
@@ -305 +306 @@
 static void relink_rq(int start)
 {
-        if (CPU->current_clock_tick < CPU->relink_deadline)
+        if (CPU_LOCAL->current_clock_tick < CPU_LOCAL->relink_deadline)
                 return;
 
-        CPU->relink_deadline = CPU->current_clock_tick + NEEDS_RELINK_MAX;
+        CPU_LOCAL->relink_deadline = CPU_LOCAL->current_clock_tick + NEEDS_RELINK_MAX;
 
         /* Temporary cache for lists we are moving. */
@@ -401 +402 @@
          *
          */
-        current_copy(CURRENT, (current_t *) CPU->stack);
+        current_copy(CURRENT, (current_t *) CPU_LOCAL->stack);
 
         /*
@@ -419 +420 @@
         context_save(&ctx);
         context_set(&ctx, FADDR(scheduler_separated_stack),
-            (uintptr_t) CPU->stack, STACK_SIZE);
+            (uintptr_t) CPU_LOCAL->stack, STACK_SIZE);
         context_restore(&ctx);