sfc: Remove EFX_WORKAROUND_9141 macro
Subsystems (2)
PATCH 6/12 sfc: Remove EFX_WORKAROUND_9141 macro
From: Matthew Slattery <[email protected]>
The "bug9141 workaround" of setting TX_FLUSH_MIN_LEN_EN should really be considered as a normal bit of configuration rather than a workaround.
Signed-off-by Ben Hutchings <[email protected]>
--- drivers/net/sfc/falcon.c | 2 +- drivers/net/sfc/workarounds.h | 2 -- 2 files changed, 1 insertions(+), 3 deletions(-)
drivers/net/sfc/falcon.c +1 -1
@@ -3128,7 +3128,7 @@ int falcon_init_nic(struct efx_nic *efx)/* Prefetch threshold 2 => fetch when descriptor cache half empty */ EFX_SET_OWORD_FIELD (temp,FRF_AZ_TX_PREF_THRESHOLD ,2 );/* Squash TX of packets of 16 bytes or less */ -if (falcon_rev (efx) >=FALCON_REV_B0 &&EFX_WORKAROUND_9141 (efx)) +if (falcon_rev (efx) >=FALCON_REV_B0 )EFX_SET_OWORD_FIELD (temp,FRF_BZ_TX_FLUSH_MIN_LEN_EN ,1 );efx_writeo (efx, &temp,FR_AZ_TX_RESERVED );
drivers/net/sfc/workarounds.h +0 -2
@@ -28,8 +28,6 @@#define EFX_WORKAROUND_7575 EFX_WORKAROUND_ALWAYS/* Bit-bashed I2C reads cause performance drop */ #define EFX_WORKAROUND_7884 EFX_WORKAROUND_10G -/* TX pkt parser problem with <= 16 byte TXes */ -#define EFX_WORKAROUND_9141 EFX_WORKAROUND_ALWAYS/* 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
Signature · 9 lines
1.6.5.3 -- Ben Hutchings, Senior Software Engineer, Solarflare Communications Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked.
From: Ben Hutchings <[email protected]>
Date: Sat, 28 Nov 2009 15:35:24 +0000
Quoted 7 lines
From: Matthew Slattery <[email protected]>
The "bug9141 workaround" of setting
TX_FLUSH_MIN_LEN_ENshould really be considered as a normal bit of configuration rather than a workaround.Signed-off-by Ben Hutchings <[email protected]>
Applied.