diff options
| author | Leon Romanovsky <leon@kernel.org> | 2016-11-03 16:44:25 +0200 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2016-12-03 13:12:52 -0500 |
| commit | 74226649f42d34a8ade2799bfb4f3941f4adfa95 (patch) | |
| tree | ad34cf8ea667efcf909cfd2b4dc066c2bb218f9a /drivers/infiniband/ulp/ipoib/ipoib_ib.c | |
| parent | 93b80f29044639ce98baabd3431af9d78e9d1223 (diff) | |
| download | lwn-74226649f42d34a8ade2799bfb4f3941f4adfa95.tar.gz lwn-74226649f42d34a8ade2799bfb4f3941f4adfa95.zip | |
IB/ipoib: Remove and fix debug prints after allocation failure
The prints after [k|v][m|z|c]alloc() functions are not needed,
because in case of failure, allocator will print their internal
error prints anyway.
Signed-off-by: Leon Romanovsky <leon@kernel.org>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/ulp/ipoib/ipoib_ib.c')
| -rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_ib.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_ib.c b/drivers/infiniband/ulp/ipoib/ipoib_ib.c index be11d5d5b8c1..43cf8b8a8d2e 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_ib.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_ib.c @@ -418,11 +418,8 @@ static void ipoib_ib_handle_tx_wc(struct net_device *dev, struct ib_wc *wc) "(status=%d, wrid=%d vend_err %x)\n", wc->status, wr_id, wc->vendor_err); qp_work = kzalloc(sizeof(*qp_work), GFP_ATOMIC); - if (!qp_work) { - ipoib_warn(priv, "%s Failed alloc ipoib_qp_state_validate for qp: 0x%x\n", - __func__, priv->qp->qp_num); + if (!qp_work) return; - } INIT_WORK(&qp_work->work, ipoib_qp_state_validate_work); qp_work->priv = priv; |