xiaoxiang781216 commented on code in PR #17352:
URL: https://github.com/apache/nuttx/pull/17352#discussion_r2558515786
##########
sched/Kconfig:
##########
@@ -44,6 +44,16 @@ config DISABLE_ENVIRON
bool "Disable environment variable support"
default DEFAULT_SMALL
+config DISABLE_SIGNALS
+ bool "Disable signal support"
+ default n
Review Comment:
> Done. However, I think disabling signals is not only for reducing
footprint, but also for improving performance and safety.
yes, that's why you and we develop the patch to make the signal disable.
> When signals are enabled, a task’s stack may be reused to run a signal
handler, which is not entirely safe.
the partial signal disable remove this path like the full disable, so this
problem doesn't exist anymore with both approach.
> Moreover, various signal-related operations must be performed during task
initialization, creation, context switching, and exit, all of which introduce
additional performance overhead.
yes, that's why the full signal disable is useful, if the userspace
application doesn't use any signal related functionality.
--
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]
