arch: fix the sched parameter update when exiting

Signed-off-by: buxiasen <buxiasen@xiaomi.com>

arch/arm/src/arm/arm_syscall.c

@@ -152,6 +152,12 @@ uint32_t *arm_syscall(uint32_t *regs)
 
       cpu = this_cpu();
       tcb = current_task(cpu);
+
+      /* Update scheduler parameters */
+
+      nxsched_suspend_scheduler(g_running_tasks[cpu]);
+      nxsched_resume_scheduler(tcb);
+
       g_running_tasks[cpu] = tcb;
 
       /* Restore the cpu lock */

arch/arm/src/common/arm_exit.c

@@ -66,11 +66,9 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases
+  /* Scheduler parameters will update inside syscall */
-   * NOTE: the API also adjusts the global IRQ control for SMP
-   */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/arm64/src/common/arm64_exit.c

@@ -70,11 +70,9 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases
+  /* Scheduler parameters will update inside syscall */
-   * NOTE: the API also adjusts the global IRQ control for SMP
-   */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/avr/src/common/avr_exit.c

@@ -68,6 +68,7 @@ void up_exit(int status)
   /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
 #ifdef CONFIG_ARCH_ADDRENV
   /* Make sure that the address environment for the previously running

arch/hc/src/common/hc_exit.c

@@ -67,6 +67,7 @@ void up_exit(int status)
   /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
 #ifdef CONFIG_ARCH_ADDRENV
   /* Make sure that the address environment for the previously running

arch/mips/src/common/mips_exit.c

@@ -66,11 +66,9 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases
+  /* Scheduler parameters will update inside syscall */
-   * NOTE: the API also adjusts the global IRQ control for SMP
-   */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/misoc/src/lm32/lm32_exit.c

@@ -68,7 +68,7 @@ void up_exit(int status)
 
   /* Adjusts time slice for RR & SPORADIC cases */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/misoc/src/minerva/minerva_exit.c

@@ -68,7 +68,7 @@ void up_exit(int status)
 
   /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/or1k/src/common/or1k_exit.c

@@ -69,6 +69,7 @@ void up_exit(int status)
   /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
 #ifdef CONFIG_ARCH_ADDRENV
   /* Make sure that the address environment for the previously running

arch/renesas/src/common/renesas_exit.c

@@ -64,9 +64,10 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Adjusts time slice for RR & SPORADIC cases */
+  /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
 #ifdef CONFIG_ARCH_ADDRENV
   /* Make sure that the address environment for the previously running

arch/risc-v/src/common/riscv_exit.c

@@ -66,13 +66,7 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases
+  /* Scheduler parameters will update inside syscall */
-   * NOTE: the API also adjusts the global IRQ control for SMP
-   */
-
-  nxsched_resume_scheduler(tcb);
-
-  /* g_running_tasks is not valid now */
 
   g_running_tasks[this_cpu()] = NULL;
 

arch/sim/src/sim/sim_exit.c

@@ -68,6 +68,7 @@ void up_exit(int status)
    */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
   /* Restore the cpu lock */
 

arch/sparc/src/common/sparc_exit.c

@@ -70,9 +70,9 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Reset scheduler parameters */
+  /* Scheduler parameters will update inside syscall */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/tricore/src/common/tricore_exit.c

@@ -66,11 +66,9 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases
+  /* Scheduler parameters will update inside syscall */
-   * NOTE: the API also adjusts the global IRQ control for SMP
-   */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/x86/src/common/x86_exit.c

@@ -70,6 +70,7 @@ void up_exit(int status)
    */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
 #ifdef CONFIG_ARCH_ADDRENV
   /* Make sure that the address environment for the previously running

arch/x86_64/src/common/x86_64_exit.c

@@ -69,6 +69,7 @@ void up_exit(int status)
    */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
   /* Context switch, rearrange MMU */
 

arch/xtensa/src/common/xtensa_exit.c

@@ -66,11 +66,9 @@ void up_exit(int status)
 
   tcb = this_task();
 
-  /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases
+  /* Scheduler parameters will update inside syscall */
-   * NOTE: the API also adjusts the global IRQ control for SMP
-   */
 
-  nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = NULL;
 
   /* Then switch contexts */
 

arch/z16/src/common/z16_exit.c

@@ -68,6 +68,7 @@ void up_exit(int status)
   /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
   /* Then switch contexts */
 

arch/z80/src/common/z80_exit.c

@@ -70,6 +70,7 @@ void up_exit(int status)
   /* Adjusts time slice for SCHED_RR & SCHED_SPORADIC cases */
 
   nxsched_resume_scheduler(tcb);
+  g_running_tasks[this_cpu()] = tcb;
 
 #ifdef CONFIG_ARCH_ADDRENV
   /* Make sure that the address environment for the previously running

sched/sched/sched_suspendscheduler.c

@@ -61,6 +61,13 @@
 
 void nxsched_suspend_scheduler(FAR struct tcb_s *tcb)
 {
+  /* Handle the task exiting case */
+
+  if (tcb != NULL)
+    {
+      return;
+    }
+
 #ifdef CONFIG_SCHED_SPORADIC
   /* Perform sporadic schedule operations */