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

[Backport v1.14-branch] arch: arm: move stack switching back to ASM code #16214

Merged
merged 2 commits into from
May 21, 2019

Conversation

backporting[bot]
Copy link

@backporting backporting bot commented May 16, 2019

Backport #16099.

Fixes #15794.

wentongwu added 2 commits May 16, 2019 15:40
With -O0 optimizion, gcc compiler doesn't inline "static inline"
marked function. So when function call return from function
set_and_switch_to_psp which is to switch sp from MSP to PSP, the
ending "mov sp, r7" instruction will overwrite the just updated
sp value(PSP) with the beginning stack pointer(should be MSP)
stored in r7 register, so the switch doesn't happen. And it causes
unpredictable problems in the initialization process, the backward
analysis for this problem can be found on Github issue #15794.

Fixes: #15794.

Signed-off-by: Wentong Wu <wentong.wu@intel.com>
increase stack buffer when code coverage enabled.

Fixes: #15794.

Signed-off-by: Wentong Wu <wentong.wu@intel.com>
@ioannisg ioannisg added the Backport Backport PR and backport failure issues label May 17, 2019
@nashif nashif merged commit 7526dab into v1.14-branch May 21, 2019
@nashif nashif deleted the backport-16099-to-v1.14-branch branch May 21, 2019 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backport Backport PR and backport failure issues
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants