diff options
| author | Rich Felker <dalias@aerifal.cx> | 2022-08-20 12:24:49 -0400 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2022-08-20 12:24:49 -0400 |
| commit | 2e5fff43dd7fc808197744c67cca7908ac19bb4f (patch) | |
| tree | e62bcaa00019889aa5fa574a71336b364934c7f7 /src/thread/pthread_cancel.c | |
| parent | 379b18218d18a049b5423bbb9bb22e066ffc3f78 (diff) | |
| download | musl-2e5fff43dd7fc808197744c67cca7908ac19bb4f.tar.gz | |
use alt signal stack when present for implementation-internal signals
a request for this behavior has been open for a long time. the
motivation is that application code, particularly under some language
runtimes designed around very-low-footprint coroutine type constructs,
may be operating with extremely small stack sizes unsuitable for
receiving signals, using a separate signal stack for any signals it
might handle.
progress on this was blocked at one point trying to determine whether
the implementation is actually entitled to clobber the alt stack, but
the phrasing "available to the implementation" in the POSIX spec for
sigaltstack seems to make it clear that the application cannot rely on
the contents of this memory to be preserved in the absence of signal
delivery (on the abstract machine, excluding implementation-internal
signals) and that we can therefore use it for delivery of signals that
"don't exist" on the abstract machine.
no change is made for SIGTIMER since it is always blocked when used,
and accepted via sigwaitinfo rather than execution of the signal
handler.
Diffstat (limited to 'src/thread/pthread_cancel.c')
| -rw-r--r-- | src/thread/pthread_cancel.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/thread/pthread_cancel.c b/src/thread/pthread_cancel.c index 2f9d5e97..2d3a98ea 100644 --- a/src/thread/pthread_cancel.c +++ b/src/thread/pthread_cancel.c @@ -77,7 +77,7 @@ void __testcancel() static void init_cancellation() { struct sigaction sa = { - .sa_flags = SA_SIGINFO | SA_RESTART, + .sa_flags = SA_SIGINFO | SA_RESTART | SA_ONSTACK, .sa_sigaction = cancel_handler }; memset(&sa.sa_mask, -1, _NSIG/8); |