diff options
| author |   Divy Le Ray <divy@chelsio.com> | 2010-10-25 07:35:02 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2010-10-25 12:14:09 -0700 |
| commit | 18edc84cf15cbe114dcdf7e8612ee52586d0aad7 (patch) | |
| tree | 304d01879a6928586ac68313c54411e2185cc617 /drivers | |
| parent | l2tp: static functions should not be exported (diff) | |
| download | linux-dev-18edc84cf15cbe114dcdf7e8612ee52586d0aad7.tar.xz linux-dev-18edc84cf15cbe114dcdf7e8612ee52586d0aad7.zip | |
cxgb3: fix device opening error path
Only negative return from bind_qsets() should be considered an error and
propagated.
It fixes an issue reported by IBM on P Series platform.
Signed-off-by: Divy Le Ray <divy@chelsio.com>
Tested-by: Nishanth Aravamudan <nacc@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/cxgb3/cxgb3_main.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/net/cxgb3/cxgb3_main.c b/drivers/net/cxgb3/cxgb3_main.c index a04ce6a5f637..4e3c12371aae 100644 --- a/drivers/net/cxgb3/cxgb3_main.c +++ b/drivers/net/cxgb3/cxgb3_main.c @@ -1266,11 +1266,13 @@ static int cxgb_up(struct adapter *adap) } if (!(adap->flags & QUEUES_BOUND)) { - err = bind_qsets(adap); - if (err) { - CH_ERR(adap, "failed to bind qsets, err %d\n", err); + int ret = bind_qsets(adap); + + if (ret < 0) { + CH_ERR(adap, "failed to bind qsets, err %d\n", ret); t3_intr_disable(adap); free_irq_resources(adap); + err = ret; goto out; } adap->flags |= QUEUES_BOUND; |