sched/sched: CONFIG_SCHED_RESUMESCHEDULER macro define error

we removed "select SCHED_RESUMESCHEDULER", As SCHED_RESUMESCHEDULER is not a necessary feature in SMP, turning it on by default may affect performance. Signed-off-by: hujun5 <hujun5@xiaomi.com>
hujun260 · May 7, 2024 · 0727706 · 0727706
1 parent 72de3fe
commit 0727706
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/include/nuttx/sched.h b/include/nuttx/sched.h
@@ -1210,7 +1210,7 @@ int group_exitinfo(pid_t pid, FAR struct binary_s *bininfo);
  *
  ****************************************************************************/
 
-#if CONFIG_RR_INTERVAL > 0 || defined(CONFIG_SCHED_RESUMESCHEDULER)
+#if defined(CONFIG_SCHED_RESUMESCHEDULER)
 void nxsched_resume_scheduler(FAR struct tcb_s *tcb);
 #else
 #  define nxsched_resume_scheduler(tcb)

diff --git a/sched/Kconfig b/sched/Kconfig
@@ -351,7 +351,6 @@ config SMP
 	depends on ARCH_HAVE_TESTSET
 	depends on ARCH_INTERRUPTSTACK != 0
 	select SPINLOCK
-	select SCHED_RESUMESCHEDULER
 	select IRQCOUNT
 	---help---
 		Enables support for Symmetric Multi-Processing (SMP) on a multi-CPU

diff --git a/sched/sched/sched_resumescheduler.c b/sched/sched/sched_resumescheduler.c
@@ -33,7 +33,7 @@
 #include "irq/irq.h"
 #include "sched/sched.h"
 
-#if CONFIG_RR_INTERVAL > 0 || defined(CONFIG_SCHED_RESUMESCHEDULER)
+#if defined(CONFIG_SCHED_RESUMESCHEDULER)
 
 /****************************************************************************
  * Public Functions