diff options
author | Steve Hodgson <shodgson@solarflare.com> | 2010-06-01 07:17:51 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-06-02 05:21:06 -0400 |
commit | fd371e32fe53f137a0f940d61772bda92180007b (patch) | |
tree | c2ce5d564aae698ecc643d8af42696e374a60728 /drivers/net/sfc/nic.c | |
parent | 319ba649af30321ea221740833785b46e1fe6af3 (diff) |
sfc: Workaround flush failures on Falcon B0
Under certain conditions a PHY may backpressure Falcon B0
in such a way that flushes timeout. In normal circumstances
the phy poller would fix the PHY, and the flush could complete.
But efx_nic_flush_queues() is always called after efx_stop_all(),
so the poller has been stopped. Even if this weren't the case,
how long would we have to wait for the poller to fix this? And
several callers of efx_nic_flush_queues() are about to reset
the device anyway - so we don't need to do anything.
Work around this bug by scheduling a reset. Ensure that the
MAC is never rewired back into the datapath before the reset
runs (we already ignore all rx events anyway).
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/sfc/nic.c')
-rw-r--r-- | drivers/net/sfc/nic.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/net/sfc/nic.c b/drivers/net/sfc/nic.c index 5d3aaec58556..ec0bb80f5ae8 100644 --- a/drivers/net/sfc/nic.c +++ b/drivers/net/sfc/nic.c | |||
@@ -1219,9 +1219,6 @@ int efx_nic_flush_queues(struct efx_nic *efx) | |||
1219 | rx_queue->flushed = FLUSH_DONE; | 1219 | rx_queue->flushed = FLUSH_DONE; |
1220 | } | 1220 | } |
1221 | 1221 | ||
1222 | if (EFX_WORKAROUND_7803(efx)) | ||
1223 | return 0; | ||
1224 | |||
1225 | return -ETIMEDOUT; | 1222 | return -ETIMEDOUT; |
1226 | } | 1223 | } |
1227 | 1224 | ||