Skip to content

Commit e5e72bd

Browse files
committed
Fix switch_to(STANDBY) example and bfdsyncd socket type
- Event 2 in Section 4.3.6: NH-B is still in m_valid_members when switch_to(STANDBY) is called, so Step 2 does remove NH-B - Section 1.2: bfdsyncd uses TCP socket (PF_INET), not Unix socket Signed-off-by: yuezhou.jk <yuezhou.jk@alibaba-inc.com>
1 parent 2ce7e93 commit e5e72bd

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

doc/bfd/BFD_SAI_Triggered_NHG_Fast_Switchover_HLD.md

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ This path involves multiple Redis operations, container boundary crossings (BGP
121121
| BFD detection | 3 × interval (e.g., 300ms @ 100ms interval) | Configurable, not the focus here |
122122
| SAI notification → BFDOrch | < 1ms | Fast (in-process callback) |
123123
| BFDOrch → STATE_DB → bfdsyncd | 1–5ms | Redis publish + subscribe |
124-
| bfdsyncd → bfdd (Unix socket) | < 1ms | Fast |
124+
| bfdsyncd → bfdd (TCP socket) | < 1ms | Fast |
125125
| bfdd → pathd (ZAPI) | < 1ms | Fast |
126126
| pathd re-evaluation + route update | 1–10ms | Path selection algorithm |
127127
| zebra → fpmsyncd → APPL_DB | 1–5ms | Netlink + Redis |
@@ -376,10 +376,9 @@ Event 1: BFD-100 DOWN
376376
→ ECMP forwarding: {NH-B}
377377
378378
Event 2: BFD-200 DOWN
379-
→ ASIC removes NH-B from ECMP
380-
→ All PRIMARY removed → switch_to(STANDBY)
379+
→ NH-B is last active PRIMARY → switch_to(STANDBY)
381380
Step 1 (Make): Add NH-C (BFD-300=UP → allowed)
382-
Step 2 (Break): No non-STANDBY active members to remove
381+
Step 2 (Break): Remove NH-B (PRIMARY) from ECMP
383382
→ NHG = { NH-A(inactive), NH-B(inactive), NH-C(active) }
384383
→ ECMP forwarding: {NH-C}
385384

0 commit comments

Comments
 (0)