Skip to content

Overlay ECMP with BFD support#861

Merged
prsunny merged 6 commits intomasterfrom
prsunny-overlayecmp
Dec 6, 2021
Merged

Overlay ECMP with BFD support#861
prsunny merged 6 commits intomasterfrom
prsunny-overlayecmp

Conversation

Vxlan Overlay ECMP
BFD orchagent and HW support
- Better performance in re-programming routes in ASIC instead of separate process to monitor and modify each route prefix by updating DB entries

### BfdOrch
Sonic may offload the BFD session handling to hardware that has BFD capabilities. A new module, BfdOrch shall be introduced to handle BFD session to monitoring endpoints and check the health of remote endpoints. BfdOrch shall offload the session initiation/sustenance to hardware via SAI APIs and gets the notifications of session state from SAI. The session state shall be updated in STATE_DB and to any other observer orchestration agents.
Copy link
Contributor

Choose a reason for hiding this comment

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

The path from SYNCD to ORCH is quite slow, in such cases the BFD state notification from SYNCD to BFDOrch may add a delta to the desired BFD timeout. The actual BFD timeout achieved may be significantly higher than desired timeout.

@prsunny prsunny changed the title Overlay ECMP with BFD support HLD Overlay ECMP with BFD support Sep 14, 2021
@prsunny prsunny requested a review from shi-su October 9, 2021 00:54
@zhangyanzhao
Copy link
Collaborator

All the comments have been addressed and need reviewer to approve

shi-su
shi-su previously approved these changes Oct 19, 2021
@zhangyanzhao
Copy link
Collaborator

@prsunny can you please add the code PRs by following example #806 ? Thanks.

@prsunny prsunny merged commit 0a01667 into master Dec 6, 2021
@prsunny prsunny deleted the prsunny-overlayecmp branch December 6, 2021 18:38
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.

9 participants