patacongo commented on PR #11226: URL: https://github.com/apache/nuttx/pull/11226#issuecomment-1819814269
I am a little confused. sched_removereadytorun() is part of the level implementation of a context switch. High level checks such as if the scheduler is locked should have already been performed at a much higher level before this function is ever called. My preference would be to keep the layering of higher level checks and lower level manipulations separate. There is already a check if the scheduler is locked. Does it now work? Is there some issue that makes that higher level check not work for SMP? Perhaps what we really need to do is enhance the higher level checks for locking in the SMP case? -- 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]
