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

thread_pthread: Grow main_stack if required (fixes tests on HPPA) #7676

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thesamesam
Copy link
Contributor

On HPPA, test_insns.rb fails (along with various Ruby gems) with 'stack level too deep (SystemStackError)'. This turns out to be because HPPA defaults to a small(er) stack.

With this change, Ruby's test suite passes on HPPA.

Thanks to both Dave and Helge for the investigation and coming up with the patch.

Bug: https://bugs.gentoo.org/701494
Bug: https://bugs.debian.org/881773
Bug: https://bugs.debian.org/881772 (for PPC64)
Bug: rack/rack#1640
Thanks-to: John David Anglin dave.anglin@bell.net
Thanks-to: Helge Deller deller@gmx.de

@thesamesam
Copy link
Contributor Author

cc @glaubitz @danglin44 @hdeller

@thesamesam thesamesam marked this pull request as draft April 7, 2023 15:30
@thesamesam thesamesam force-pushed the hppa-pthreads branch 3 times, most recently from 3ebe81f to 6af7947 Compare April 7, 2023 16:30
On HPPA, test_insns.rb fails (along with various Ruby gems) with
'stack level too deep (SystemStackError)'. This turns out to be because HPPA
defaults to a small(er) stack.

With this change, most of Ruby's test suite now passes on HPPA.

Thanks to both Dave and Helge for the investigation and coming up with the
patch.

Bug: https://bugs.gentoo.org/701494
Bug: https://bugs.debian.org/881773
Bug: https://bugs.debian.org/881772 (for PPC64)
Bug: rack/rack#1640
Thanks-to: John David Anglin <dave.anglin@bell.net>
Thanks-to: Helge Deller <deller@gmx.de>
Signed-off-by: Sam James <sam@gentoo.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant