diff options
| author | Breno Leitao <leitao@linux.vnet.ibm.com> | 2010-08-26 04:27:58 -0400 | 
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-08-26 17:09:14 -0400 | 
| commit | fe5f098055ed7f701bfb16f8f87378cf67de9a0e (patch) | |
| tree | 522d231c37c12afe7b0b76a8fe252094ce49b189 | |
| parent | bfc960a8eec023a170a80697fe65157cd4f44f81 (diff) | |
qlge: reset the chip before freeing the buffers
Qlge is freeing the buffers before stopping the card DMA, and
this can cause some severe error, as a EEH event on PPC.
This patch just stop the card and then free the resources.
Signed-off-by: Breno Leitao <leitao@linux.vnet.ibm.com>
Signed-off-by: Ron Mercer <ron.mercer@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/qlge/qlge_main.c | 4 | 
1 files changed, 2 insertions, 2 deletions
| diff --git a/drivers/net/qlge/qlge_main.c b/drivers/net/qlge/qlge_main.c index 8d63f69b27d9..5f89e83501f4 100644 --- a/drivers/net/qlge/qlge_main.c +++ b/drivers/net/qlge/qlge_main.c | |||
| @@ -3919,12 +3919,12 @@ static int ql_adapter_down(struct ql_adapter *qdev) | |||
| 3919 | for (i = 0; i < qdev->rss_ring_count; i++) | 3919 | for (i = 0; i < qdev->rss_ring_count; i++) | 
| 3920 | netif_napi_del(&qdev->rx_ring[i].napi); | 3920 | netif_napi_del(&qdev->rx_ring[i].napi); | 
| 3921 | 3921 | ||
| 3922 | ql_free_rx_buffers(qdev); | ||
| 3923 | |||
| 3924 | status = ql_adapter_reset(qdev); | 3922 | status = ql_adapter_reset(qdev); | 
| 3925 | if (status) | 3923 | if (status) | 
| 3926 | netif_err(qdev, ifdown, qdev->ndev, "reset(func #%d) FAILED!\n", | 3924 | netif_err(qdev, ifdown, qdev->ndev, "reset(func #%d) FAILED!\n", | 
| 3927 | qdev->func); | 3925 | qdev->func); | 
| 3926 | ql_free_rx_buffers(qdev); | ||
| 3927 | |||
| 3928 | return status; | 3928 | return status; | 
| 3929 | } | 3929 | } | 
| 3930 | 3930 | ||
