xiaoxiang781216 commented on a change in pull request #5264:
URL: https://github.com/apache/incubator-nuttx/pull/5264#discussion_r787750272



##########
File path: sched/Kconfig
##########
@@ -282,11 +282,10 @@ config SMP
                CPU0 IRQ handler use thread0's stack, but thread0 may switch to 
CPU1, that
                will caused IRQ handler stack corruption.
 
-if SMP
-
 config SMP_NCPUS

Review comment:
       @gustavonihei after discuss in 
https://github.com/apache/incubator-nuttx/pull/5247, this change just ensure 
CONFIG_SMP_NCPUS equal 1 in no SMP case to avoid #ifdef/#else/#endif spread in 
the code base, but don't allow the user to select value > 1. So I think we can 
keep it as CONFIG_SMP_NCPUS.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to