Skip to content

Pin M33MU to upstream master with deterministic USART timing - #55

Merged
mattia-moffa merged 1 commit into
wolfSSL:mainfrom
aidangarske:m33mu-pin-uart-determinism
Oct 1, 2026
Merged

mattia-moffa merged 1 commit into
wolfSSL:mainfrom
aidangarske:m33mu-pin-uart-determinism

Conversation

@aidangarske

Copy link
Copy Markdown
Member
  • Bump M33MU_REF to f3c03675 (danielinux/m33mu master with # 21) in the M33MU workflow, the STM32H563 runner and the MIMXRT700 runner.
  • m33mu # 21 raises the STM32 USART interrupt at the register write instead of the next host poll, and makes --uart-stdout lossless and ordered under a slow host. This removes the per-run interrupt-timing drift behind the occasional confboot test 66 failure and the lost console lines on CI.
  • Should close M33MU runs occasionally lose console output or shift interrupt timing on GitHub runners #34

@aidangarske aidangarske self-assigned this Oct 1, 2026
@aidangarske
aidangarske requested review from mattia-moffa and a balanced review from Copilot October 1, 2026 16:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Both runners can silently reuse cached binaries built from an older M33MU revision.

Review effort: Balanced
Findings: 2 Medium severity

Open (2)
What changed in this PR

Pins M33MU to the upstream revision containing deterministic USART timing and lossless console output, while removing the now-upstreamed RT700 patch.

Changes:

  • Updates M33MU references across CI and target runners.
  • Removes the RT700 emulator patch.
  • Updates testing documentation for unpatched upstream M33MU.
File Description
.github/​workflows/​m33mu.yml Pins CI to the new revision.
tests/​target/​run_m33mu_scenario.sh Updates the STM32H563 runner pin.
tests/​target/​run_rt700_m33mu.sh Updates the RT700 pin and removes patch application.
tests/​target/​m33mu-imxrt700.patch Deletes the upstreamed patch.
docs/​Testing.md Documents using unpatched upstream M33MU.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread tests/target/run_m33mu_scenario.sh
Comment thread tests/target/run_rt700_m33mu.sh
@mattia-moffa
mattia-moffa merged commit ddb1f64 into wolfSSL:main Oct 1, 2026
104 checks passed
@aidangarske
aidangarske deleted the m33mu-pin-uart-determinism branch October 1, 2026 16:28
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.

M33MU runs occasionally lose console output or shift interrupt timing on GitHub runners

3 participants