Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reevaluate signal handling in POSIX #348

Open
jphickey opened this issue Jan 13, 2020 · 1 comment
Open

Reevaluate signal handling in POSIX #348

jphickey opened this issue Jan 13, 2020 · 1 comment

Comments

@jphickey
Copy link
Contributor

jphickey commented Jan 13, 2020

Is your feature request related to a problem? Please describe.
There may be some inefficiencies / room for improvement in how POSIX is handling the signal masks, that should be evaluated and considered.

For the RT signals, these are masked all the time and timers use sigwait() to synchronously receive them. However that potential to leave stray signals queued. This has been worked around by ensuing that no signals are queued before setting up a timer, but there may be a more efficient way to accomplish this.

Describe the solution you'd like
Consider setting all the unused signals to "IGNORE" such that the kernel will not even deliver them to the process, rather than masking them at the process.

Signal masks are also changed as part of the global lock/unlock procedure what may not be necessary. This should be re-evaluated with the current OSAL design. Historically this may have been necessary for async signals but the current OSAL now avoids async signals for all OSAL-created tasks. Already done in #673/#678

Describe alternatives you've considered
Leave as-is.

Additional context
See related issue #335

Requester Info
Joseph Hickey, Vantage Systems, Inc.

@skliper
Copy link
Contributor

skliper commented Dec 9, 2020

See also #673.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants