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.

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

b/402103828: Increase xstream stack size #16085

Merged
merged 1 commit into from
Mar 14, 2025
Merged

Conversation

mjmac
Copy link
Contributor

@mjmac mjmac commented Mar 12, 2025

The stack size set for the server xstreams and
the NVMe polling ULT is 64K. This size seems to
result in occasional segfaults on engine startup,
likely due to being near a limit. Doubling the
stack size for these ULTs should avoid the problem.

Change-Id: Id339cc48985825ccce291ad288e1490920facf5c
Signed-off-by: Michael MacDonald mjmac@google.com

Copy link

Errors are component not formatted correctly,Ticket number prefix incorrect,PR title is malformatted. See https://daosio.atlassian.net/wiki/spaces/DC/pages/11133911069/Commit+Comments,Unable to load ticket data
https://daosio.atlassian.net/browse/b/402103828:

@mjmac mjmac force-pushed the dev/mjmac/b402103828 branch from e9100f7 to 89fdc71 Compare March 12, 2025 18:57
@mjmac mjmac requested review from wangdi1 and jolivier23 March 13, 2025 13:50
The stack size set for the server xstreams and
the NVMe polling ULT is 64K. This size seems to
result in occasional segfaults on engine startup,
likely due to being near a limit. Doubling the
stack size for these ULTs should avoid the problem.

Change-Id: Id339cc48985825ccce291ad288e1490920facf5c
Signed-off-by: Michael MacDonald <mjmac@google.com>
@mjmac mjmac force-pushed the dev/mjmac/b402103828 branch from 89fdc71 to 2d28e88 Compare March 13, 2025 16:22
@mjmac mjmac changed the title b/402103828: Increase stack size for NVMe polling ULT b/402103828: Increase xstream stack size Mar 13, 2025
@mjmac mjmac merged commit b3612b5 into google/2.6 Mar 14, 2025
54 checks passed
@mjmac mjmac deleted the dev/mjmac/b402103828 branch March 14, 2025 14:22
@mjmac mjmac mentioned this pull request Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants