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.

Sign up for GitHub

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

Jump to bottom

Fix paltest_pal_sxs_test1 on illumos #105207

Merged
merged 2 commits into from
Aug 21, 2024
Merged

Fix paltest_pal_sxs_test1 on illumos #105207

merged 2 commits into from
Aug 21, 2024

Commits on Aug 19, 2024

  1. Fix RPATH for exception handling PAL test

    @AustinWise
    AustinWise committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    6ed5db5 View commit details
    Browse the repository at this point in the history

  2. Don't use alternate stack on illumos or Solaris. 

    When .NET translates SIGSEV to NullReferenceException, it does not return
from the signal handler. Instead it resumes execution at the catch handler
for the exception. This is not recommend by the manpage for sigaction(2):

> It is not recommended that [the ucontext] arg be used by the handler to
> restore the context from before the signal delivery.

The practical effect of resuming execution without returning from a handler
is that the alternate stack will not be used for subsequent signal delivery.
This is in contrast to the behavior on linux, which will always use the
alternate stack if the stack pointer at the time of fault does not fall on
the alternate stack.

Since the alternate stack is only usable for a single exception, don't
bother using it for any exceptions.
    @AustinWise
    AustinWise committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    85b5c51 View commit details
    Browse the repository at this point in the history