Skip to content

[Nexthop][fboss2] Populate serdes params for G202x in show interface phy #1663

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.

Sign up for GitHub

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

Open
vvasavada-nexthop wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[Nexthop][fboss2] Populate serdes params for G202x in show interface phy #1663

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
418 changes: 363 additions & 55 deletions fboss/agent/hw/sai/switch/SaiPortManager.cpp
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/ChenabAsic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -263,6 +263,7 @@ bool ChenabAsic::isSupportedNonFabric(Feature feature) const {
case HwAsic::Feature::SWITCH_WIDE_LINK_UP_DEBOUNCE:
case HwAsic::Feature::ACL_DST_IPV6_WORD_QUALIFIERS:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::MY_MAC:
case HwAsic::Feature::ISOLATION_GROUP:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/EbroAsic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -280,6 +280,7 @@ bool EbroAsic::isSupportedNonFabric(Feature feature) const {
case HwAsic::Feature::SWITCH_WIDE_LINK_UP_DEBOUNCE:
case HwAsic::Feature::ACL_DST_IPV6_WORD_QUALIFIERS:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::MY_MAC:
case HwAsic::Feature::ISOLATION_GROUP:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/G202xAsic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ bool G202xAsic::isSupportedNonFabric(Feature feature) const {
case HwAsic::Feature::L3_MTU_ERROR_TRAP:
case HwAsic::Feature::ACL_COUNTER_LABEL:
case HwAsic::Feature::SAI_PORT_SERDES_PROGRAMMING:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::PORT_WRED_COUNTER:
case HwAsic::Feature::BRIDGE_PORT_8021Q:
case HwAsic::Feature::WARMBOOT:
Expand Down
5 changes: 5 additions & 0 deletions fboss/agent/hw/switch_asics/HwAsic.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -627,6 +627,11 @@ class HwAsic {
// Each attribute returns the drop reasons seen since the last read, which
// is cleared on read. Counterpart to SWITCH_CUSTOM_DROP_BITMAP_SUPPORT.
SWITCH_DROP_REASON_LIST_SUPPORT,
// Read-back of Cisco SiliconOne (Tajo/G202X) native serdes parameters via
// the SAI_PORT_SERDES_ATTR_EXT_* attributes (TX FFE/driver-swing and RX
// instg/ctle/dsp/afe). Distinct from the Broadcom RX_SERDES_PARAMETERS
// feature, which reads a different (BCM-only) attribute set.
TAJO_RX_SERDES_PARAMETERS,
// SDK reports the type of a received packet, so the application does
// not have to classify the packet itself to identify specific types.
RX_PACKET_TYPE,
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Jericho2Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -264,6 +264,7 @@ bool Jericho2Asic::isSupported(Feature feature) const {
case HwAsic::Feature::HW_PROTECTION_SWITCHOVER_COUNTERS:
case HwAsic::Feature::PBR_ACL_BYTE_COUNTER:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::MY_MAC:
case HwAsic::Feature::ISOLATION_GROUP:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Jericho3Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -308,6 +308,7 @@ bool Jericho3Asic::isSupported(Feature feature) const {
case HwAsic::Feature::XPHY_SERDES_TX_FIR_POST3:
case HwAsic::Feature::LAG_TTL_DECREMENT_DISABLE:
case HwAsic::Feature::PORT_LATCH_STATUS:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
return false;
}
return false;
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Jericho4Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -308,6 +308,7 @@ bool Jericho4Asic::isSupported(Feature feature) const {
case HwAsic::Feature::XPHY_SERDES_TX_FIR_POST3:
case HwAsic::Feature::LAG_TTL_DECREMENT_DISABLE:
case HwAsic::Feature::PORT_LATCH_STATUS:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
return false;
}
return false;
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/P200Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -277,6 +277,7 @@ bool P200Asic::isSupportedNonFabric(Feature feature) const {
case HwAsic::Feature::SWITCH_WIDE_LINK_UP_DEBOUNCE:
case HwAsic::Feature::ACL_DST_IPV6_WORD_QUALIFIERS:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::MY_MAC:
case HwAsic::Feature::ISOLATION_GROUP:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Qumran4DAsic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,6 +301,7 @@ bool Qumran4DAsic::isSupported(Feature feature) const {
case HwAsic::Feature::HW_PROTECTION_SWITCHOVER_COUNTERS:
case HwAsic::Feature::PBR_ACL_BYTE_COUNTER:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::MY_MAC:
case HwAsic::Feature::ISOLATION_GROUP:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Tomahawk3Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -258,6 +258,7 @@ bool Tomahawk3Asic::isSupported(Feature feature) const {
case HwAsic::Feature::HW_PROTECTION_SWITCHOVER_COUNTERS:
case HwAsic::Feature::PBR_ACL_BYTE_COUNTER:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::XPHY_SERDES_TX_FIR_POST3:
case HwAsic::Feature::LAG_TTL_DECREMENT_DISABLE:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Tomahawk4Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -275,6 +275,7 @@ bool Tomahawk4Asic::isSupported(Feature feature) const {
case HwAsic::Feature::PBR_ACL_BYTE_COUNTER:
case HwAsic::Feature::ARS_CURRENT_PORT_LOAD:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::ISOLATION_GROUP:
case HwAsic::Feature::XPHY_SERDES_TX_FIR_POST3:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Tomahawk5Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -258,6 +258,7 @@ bool Tomahawk5Asic::isSupported(Feature feature) const {
case HwAsic::Feature::ACL_DST_IPV6_WORD_QUALIFIERS:
case HwAsic::Feature::SLL_HLL_DISCARD_COUNTERS:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::ISOLATION_GROUP:
case HwAsic::Feature::XPHY_SERDES_TX_FIR_POST3:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/Tomahawk6Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -264,6 +264,7 @@ bool Tomahawk6Asic::isSupported(Feature feature) const {
case HwAsic::Feature::LINK_LAYER_RETRANSMISSION:
case HwAsic::Feature::SLL_HLL_DISCARD_COUNTERS:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::ISOLATION_GROUP:
case HwAsic::Feature::CSIG:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/TomahawkAsic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -256,6 +256,7 @@ bool TomahawkAsic::isSupported(Feature feature) const {
case HwAsic::Feature::HW_PROTECTION_SWITCHOVER_COUNTERS:
case HwAsic::Feature::PBR_ACL_BYTE_COUNTER:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::ISOLATION_GROUP:
case HwAsic::Feature::XPHY_SERDES_TX_FIR_POST3:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/TomahawkUltra1Asic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -259,6 +259,7 @@ bool TomahawkUltra1Asic::isSupported(Feature feature) const {
case HwAsic::Feature::HW_PROTECTION_SWITCHOVER_COUNTERS:
case HwAsic::Feature::PBR_ACL_BYTE_COUNTER:
case HwAsic::Feature::NEXT_HOP_GROUP_MEMBER_MONITORED_OBJECT:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::MY_MAC:
case HwAsic::Feature::ISOLATION_GROUP:
Expand Down
1 change: 1 addition & 0 deletions fboss/agent/hw/switch_asics/YubaAsic.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -274,6 +274,7 @@ bool YubaAsic::isSupportedNonFabric(Feature feature) const {
case HwAsic::Feature::LINK_LAYER_RETRANSMISSION:
case HwAsic::Feature::SLL_HLL_DISCARD_COUNTERS:
case HwAsic::Feature::HW_PROTECTION_SWITCHOVER_COUNTERS:
case HwAsic::Feature::TAJO_RX_SERDES_PARAMETERS:
case HwAsic::Feature::RX_PACKET_TYPE:
case HwAsic::Feature::MY_MAC:
case HwAsic::Feature::ISOLATION_GROUP:
Expand Down
Loading
Loading