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

[release/6.0] [mono/win] Increase stack size (reserve) to 8MB #60561

Merged
merged 1 commit into from
Nov 9, 2021

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 18, 2021

Backport of #60265 to release/6.0

/cc @lewing @radekdoulik

Increase the AOT compilers reserve stack size to 8MB to avoid an AOT compilation failure due to insufficient stack space.

Customer Impact

Resolves a potential crash in AOT compilation.

Testing

Manual and Automatic

Risk

Very Low

Fix #57141 in windows build.

Set the stack size (reserve) to 8MB, which is usually a default
on linux. This way we should get similar behavior on windows.
@Anipik
Copy link
Contributor

Anipik commented Oct 21, 2021

cc @marek-safar

@lewing lewing added this to the 6.0.1 milestone Oct 21, 2021
@marek-safar marek-safar added the Servicing-consider Issue for next servicing release review label Oct 22, 2021
@jacob-l
Copy link

jacob-l commented Oct 30, 2021

I tested this PR, and it solves the issue; we can compile projects successfully. Thanks!

@leecow leecow added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Nov 2, 2021
@Anipik Anipik merged commit f9c4b56 into release/6.0 Nov 9, 2021
@jkotas jkotas deleted the backport/pr-60265-to-release/6.0 branch November 13, 2021 22:59
@ghost ghost locked as resolved and limited conversation to collaborators Dec 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Codegen-AOT-mono Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants