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.

Sign up for GitHub

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

Jump to bottom

freertos: Apply upstream stack masking fix for xtensa/port.c (IDFGH-5549) #7274

Closed
wants to merge 1 commit into from
Closed

freertos: Apply upstream stack masking fix for xtensa/port.c (IDFGH-5549) #7274

wants to merge 1 commit into from

Conversation

AxelLin
Copy link
Contributor

@AxelLin AxelLin commented Jul 15, 2021

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jul 15, 2021
@github-actions github-actions bot changed the title freertos: Apply upstream stack masking fix for xtensa/port.c freertos: Apply upstream stack masking fix for xtensa/port.c (IDFGH-5549) Jul 15, 2021
@igrr
Copy link
Member

igrr commented Jul 19, 2021

Thanks for the PR @AxelLin. It seems this issue doesn't affect our chips since we use smaller values of portBYTE_ALIGNMENT than the reporter of the original issue.
The fix makes sense, though, and all our internal tests pass with it, we'll merge it soon.

@espressif-bot espressif-bot added Status: In Progress Work is in progress Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Status: In Progress Work is in progress labels Jul 19, 2021
espressif-bot pushed a commit that referenced this pull request Jul 25, 2021
@AxelLin @igrr
freertos: Apply upstream stack masking fix for xtensa/port.c
6c2aa59 
Link: FreeRTOS/FreeRTOS-Kernel@6a57845#diff-cfa9a8b71a9665b5610f59bd2f56cb81b3ee73beaa6cac3fc965884069588d47
Signed-off-by: Axel Lin <[email protected]>

Merges #7274
@AxelLin AxelLin closed this Jul 25, 2021
@AxelLin AxelLin deleted the freertos-upstream-stack-fix branch July 25, 2021 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AxelLin @igrr @espressif-bot