Skip to content

Fix Xtensa ESP32 FreeRTOS build break from interrupt-priority macro mismatch#68

Draft
JohnAmadis with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-building-issue-freertos-again
Draft

Fix Xtensa ESP32 FreeRTOS build break from interrupt-priority macro mismatch#68
JohnAmadis with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-building-issue-freertos-again

Conversation

Copilot AI commented May 15, 2026

Copy link
Copy Markdown
Contributor

ESP32/Xtensa builds failed because configMAX_SYSCALL_INTERRUPT_PRIORITY was set to the generic fallback (0) in FreeRTOSConfig.h and later redefined by the Xtensa port, causing an inconsistent configuration path that surfaced as assembler errors in xtensa_context.S.

  • Root cause alignment (Xtensa arch config)

    • Updated config/arch/xtensa_esp32/FreeRTOSConfigArch.h to define Xtensa-required interrupt priority macros before the generic defaults apply.
  • Macro definitions added

    • Added guarded definitions for:
      • configMAX_SYSCALL_INTERRUPT_PRIORITYXCHAL_EXCM_LEVEL
      • configMAX_API_CALL_INTERRUPT_PRIORITYconfigMAX_SYSCALL_INTERRUPT_PRIORITY
  • Impact

    • Keeps Xtensa interrupt-priority configuration consistent across FreeRTOSConfig.h and the Xtensa port headers, eliminating the conflicting macro path that led to the build failure.
#ifndef configMAX_SYSCALL_INTERRUPT_PRIORITY
    #define configMAX_SYSCALL_INTERRUPT_PRIORITY    XCHAL_EXCM_LEVEL
#endif

#ifndef configMAX_API_CALL_INTERRUPT_PRIORITY
    #define configMAX_API_CALL_INTERRUPT_PRIORITY    configMAX_SYSCALL_INTERRUPT_PRIORITY
#endif

Copilot AI linked an issue May 15, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix building issue for FreeRTOS on ESP Fix Xtensa ESP32 FreeRTOS build break from interrupt-priority macro mismatch May 15, 2026
Copilot AI requested a review from JohnAmadis May 15, 2026 08:22
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.

Fix building issue

2 participants