Skip to content

Commit

Permalink
Update doc
Browse files Browse the repository at this point in the history
  • Loading branch information
longhuan-cisco committed Oct 3, 2023
1 parent ddd2a28 commit 9c5e3c3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion doc/port-fec/fec-auto-determination.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ This document is the design document for FEC auto-determination feature on SONiC
If not configured in CONFIG_DB, FEC mode is set to default value `none` at [SAI/SDK](https://github.com/opencomputeproject/SAI/blob/a94bbbe43242a4d9e1a4d9f70780ea9251127f5d/inc/saiport.h#L1012) layer, which might not be the proper FEC mode for this port/optics, and link might not come up.

Two scenarios:
1. In DPB case, today's DPB CLI doesn't generate FEC config for newly created ports, FEC mode is default to none at SAI layer.
1. In DPB(Dynamic Port Breakout) case, today's DPB CLI doesn't generate FEC config for newly created ports, FEC mode is default to none at SAI layer.
2. In non-DPB case,
- Some platforms have no FEC configured in CONFIG_DB by default. The FEC mode can be either default to none at SAI layer or manually configured by user who might not have enough domain knowledge.
- Some platforms have default FEC defined in port_config.ini, which however might not be suitable for the specific port/optics on the system.
Expand Down

0 comments on commit 9c5e3c3

Please sign in to comment.