Changeset 16 for trunk/kernel/syscalls/sys_thread_exit.c

Timestamp:

May 10, 2017, 5:04:01 PM (7 years ago)

Author:

alain

Message:

mprove the HAL for interrupt, exception, syscall handling.

File:

• trunk/kernel/syscalls/sys_thread_exit.c (modified) (3 diffs)

trunk/kernel/syscalls/sys_thread_exit.c

@@ -32 +32 @@
 int sys_thread_exit ( void * exit_val )
 {
-        thread_t * this = current_thread;
-    cpu_t    * cpu  = current_cpu;
-        uint32_t   state;
-        bool_t     isEmpty;
-        bool_t     isReleased;
+        thread_t  * this = CURRENT_THREAD;
+    core_t    * core = this->core;
+        bool_t      isEmpty;
+        bool_t      isReleased;
+    uint32_t    irq_state;
 
-        /* TODO: the cpu->lid must match the core index in the logical cluster */
-        if(this->process->pid != 1)
-                dqdt_update_threads_number( local_cxy , cpu->lid , -1 );
+        // update DQDT TODO must be done by thread destroy 
+        if(this->process->pid != 1) dqdt_local_update_threads( -1 );
 
         spinlock_lock( &this->lock );
@@ -46 +45 @@
         if(!(thread_isJoinable(this)))
         {
-                spinlock_unlock_nosched(&this->lock);
                 goto exit_dead;
         }
 
-        // Check if there's a thread waiting the end of callee thread
+        // Check if there's a thread waiting the end of calling thread
         isEmpty = wait_queue_isEmpty(&this->info.wait_queue);
 
@@ -72 +70 @@
 
         // Release FPU if required
-        cpu_disable_all_irq(&state);
-        if(current_cpu->fpu_owner == this)
+        hal_disable_irq( &irq_state );
+        if(core->fpu_owner == this)
         {
-                current_cpu->fpu_owner = NULL;
+                core->fpu_owner = NULL;
                 isReleased = true;
         }
-        cpu_restore_irq(state);
+        hal_restore_irq( irq_state );
 
         if(isReleased)