Skip to content

Commit

Permalink
Enable ECC and fifo overflow error interrupts in 12.0
Browse files Browse the repository at this point in the history
Summary: Now that 12.0.0.6 has support for these interrupts, enable the same in FBOSS.

Reviewed By: jasmeetbagga

Differential Revision: D67284420

fbshipit-source-id: e4ecb3e92e464b6f019967e0a7a61a2837278b5e
  • Loading branch information
Nivin Lawrence authored and facebook-github-bot committed Jan 7, 2025
1 parent c73270b commit 070f1fa
Showing 1 changed file with 0 additions and 17 deletions.
17 changes: 0 additions & 17 deletions fboss/agent/hw/sai/switch/npu/bcm/SaiSwitch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -260,10 +260,6 @@ std::string errorType(sai_switch_error_type_t type) {
return "SAI_SWITCH_ERROR_TYPE_RTP_TABLE_CHANGE";
case SAI_SWITCH_ERROR_TYPE_FABRIC_AUTO_ISOLATION:
return "SAI_SWITCH_ERROR_TYPE_FABRIC_AUTO_ISOLATION";
#if defined(SAI_VERSION_11_7_0_0_DNX_ODP)
// TODO: Fix the 11.7 specific check added. Currently support is not
// available in 12.0 or 11.3, hence using the 11.7 specific check.
// ECC errors
case SAI_SWITCH_ERROR_TYPE_FDA_ECC_ECC_2B_ERR_INT:
return "SAI_SWITCH_ERROR_TYPE_FDA_ECC_ECC_2B_ERR_INT";
case SAI_SWITCH_ERROR_TYPE_FDR_ECC_ECC_2B_ERR_INT:
Expand Down Expand Up @@ -319,7 +315,6 @@ std::string errorType(sai_switch_error_type_t type) {
return "SAI_SWITCH_ERROR_TYPE_FDR_FDR_P_1_MAC_4_P_1_IFM_OVERFLOW_BY_MFIFO_MAC_4";
case SAI_SWITCH_ERROR_TYPE_FDR_FDR_P_2_MAC_4_P_2_IFM_OVERFLOW_BY_MFIFO_MAC_4:
return "SAI_SWITCH_ERROR_TYPE_FDR_FDR_P_2_MAC_4_P_2_IFM_OVERFLOW_BY_MFIFO_MAC_4";
#endif // BRCM_SAI_SDK_DNX_GTE_12_0
#endif
#if defined(BRCM_SAI_SDK_DNX_GTE_11_7)
case SAI_SWITCH_ERROR_TYPE_FIRMWARE_CRASH:
Expand Down Expand Up @@ -490,16 +485,12 @@ bool isEccError(sai_switch_error_type_t type) {
case SAI_SWITCH_ERROR_TYPE_ALIGNER_ECC_ECC_2B_ERR_INT:
case SAI_SWITCH_ERROR_TYPE_EPNI_ECC_ECC_2B_ERR_INT:
case SAI_SWITCH_ERROR_TYPE_FQP_ECC_ECC_2B_ERR_INT:
#if defined(SAI_VERSION_11_7_0_0_DNX_ODP)
// TODO: Fix the 11.7 specific check added. Currently support is not
// available in 12.0 or 11.3, hence using the 11.7 specific check.
case SAI_SWITCH_ERROR_TYPE_FDA_ECC_ECC_2B_ERR_INT:
case SAI_SWITCH_ERROR_TYPE_FDR_ECC_ECC_2B_ERR_INT:
case SAI_SWITCH_ERROR_TYPE_FMAC_ECC_ECC_2B_ERR_INT:
case SAI_SWITCH_ERROR_TYPE_FCR_ECC_ECC_2B_ERR_INT:
case SAI_SWITCH_ERROR_TYPE_FCT_ECC_ECC_2B_ERR_INT:
case SAI_SWITCH_ERROR_TYPE_FDT_ECC_ECC_2B_ERR_INT:
#endif
return true;
default:
break;
Expand All @@ -508,9 +499,6 @@ bool isEccError(sai_switch_error_type_t type) {
}

bool isFdaFifoOverflowError(sai_switch_error_type_t type) {
#if defined(SAI_VERSION_11_7_0_0_DNX_ODP)
// TODO: Fix the 11.7 specific check added. Currently support is not
// available in 12.0 or 11.3, hence using the 11.7 specific check.
switch (type) {
case SAI_SWITCH_ERROR_TYPE_FDA_P_0_OFM_FIFO_OVFLW_DROP_INT:
case SAI_SWITCH_ERROR_TYPE_FDA_P_1_OFM_FIFO_OVFLW_DROP_INT:
Expand All @@ -527,14 +515,10 @@ bool isFdaFifoOverflowError(sai_switch_error_type_t type) {
default:
break;
}
#endif
return false;
}

bool isFdrFifoOverflowError(sai_switch_error_type_t type) {
#if defined(SAI_VERSION_11_7_0_0_DNX_ODP)
// TODO: Fix the 11.7 specific check added. Currently support is not
// available in 12.0 or 11.3, hence using the 11.7 specific check.
switch (type) {
case SAI_SWITCH_ERROR_TYPE_FDR_FDR_P_1_MAC_0_P_1_IFM_OVERFLOW_BY_MFIFO_MAC_0:
case SAI_SWITCH_ERROR_TYPE_FDR_FDR_P_2_MAC_0_P_2_IFM_OVERFLOW_BY_MFIFO_MAC_0:
Expand All @@ -550,7 +534,6 @@ bool isFdrFifoOverflowError(sai_switch_error_type_t type) {
default:
break;
}
#endif
return false;
}

Expand Down

0 comments on commit 070f1fa

Please sign in to comment.