Skip to content

Revert "Async signal/bail safety"#982

Merged
pkova merged 1 commit intodevelopfrom
revert-940-dozreg/async-signal-bail-safety
Mar 23, 2026
Merged

Revert "Async signal/bail safety"#982
pkova merged 1 commit intodevelopfrom
revert-940-dozreg/async-signal-bail-safety

Conversation

@pkova
Copy link
Collaborator

@pkova pkova commented Mar 23, 2026

Reverts #940

@pkova pkova requested a review from a team as a code owner March 23, 2026 15:27
@dozreg-toplud
Copy link
Contributor

#940 was reverted because the cold state accumulated in a child road cannot be simply discarded as the traps accumulated in the stacktrace depend on it. In practice it means having to walk the stacktrace, removing the traps, removing crash information. Not good!

A better approach would be to actually make our hashtables async-signal (i.e. they should be traversable at all times).

@pkova pkova merged commit 43ad541 into develop Mar 23, 2026
2 checks passed
@pkova pkova deleted the revert-940-dozreg/async-signal-bail-safety branch March 23, 2026 17:58
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.

2 participants