arch/arm/armv7-a: move up_set_cpu_state call to arm_start_handler

Previously, up_set_cpu_state(CPU_RUNNING) was called in up_cpu_on()
before sending SGI1. This caused the CPU state to be updated even
when the target CPU failed to receive SGI1 and did not actually start.

Move up_set_cpu_state() to arm_start_handler(), which is the SGI1
handler executed on the target CPU, ensuring the state is only
updated after the CPU has successfully started.

Signed-off-by: Eunwoo Nam <eunwoo.nam@samsung.com>

Author: ewoodev

os/arch/arm/src/armv7-a/arm_cpuon.c

@@ -111,6 +111,10 @@ int arm_start_handler(int irq, void *context, void *arg)
 
 	svdbg("CPU%d Started\n", this_cpu());
 
+#ifdef CONFIG_CPU_HOTPLUG
+	up_set_cpu_state(this_cpu(), CPU_RUNNING);
+#endif
+
 #ifdef CONFIG_SCHED_INSTRUMENTATION
 	/* Notify that this CPU has started */
 
@@ -190,10 +194,6 @@ int up_cpu_on(int cpu)
 	sched_note_cpu_start(this_task(), cpu);
 #endif
 
-#ifdef CONFIG_CPU_HOTPLUG
-	up_set_cpu_state(cpu, CPU_RUNNING);
-#endif
-
 	/* Execute SGI1 */
 	ret = arm_cpu_sgi(GIC_IRQ_SGI1, (1 << cpu));
 	if (ret < 0) {