Skip to content

Conversation

@aggarg
Copy link
Member

@aggarg aggarg commented Jan 23, 2025

Description

Ensure the saved context location falls within the reserved context area rather than overlapping with the next MPU_SETTINGS structure member.

This never caused a problem because actual read/write operations start from one word before the saved context location.

Thanks to @dajiang0055 for reporting this.

Test Steps

Verified on Cortex-M33 hardware.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • [NA] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

NA.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Ensure the saved context location falls within the reserved context area
rather than overlapping with the next MPU_SETTINGS structure member.

This never caused a problem because actual read/write operations
start from one word before the saved context location.

Signed-off-by: Gaurav Aggarwal <[email protected]>
@aggarg aggarg requested a review from a team as a code owner January 23, 2025 15:11
Signed-off-by: Gaurav Aggarwal <[email protected]>
@sonarqubecloud
Copy link

@aggarg aggarg merged commit 11d0caa into FreeRTOS:main Jan 25, 2025
17 checks passed
@aggarg aggarg deleted the mpu_context_size branch January 25, 2025 08:04
aggarg added a commit to aggarg/FreeRTOS-Kernel-Community-Supported-Ports that referenced this pull request Aug 8, 2025
Ensure the saved context location falls within the reserved context area
rather than overlapping with the next MPU_SETTINGS structure member.

This was applied to the upstream port in the following PR: FreeRTOS/FreeRTOS-Kernel#1230.

Signed-off-by: Gaurav Aggarwal <[email protected]>
moninom1 pushed a commit to FreeRTOS/FreeRTOS-Kernel-Community-Supported-Ports that referenced this pull request Aug 8, 2025
Ensure the saved context location falls within the reserved context area
rather than overlapping with the next MPU_SETTINGS structure member.

This was applied to the upstream port in the following PR: FreeRTOS/FreeRTOS-Kernel#1230.

Signed-off-by: Gaurav Aggarwal <[email protected]>
moninom1 pushed a commit to moninom1/FreeRTOS-Kernel that referenced this pull request Sep 30, 2025
Add Cygwin option for compilation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants