anchao commented on PR #17357: URL: https://github.com/apache/nuttx/pull/17357#issuecomment-3558205601
> > 1. Please refer to [sched/signal: Add support to disable signals #17352](https://github.com/apache/nuttx/pull/17352) as the primary commit; @wangchdo submitted this change first. > > 2. cc NuttX PMC and commiterrs, Please do not merge this PR, especially @xiaoxiang781216 > > The previous submission at #17352 had an overly broad scope when disabling signals, as it also disabled functionalities like slee, kill etc. The current patchset narrows this scope, ensuring that APIs such as sleep and kill remain available while only certain signal functionalities are disabled. > > Alternatively, we could introduce a concept like "disable levels" to differentiate the extent of signal functionality being disabled. For sleep()/usleep(), compatibility with the CONFIG_DISABLE_SIGNALS scenario should be considered in the implementation. For other APIs that have mandatory dependencies on signals (such as kill()/pkill()), scenario-based optimization can be performed—even the interfaces can be retained with error code returns supported. -- 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]
