Changeset 5861b60 in mainline

Timestamp:

2024-01-15T15:15:14Z (12 months ago)

Author:

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

Branches:

master

Children:

a064d4f

Parents:

c7ceacf

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2023-03-12 15:06:58)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2024-01-15 15:15:14)

Message:

Lift actions that do not need separate context out of scheduler_separated_stack()

We're making scheduler_separated_stack() as simple as possible so that later
we can switch from thread to thread directly whenever we don't need to wait.

File:

• kernel/generic/src/proc/scheduler.c (modified) (2 diffs)

kernel/generic/src/proc/scheduler.c

@@ -449 +449 @@
         THREAD->kcycles += get_cycle() - THREAD->last_cycle;
 
+        after_thread_ran_arch();
+
+        if (new_state == Sleeping) {
+                /* Prefer the thread after it's woken up. */
+                THREAD->priority = -1;
+        }
+
         if (!context_save(&THREAD->saved_context)) {
                 /*
@@ -506 +513 @@
 
         if (THREAD) {
-                after_thread_ran_arch();
-
                 state_t state = THREAD->state;
-
-                if (state == Sleeping) {
-                        /* Prefer the thread after it's woken up. */
-                        THREAD->priority = -1;
-                }
-
                 irq_spinlock_unlock(&THREAD->lock, false);