Skip to content

Page guard fixes#2643

Open
panos-lunarg wants to merge 3 commits intoLunarG:devfrom
panos-lunarg:page_guard_fixes
Open

Page guard fixes#2643
panos-lunarg wants to merge 3 commits intoLunarG:devfrom
panos-lunarg:page_guard_fixes

Conversation

@panos-lunarg
Copy link
Contributor

Two commits in this PR:

  • The installation of the signal handler watcher is moved so that it is installed only when Android's libsigchain is not used
  • Give option to disable libsigchain

The watcher should not be installed when the handler is set with the new
AddSpecialSignalHandlerFn but only when sigaction is used
Android's libsigaction gives the option to install signal handler with
elevated priority compared the ones installed with sigaction. There are
cases though which this does not work well.
@panos-lunarg panos-lunarg requested a review from a team as a code owner January 30, 2026 14:09
@ci-tester-lunarg
Copy link
Collaborator

CI gfxreconstruct build queued with queue ID 636273.

@ci-tester-lunarg
Copy link
Collaborator

CI gfxreconstruct build # 8753 running.

@ci-tester-lunarg
Copy link
Collaborator

CI gfxreconstruct build # 8753 passed.

Copy link
Contributor

@mikes-lunarg mikes-lunarg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I'm curious what kinds of issues using libsigchain caused. Prior to this I assumed it worked the same as sigaction, just with a higher priority.

@ci-tester-lunarg
Copy link
Collaborator

CI gfxreconstruct build queued with queue ID 637883.

@ci-tester-lunarg
Copy link
Collaborator

CI gfxreconstruct build # 8763 running.

@ci-tester-lunarg
Copy link
Collaborator

CI gfxreconstruct build # 8763 passed.

@panos-lunarg
Copy link
Contributor Author

LGTM. I'm curious what kinds of issues using libsigchain caused. Prior to this I assumed it worked the same as sigaction, just with a higher priority.

Tbh I am not sure. There's an app that simply terminates without anything enlightening in logcat. Switching to sigaction just makes it work. It has to be related to the Android version as on a different phone with Android 15 (vs 14 ) it just works without this PR

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants