Repository navigation
Increase ACE OpenSBI hart stack size to 16 KiB - #146
Merged
Merged
Conversation
Signed-off-by: whensun <[email protected]>
whensun
force-pushed
the
fix/ace-opensbi-hart-stack
branch
from
November 4, 2026 02:53
298ce2a to
7516bfe
Compare
Member
|
Thank you for this contribution! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of the changes
This PR increases the ACE OpenSBI per-hart stack region from 8 KiB to 16 KiB to prevent stack exhaustion during security monitor initialization.
Previously, the default 8 KiB region reserved 4 KiB for OpenSBI scratch data, leaving only 4 KiB of usable stack space. During initialization with RISC-V Vector support, the observed stack usage reached 5,120 bytes, exceeding the available stack space and coinciding with intermittent security monitor startup panics.
This PR introduces:
ACE_OPENSBI_HART_STACK_SIZEconstant set to 16 KiB.The change provides additional stack space for ACE security monitor initialization while keeping the modification limited to the ACE platform configuration.
Type of change
How to test this PR?
The stack exhaustion issue was observed during the development of Vector context-switch preservation in PR #145. The updated stack configuration was validated together with that implementation.