Changeset a8f9a82 in mainline

Timestamp:

2005-08-21T22:38:57Z (19 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

6e787c4

Parents:

3526da33

Message:

Add comment to scheduler.c explaining the reason behind saving CPU priority of preempted thread.

Remove two unnecessary cpu_priority_high() calls.

Files:

• arch/ia32/src/smp/smp.c (modified) (1 diff)
• src/main/kinit.c (modified) (1 diff)
• src/proc/scheduler.c (modified) (1 diff)

arch/ia32/src/smp/smp.c

@@ -148 +148 @@
                          */
                         waitq_sleep(&ap_completion_wq);
-                        cpu_priority_high();
                 }
                 else {

src/main/kinit.c

@@ -78 +78 @@
                         thread_ready(t);
                         waitq_sleep(&kmp_completion_wq);
-                        cpu_priority_high();
                 }
                 else panic("thread_create/kmp");

src/proc/scheduler.c

@@ -243 +243 @@
                         return;
                 }
+
+                /*
+                 * CPU priority of preempted thread is recorded here
+                 * to facilitate scheduler() invocations from
+                 * cpu_priority_high()'ed code (e.g. waitq_sleep_timeout()). 
+                 */
                 THREAD->saved_context.pri = pri;
         }