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

arm: fix enable interrupt too earlier caused system crash #4013

Merged
merged 1 commit into from
Jul 2, 2021

Conversation

GUIDINGLI
Copy link
Contributor

Summary

arm: fix enable interrupt too earlier caused system crash

reason:
msr cpsr, r2 /* Set the CPSR */

// interrupt hanppend, context switch

pop	{r0-r2, pc}

resolve:
use SPSR instead, and recover with ldmia ^

Impact

Testing

reason:
	msr	cpsr, r2   /* Set the CPSR */

    // interrupt hanppend, context switch

	pop	{r0-r2, pc}

resolve:
    use SPSR instead, and recover with ldmia ^

Change-Id: Id7cee6452997ec19919eeecf6e7616164b3a0ab3
Signed-off-by: ligd <liguiding1@xiaomi.com>
(cherry picked from commit 6fa6c1676932d7babb7ff22ef556a17bb18f1c0d)
Signed-off-by: ligd <liguiding1@xiaomi.com>
Copy link
Contributor

@davids5 davids5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@davids5 davids5 requested a review from btashton July 1, 2021 19:52
@xiaoxiang781216 xiaoxiang781216 merged commit 50eee2f into apache:master Jul 2, 2021
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