Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add details of sff_mgr regarding deterministic bringup for SFF compliant modules #1371

Merged
merged 13 commits into from
Dec 6, 2023

Conversation

longhuan-cisco
Copy link
Contributor

@longhuan-cisco longhuan-cisco commented Jun 26, 2023

According to Interface-Link-bring-up-sequence.md, add a new thread sff_mgr under xcvrd to provide deterministic link bringup feature for SFF compliant modules (100G/40G). Update HLD as such.

Implementation PRs:
sonic-net/sonic-platform-daemons#383
sonic-net/sonic-platform-daemons#430
sonic-net/sonic-buildimage#18054

Note
Please click on "view file" (from the ... button on the right-top corner of the diff file) to preview the mermaid flow chart.

@longhuan-cisco
Copy link
Contributor Author

@prgeor @mihirpat1 could you please take a look this HLD update?

CC @shyam77git @jaganbal-a

@prgeor prgeor requested review from keboliu and mihirpat1 July 14, 2023 19:26
@prgeor
Copy link
Contributor

prgeor commented Jul 14, 2023

@keboliu @mihirpat1 please review

prgeor
prgeor previously approved these changes Nov 27, 2023
@prgeor
Copy link
Contributor

prgeor commented Nov 27, 2023

@mihirpat1 please review

@prgeor prgeor merged commit 6e92bbe into sonic-net:master Dec 6, 2023
1 check passed
rlhui pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Mar 12, 2024
This PR is a dependency of sonic-net/sonic-platform-daemons#383
HLD of sff_mgr: sonic-net/SONiC#1371

Why I did it
Add enable_xcvrd_sff_mgr flag support for sff_mgr
saksarav-nokia pushed a commit to saksarav-nokia/sonic-buildimage that referenced this pull request Mar 12, 2024
This PR is a dependency of sonic-net/sonic-platform-daemons#383
HLD of sff_mgr: sonic-net/SONiC#1371

Why I did it
Add enable_xcvrd_sff_mgr flag support for sff_mgr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: DeferredForNextRelease
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants