Commit 2d18835d authored by Steve Hodgson's avatar Steve Hodgson Committed by David S. Miller
Browse files

sfc: SFX7101: Remove workaround for bad link training 



Early versions of the SFX7101 firmware could complete link training in
a state where it would not adequately cancel noise (Solarflare bug
10750).  We previously worked around this by resetting the PHY after
seeing many Ethernet CRC errors.  This workaround is unsafe since it
takes no account of the interval between errors; it also appears to
be unnecessary with production firmware.  Therefore remove it.

Signed-off-by: default avatarBen Hutchings <bhutchings@solarflare.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 869b5b38

drivers/net/sfc/falcon.c

+0 −4
Original line number Diff line number Diff line
@@ -824,10 +824,6 @@ static void falcon_handle_rx_not_ok(struct efx_rx_queue *rx_queue, 
			    rx_ev_pause_frm ? " [PAUSE]" : "");

	}

#endif



	if (unlikely(rx_ev_eth_crc_err && EFX_WORKAROUND_10750(efx) &&

		     efx->phy_type == PHY_TYPE_SFX7101))

		tenxpress_crc_err(efx);

}



/* Handle receive events that are not in-order. */

drivers/net/sfc/phy.h

+0 −1
Original line number Diff line number Diff line
@@ -17,7 +17,6 @@ extern struct efx_phy_operations falcon_sfx7101_phy_ops; 
extern struct efx_phy_operations falcon_sft9001_phy_ops;



extern void tenxpress_phy_blink(struct efx_nic *efx, bool blink);

extern void tenxpress_crc_err(struct efx_nic *efx);



/****************************************************************************

 * Exported functions from the driver for XFP optical PHYs

drivers/net/sfc/tenxpress.c

+0 −20
Original line number Diff line number Diff line
@@ -189,25 +189,12 @@ 
 * rails */

#define LNPGA_PDOWN_WAIT	(HZ / 5)



static int crc_error_reset_threshold = 100;

module_param(crc_error_reset_threshold, int, 0644);

MODULE_PARM_DESC(crc_error_reset_threshold,

		 "Max number of CRC errors before XAUI reset");



struct tenxpress_phy_data {

	enum efx_loopback_mode loopback_mode;

	atomic_t bad_crc_count;

	enum efx_phy_mode phy_mode;

	int bad_lp_tries;

};



void tenxpress_crc_err(struct efx_nic *efx)

{

	struct tenxpress_phy_data *phy_data = efx->phy_data;

	if (phy_data != NULL)

		atomic_inc(&phy_data->bad_crc_count);

}



static ssize_t show_phy_short_reach(struct device *dev,

				    struct device_attribute *attr, char *buf)

{
@@ -627,13 +614,6 @@ static void tenxpress_phy_poll(struct efx_nic *efx) 


	if (phy_data->phy_mode != PHY_MODE_NORMAL)

		return;



	if (EFX_WORKAROUND_10750(efx) &&

	    atomic_read(&phy_data->bad_crc_count) > crc_error_reset_threshold) {

		EFX_ERR(efx, "Resetting XAUI due to too many CRC errors\n");

		falcon_reset_xaui(efx);

		atomic_set(&phy_data->bad_crc_count, 0);

	}

}



static void tenxpress_phy_fini(struct efx_nic *efx)

drivers/net/sfc/workarounds.h

+0 −3
Original line number Diff line number Diff line
@@ -18,7 +18,6 @@ 
#define EFX_WORKAROUND_ALWAYS(efx) 1

#define EFX_WORKAROUND_FALCON_A(efx) (falcon_rev(efx) <= FALCON_REV_A1)

#define EFX_WORKAROUND_10G(efx) EFX_IS10G(efx)

#define EFX_WORKAROUND_SFX7101(efx) ((efx)->phy_type == PHY_TYPE_SFX7101)

#define EFX_WORKAROUND_SFT9001A(efx) ((efx)->phy_type == PHY_TYPE_SFT9001A)



/* XAUI resets if link not detected */
@@ -29,8 +28,6 @@ 
#define EFX_WORKAROUND_7884 EFX_WORKAROUND_10G

/* TX pkt parser problem with <= 16 byte TXes */

#define EFX_WORKAROUND_9141 EFX_WORKAROUND_ALWAYS

/* Low rate CRC errors require XAUI reset */

#define EFX_WORKAROUND_10750 EFX_WORKAROUND_SFX7101

/* TX_EV_PKT_ERR can be caused by a dangling TX descriptor

 * or a PCIe error (bug 11028) */

#define EFX_WORKAROUND_10727 EFX_WORKAROUND_ALWAYS

CRA Git | Maintained and supported by SUSTech CRA and CCSE