arch: Move the DUMP_ON_EXIT logic after nxtask_exit.

Otherwise we will try to dump the state of the current task, however the
exit handler has already started doing some cleanup and invalidated its
group.  Accessing the group from dumponexit will crash.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

arch/arm/src/common/arm_exit.c

@@ -115,15 +115,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/avr/src/common/up_exit.c

@@ -114,15 +114,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/hc/src/common/up_exit.c

@@ -114,15 +114,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/mips/src/common/mips_exit.c

@@ -116,15 +116,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/misoc/src/lm32/lm32_exit.c

@@ -108,15 +108,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/misoc/src/minerva/minerva_exit.c

@@ -108,15 +108,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the head
    * of the list.
    */

arch/or1k/src/common/up_exit.c

@@ -115,15 +115,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/renesas/src/common/up_exit.c

@@ -114,15 +114,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

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

@@ -116,15 +116,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/sparc/src/common/up_exit.c

@@ -137,11 +137,6 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
-#ifdef CONFIG_DUMP_ON_EXIT
-  sinfo("Other tasks:\n");
-  sched_foreach(_up_dumponexit, NULL);
-#endif
-
   /* Update scheduler parameters */
 
   nxsched_suspend_scheduler(tcb);
@@ -150,6 +145,11 @@ void up_exit(int status)
 
   (void)nxtask_exit();
 
+#ifdef CONFIG_DUMP_ON_EXIT
+  sinfo("Other tasks:\n");
+  sched_foreach(_up_dumponexit, NULL);
+#endif
+
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/x86/src/common/up_exit.c

@@ -114,15 +114,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/x86_64/src/common/up_exit.c

@@ -114,15 +114,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", this_task());
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/xtensa/src/common/xtensa_exit.c

@@ -116,11 +116,6 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
-#ifdef CONFIG_DUMP_ON_EXIT
-  sinfo("Other tasks:\n");
-  nxsched_foreach(_xtensa_dumponexit, NULL);
-#endif
-
 #if XCHAL_CP_NUM > 0
   /* Disable co-processor support for the task that is exit-ing. */
 
@@ -131,6 +126,11 @@ void up_exit(int status)
 
   nxtask_exit();
 
+#ifdef CONFIG_DUMP_ON_EXIT
+  sinfo("Other tasks:\n");
+  nxsched_foreach(_xtensa_dumponexit, NULL);
+#endif
+
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/z16/src/common/z16_exit.c

@@ -114,15 +114,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_z16_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */

arch/z80/src/common/z80_exit.c

@@ -116,15 +116,15 @@ void up_exit(int status)
 
   sinfo("TCB=%p exiting\n", tcb);
 
+  /* Destroy the task at the head of the ready to run list. */
+
+  nxtask_exit();
+
 #ifdef CONFIG_DUMP_ON_EXIT
   sinfo("Other tasks:\n");
   nxsched_foreach(_up_dumponexit, NULL);
 #endif
 
-  /* Destroy the task at the head of the ready to run list. */
-
-  nxtask_exit();
-
   /* Now, perform the context switch to the new ready-to-run task at the
    * head of the list.
    */