Commit ef524f2e authored by Steve Hodgson's avatar Steve Hodgson Committed by David S. Miller

sfc: Stop masking out XGMII faults over reconfigures

The aim of this code was to avoid a spurious XGMII fault over a MAC
reconfigure. It's less relevant now that the PHY reconfigure isn't
called from the MAC reconfigure.

After applying this patch, our link stress test passed 48 hours of
testing without ever resetting the PHY.
Signed-off-by: default avatarBen Hutchings <bhutchings@solarflare.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 63695459
...@@ -85,14 +85,14 @@ int falcon_reset_xaui(struct efx_nic *efx) ...@@ -85,14 +85,14 @@ int falcon_reset_xaui(struct efx_nic *efx)
return -ETIMEDOUT; return -ETIMEDOUT;
} }
static void falcon_mask_status_intr(struct efx_nic *efx, bool enable) static void falcon_ack_status_intr(struct efx_nic *efx)
{ {
efx_oword_t reg; efx_oword_t reg;
if ((efx_nic_rev(efx) != EFX_REV_FALCON_B0) || LOOPBACK_INTERNAL(efx)) if ((efx_nic_rev(efx) != EFX_REV_FALCON_B0) || LOOPBACK_INTERNAL(efx))
return; return;
/* We expect xgmii faults if the wireside link is up */ /* We expect xgmii faults if the wireside link is down */
if (!EFX_WORKAROUND_5147(efx) || !efx->link_state.up) if (!EFX_WORKAROUND_5147(efx) || !efx->link_state.up)
return; return;
...@@ -101,14 +101,7 @@ static void falcon_mask_status_intr(struct efx_nic *efx, bool enable) ...@@ -101,14 +101,7 @@ static void falcon_mask_status_intr(struct efx_nic *efx, bool enable)
if (efx->xmac_poll_required) if (efx->xmac_poll_required)
return; return;
/* Flush the ISR */ efx_reado(efx, &reg, FR_AB_XM_MGT_INT_MSK);
if (enable)
efx_reado(efx, &reg, FR_AB_XM_MGT_INT_MSK);
EFX_POPULATE_OWORD_2(reg,
FRF_AB_XM_MSK_RMTFLT, !enable,
FRF_AB_XM_MSK_LCLFLT, !enable);
efx_writeo(efx, &reg, FR_AB_XM_MGT_INT_MASK);
} }
static bool falcon_xgxs_link_ok(struct efx_nic *efx) static bool falcon_xgxs_link_ok(struct efx_nic *efx)
...@@ -283,15 +276,13 @@ static bool falcon_xmac_check_fault(struct efx_nic *efx) ...@@ -283,15 +276,13 @@ static bool falcon_xmac_check_fault(struct efx_nic *efx)
static int falcon_reconfigure_xmac(struct efx_nic *efx) static int falcon_reconfigure_xmac(struct efx_nic *efx)
{ {
falcon_mask_status_intr(efx, false);
falcon_reconfigure_xgxs_core(efx); falcon_reconfigure_xgxs_core(efx);
falcon_reconfigure_xmac_core(efx); falcon_reconfigure_xmac_core(efx);
falcon_reconfigure_mac_wrapper(efx); falcon_reconfigure_mac_wrapper(efx);
efx->xmac_poll_required = !falcon_xmac_link_ok_retry(efx, 5); efx->xmac_poll_required = !falcon_xmac_link_ok_retry(efx, 5);
falcon_mask_status_intr(efx, true); falcon_ack_status_intr(efx);
return 0; return 0;
} }
...@@ -362,9 +353,8 @@ void falcon_poll_xmac(struct efx_nic *efx) ...@@ -362,9 +353,8 @@ void falcon_poll_xmac(struct efx_nic *efx)
!efx->xmac_poll_required) !efx->xmac_poll_required)
return; return;
falcon_mask_status_intr(efx, false);
efx->xmac_poll_required = !falcon_xmac_link_ok_retry(efx, 1); efx->xmac_poll_required = !falcon_xmac_link_ok_retry(efx, 1);
falcon_mask_status_intr(efx, true); falcon_ack_status_intr(efx);
} }
struct efx_mac_operations falcon_xmac_operations = { struct efx_mac_operations falcon_xmac_operations = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment