Skip to content

arch/arm64: Delete the default save for SCTLR and fix the issue of including incorrect order#18279

Merged
xiaoxiang781216 merged 2 commits intoapache:masterfrom
W-M-R:arm64_sctlr
Feb 2, 2026
Merged

arch/arm64: Delete the default save for SCTLR and fix the issue of including incorrect order#18279
xiaoxiang781216 merged 2 commits intoapache:masterfrom
W-M-R:arm64_sctlr

Conversation

@W-M-R
Copy link
Copy Markdown
Contributor

@W-M-R W-M-R commented Jan 30, 2026

Currently, the SCTLR register is only used to switch the thread MTE state and has no other uses. Because saving this register is special, it will take a long time after testing, so the default saving behavior is deleted.

Note: Please adhere to Contributing Guidelines.

Summary

  1. Currently, the SCTLR register is only used to switch the thread MTE state and has no other uses. Because saving this register is special, it will take a long time after testing, so the default saving behavior is deleted.
  2. arm64/irq.h: Fix the issue of including incorrect order:
    <nuttx/irq.h> will use the macros defined in <arch/chip/irq.h>, so the include order should be after it

Impact

none

Testing

citest

Currently, the SCTLR register is only used to switch the thread MTE state and has no other uses. Because saving this register is special, it will take a long time after testing, so the default saving behavior is deleted.

Signed-off-by: wangmingrong1 <wangmingrong1@xiaomi.com>
<nuttx/irq.h> will use the macros defined in <arch/chip/irq.h>, so the include order should be after it

Signed-off-by: wangmingrong1 <wangmingrong1@xiaomi.com>
@github-actions github-actions Bot added Arch: arm64 Issues related to ARM64 (64-bit) architecture Size: S The size of the change in this PR is small labels Jan 30, 2026
@W-M-R W-M-R changed the title arch/arm64: Delete the default save for SCTLR arch/arm64: Delete the default save for SCTLR and fix the issue of including incorrect order Jan 30, 2026
@xiaoxiang781216 xiaoxiang781216 merged commit 95ef675 into apache:master Feb 2, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Arch: arm64 Issues related to ARM64 (64-bit) architecture Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants