fix(mstp): conform master token and PFM transitions to Clause 9.5.6 - #2
fix(mstp): conform master token and PFM transitions to Clause 9.5.6#2bbartling wants to merge 1 commit into
Conversation
DONE_WITH_TOKEN no longer reverse-scans from NS+1 or emits Token TS→TS. Add done_with_token(), FindNewSuccessor via PFM after Nretry_token, and regression tests A–E for multi-master coexistence. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Consolidated on branch |
Summary
Clause 9.5.6 DONE_WITH_TOKEN / PASS_TOKEN / PFM fix stacked on
fix/mstp-clause9-crc(jscott3201#464 upstream).Root cause of Workbench/FEC drop when Rust joins as MAC 3: maintenance PFM began at
NS+1(BASRT 0) instead of scanning(TS, NS).Tests
cargo test -p bacnet-transport --lib— regressions A–E inclause956_tests.rs.Made with Cursor