IB/cxgb3: Remove debug prints after allocation failure
authorLeon Romanovsky <leon@kernel.org>
Thu, 3 Nov 2016 14:44:15 +0000 (16:44 +0200)
committerDoug Ledford <dledford@redhat.com>
Sat, 3 Dec 2016 18:12:52 +0000 (13:12 -0500)
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>
drivers/infiniband/hw/cxgb3/cxio_dbg.c

index 8bca6b4ec9af2003e40cbb48767c0f1106646e76..445e89e5e7cfe89ae175a848ffab721efe8553dc 100644 (file)
@@ -45,10 +45,9 @@ void cxio_dump_tpt(struct cxio_rdev *rdev, u32 stag)
        int size = 32;
 
        m = kmalloc(sizeof(*m) + size, GFP_ATOMIC);
-       if (!m) {
-               PDBG("%s couldn't allocate memory.\n", __func__);
+       if (!m)
                return;
-       }
+
        m->mem_id = MEM_PMRX;
        m->addr = (stag>>8) * 32 + rdev->rnic_info.tpt_base;
        m->len = size;
@@ -82,10 +81,9 @@ void cxio_dump_pbl(struct cxio_rdev *rdev, u32 pbl_addr, uint len, u8 shift)
        size = npages * sizeof(u64);
 
        m = kmalloc(sizeof(*m) + size, GFP_ATOMIC);
-       if (!m) {
-               PDBG("%s couldn't allocate memory.\n", __func__);
+       if (!m)
                return;
-       }
+
        m->mem_id = MEM_PMRX;
        m->addr = pbl_addr;
        m->len = size;
@@ -144,10 +142,9 @@ void cxio_dump_rqt(struct cxio_rdev *rdev, u32 hwtid, int nents)
        int rc;
 
        m = kmalloc(sizeof(*m) + size, GFP_ATOMIC);
-       if (!m) {
-               PDBG("%s couldn't allocate memory.\n", __func__);
+       if (!m)
                return;
-       }
+
        m->mem_id = MEM_PMRX;
        m->addr = ((hwtid)<<10) + rdev->rnic_info.rqt_base;
        m->len = size;
@@ -177,10 +174,9 @@ void cxio_dump_tcb(struct cxio_rdev *rdev, u32 hwtid)
        int rc;
 
        m = kmalloc(sizeof(*m) + size, GFP_ATOMIC);
-       if (!m) {
-               PDBG("%s couldn't allocate memory.\n", __func__);
+       if (!m)
                return;
-       }
+
        m->mem_id = MEM_CM;
        m->addr = hwtid * size;
        m->len = size;