Changeset 418 for trunk/kernel/kern

Timestamp:

Jan 8, 2018, 2:25:39 PM (6 years ago)

Author:

alain

Message:

Fix a bug in hal_kentry.S : the "uzone" pointer in the thread descriptor
must not be modified in case of interrupt.

Location:

trunk/kernel/kern

Files:

• chdev.c (modified) (3 diffs)
• thread.c (modified) (1 diff)
• thread.h (modified) (1 diff)

trunk/kernel/kern/chdev.c

@@ -123 +123 @@
     uint32_t   save_sr;       // for critical section
 
+#if CONFIG_READ_DEBUG
+enter_chdev_cmd = hal_time_stamp();
+#endif
+
     thread_t * this = CURRENT_THREAD;
 
@@ -187 +191 @@
     hal_restore_irq( save_sr );
 
+#if CONFIG_READ_DEBUG
+exit_chdev_cmd = hal_time_stamp();
+#endif
+
 }  // end chdev_register_command()
 
@@ -253 +261 @@
             xlist_unlink( XPTR( client_cxy , &client_ptr->wait_list ) );
             remote_spinlock_unlock( lock_xp );
+
+            // unblock client thread
+            thread_unblock( client_xp , THREAD_BLOCKED_IO );
 
 chdev_dmsg("\n[DBG] %s : thread %x complete operation for client %x / cycle %d\n",

trunk/kernel/kern/thread.c

@@ -868 +868 @@
 
         }
-        else                                // yield each ~ 100000 cycles
-
+        else                                // search a runable thread
         {
-             hal_fixed_delay( 500000 );
+            sched_yield( "IDLE" );
         }
-
-        // force scheduling at each iteration
-        sched_yield( "idle" );
-   }
+    }
 }  // end thread_idle()
 

trunk/kernel/kern/thread.h

@@ -133 +133 @@
         void              * cpu_context;     /*! pointer on CPU context switch            */
         void              * fpu_context;     /*! pointer on FPU context switch            */
-    void              * uzone;           /*! pointer on uzone for hal_kentry          */
+    void              * uzone;           /*! used by hal_do_syscakl & hal_do_except   */
 
         intptr_t            k_stack_base;    /*! kernel stack base address                */