diff options
author | Nathan Fontenot <nfont@linux.vnet.ibm.com> | 2017-02-10 13:29:06 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-02-11 21:24:15 -0500 |
commit | e722af6391949e8851310441bb0cec157d25611d (patch) | |
tree | 5f380ff43acdf1697d9ce6e672aedb2f3f08cccf | |
parent | db5d0b597bc27bbddf40f2f8359a73be4eb77104 (diff) | |
download | op-kernel-dev-e722af6391949e8851310441bb0cec157d25611d.zip op-kernel-dev-e722af6391949e8851310441bb0cec157d25611d.tar.gz |
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>
-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 c715034..752b082 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; } |