Changeset 821 for Whitix/trunk/kernel/sched.c

Timestamp:

08/06/08 14:33:26 (4 years ago)

Author:

mwhitworth

Message:

Fix recalculation of priorities to avoid overflowing the signed quantum variable.

Files:

• Whitix/trunk/kernel/sched.c (modified) (4 diffs)

Whitix/trunk/kernel/sched.c

@@ -95 +95 @@
 
         ListForEachEntry(curr,&threadList,list)
-        {
+        {               
                 if (curr->quantums > c && curr->state == THR_RUNNING)
                 {
@@ -107 +107 @@
         {
                 ListForEachEntry(curr,&threadList,list)
-                        curr->quantums=curr->priority + (curr->quantums << 2);
+                        curr->quantums=curr->priority + (curr->quantums >> 1);
         }
 
@@ -129 +129 @@
         DWORD flags;
         IrqSaveFlags(flags);
+
+#if 0
+        if (currThread && currThread->preemptCount)
+        {
+                KePrint("currThread->preemptCount = %u\n", currThread->preemptCount); 
+                ModuleSymbolPrint(__builtin_return_address(0));
+                cli(); hlt();
+        }
+#endif
 
         /* See what's happening to the current thread - does it want to be freed? */
@@ -143 +152 @@
         current=currThread->parent;
 
-                /* The only threads without a parent process are kernel threads - who
-                 * don't need their own address space as they only manipulate kernel
-                 * memory, which is present in every address space.
-                 */
+        /* The only threads without a parent process are kernel threads - who
+         * don't need their own address space as they only manipulate kernel
+         * memory, which is present in every address space.
+         */
 
         if (current)