Skip to content

Conversation

@gbaraldi
Copy link
Member

There is a question here, printing task backtraces (or backtraces on exit regardless) requires a ct, but only because it needs a buffer. Can we alloca that buffer (malloc is probably illegal given signal context)

@gbaraldi gbaraldi requested a review from vtjnash August 25, 2025 12:31
@vtjnash
Copy link
Member

vtjnash commented Aug 29, 2025

You can alloca one, but note that it must be small, as there isn't much stack space. A few thousand elements is probably fine though, and possibly even too many.

@gbaraldi
Copy link
Member Author

@vtjnash the one thing on that test is that race condition on the sleep. Any idea on how to make that properly reliable, without risking a zombie process?

Co-authored-by: Jameson Nash <vtjnash@gmail.com>
@gbaraldi gbaraldi added the merge me PR is reviewed. Merge when all tests are passing label Sep 9, 2025
Co-authored-by: Jameson Nash <vtjnash@gmail.com>
@DilumAluthge
Copy link
Member

Looks like this has picked up some merge conflicts :(

@DilumAluthge DilumAluthge removed the merge me PR is reviewed. Merge when all tests are passing label Sep 20, 2025
@vtjnash vtjnash added merge me PR is reviewed. Merge when all tests are passing backport 1.12 Change should be backported to release-1.12 labels Sep 24, 2025
@vtjnash vtjnash merged commit 0635285 into master Sep 24, 2025
9 checks passed
@vtjnash vtjnash deleted the gb/sigquittasks branch September 24, 2025 16:09
@DilumAluthge DilumAluthge removed the merge me PR is reviewed. Merge when all tests are passing label Sep 26, 2025
@KristofferC KristofferC mentioned this pull request Sep 30, 2025
47 tasks
xal-0 pushed a commit to xal-0/julia that referenced this pull request Sep 30, 2025
KristofferC pushed a commit that referenced this pull request Oct 10, 2025
KristofferC pushed a commit that referenced this pull request Oct 10, 2025
KristofferC pushed a commit that referenced this pull request Oct 12, 2025
KristofferC pushed a commit that referenced this pull request Oct 14, 2025
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Oct 21, 2025
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.

5 participants