Skip to content

Commit

Permalink
Update Interface-Link-bring-up-sequence.md
Browse files Browse the repository at this point in the history
Address review comments, added more details to 'Proposal' section, added 'Out of Scope' section
  • Loading branch information
shyam77git authored Jan 19, 2022
1 parent 28a67ee commit b802395
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions doc/sfp-cmis/Interface-Link-bring-up-sequence.md
Original file line number Diff line number Diff line change
Expand Up @@ -142,14 +142,11 @@ Please refer to the flow/sequence diagrams which covers the following required
Following items are not in the scope of this document. They would be taken up separately
1. xcvrd restart
- If the xcvrd goes for restart, then all the DB events will be replayed.
Here the Datapath init/activate (for 400G), tx-disable register set (for 100G), will be a no-op if the optics is already in that state

Here the Datapath init/activate (for 400G), tx-disable register set (for 100G), will be a no-op if the optics is already in that state
2. syncd/gbsyncd/swss docker container restart
- Cleanup scenario - the host_tx_ready field in STATE-DB should be updated to “False” to respective ports that a PHY/NPU interface with
-
3. CMIS API feature is not part of this design and the APIs will be used in this design. For CMIS HLD, Please refer to:
https://github.com/Azure/SONiC/blob/9d480087243fd1158e785e3c2f4d35b73c6d1317/doc/sfp-cmis/cmis-init.md

4. Error handling of SAI attributes
a) At present, If there is a set attribute failure, orch agent will exit.
Refer the error handling API : https://github.com/Azure/sonic-swss/blob/master/orchagent/orch.cpp#L885
Expand Down

0 comments on commit b802395

Please sign in to comment.