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

Changes MPU region alignment to 128 when FPU_SHARING and MPU_STACK_GUARD #83716

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rysiof
Copy link

@rysiof rysiof commented Jan 8, 2025

Changes ARM_MPU_REGION_MIN_ALIGN_AND_SIZE to 128 if ARM_MPU && FPU_SHARING && MPU_STACK_GUARD

Fixes: #83714

Signed-off-by: Maciej Kusio rysiof@gmail.com

@zephyrbot zephyrbot added size: XS A PR changing only a single line of code area: ARM ARM (32-bit) Architecture labels Jan 8, 2025
With our builds for Arm M7 we are selecting FPU_SHARING and
MPU_STACK_GUARD. For that setup, we have
ARM_MPU_REGION_MIN_ALIGN_AND_SIZE=64 and
MPU_STACK_GUARD_MIN_SIZE_FLOAT=128. So worst case scenario,
we are going to have stack that is 64-byte aligned and
128-byte guard at the bottom of the buffer.

Fixes: zephyrproject-rtos#83714

Signed-off-by: Maciej Kusio <rysiof@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ARM ARM (32-bit) Architecture size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ARM_MPU_REGION_MIN_ALIGN_AND_SIZE has invalid value when FPU_SHARING && MPU_STACK_GUARD
3 participants