diff options
| author | Nathan Fontenot <nfont@linux.vnet.ibm.com> | 2017-02-10 19:29:06 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-02-12 03:24:15 +0100 |
| commit | e722af6391949e8851310441bb0cec157d25611d (patch) | |
| tree | 5f380ff43acdf1697d9ce6e672aedb2f3f08cccf /drivers | |
| parent | ibmvnic: Initialize completion variables before starting work (diff) | |
| download | linux-e722af6391949e8851310441bb0cec157d25611d.tar.xz linux-e722af6391949e8851310441bb0cec157d25611d.zip | |
ibmvnic: Call napi_disable instead of napi_enable in failure path
The failure path in ibmvnic_open() mistakenly makes a second call
to napi_enable instead of calling napi_disable. This can result
in a BUG_ON for any queues that were enabled in the previous call
to napi_enable.
Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/ibm/ibmvnic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/ibm/ibmvnic.c b/drivers/net/ethernet/ibm/ibmvnic.c index c7150343342d..752b0822b020 100644 --- a/drivers/net/ethernet/ibm/ibmvnic.c +++ b/drivers/net/ethernet/ibm/ibmvnic.c @@ -506,7 +506,7 @@ rx_pool_alloc_failed: adapter->rx_pool = NULL; rx_pool_arr_alloc_failed: for (i = 0; i < adapter->req_rx_queues; i++) - napi_enable(&adapter->napi[i]); + napi_disable(&adapter->napi[i]); alloc_napi_failed: return -ENOMEM; } |