IB/i40iw: Remove debug prints after allocation failure
authorLeon Romanovsky <leon@kernel.org>
Thu, 3 Nov 2016 14:44:17 +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/i40iw/i40iw_cm.c
drivers/infiniband/hw/i40iw/i40iw_main.c
drivers/infiniband/hw/i40iw/i40iw_verbs.c

index 8563769..47d1bbc 100644 (file)
@@ -1675,7 +1675,6 @@ static enum i40iw_status_code i40iw_add_mqh_6(struct i40iw_device *iwdev,
                                            "Allocating child listener %p\n",
                                            child_listen_node);
                                if (!child_listen_node) {
-                                       i40iw_pr_err("listener memory allocation\n");
                                        ret = I40IW_ERR_NO_MEMORY;
                                        goto exit;
                                }
@@ -1751,7 +1750,6 @@ static enum i40iw_status_code i40iw_add_mqh_4(
                                            "Allocating child listener %p\n",
                                            child_listen_node);
                                if (!child_listen_node) {
-                                       i40iw_pr_err("listener memory allocation\n");
                                        in_dev_put(idev);
                                        ret = I40IW_ERR_NO_MEMORY;
                                        goto exit;
index ac2f3cd..a6ad913 100644 (file)
@@ -1301,10 +1301,9 @@ static enum i40iw_status_code i40iw_initialize_dev(struct i40iw_device *iwdev,
        size = sizeof(struct i40iw_hmc_pble_rsrc) + sizeof(struct i40iw_hmc_info) +
                                (sizeof(struct i40iw_hmc_obj_info) * I40IW_HMC_IW_MAX);
        iwdev->hmc_info_mem = kzalloc(size, GFP_KERNEL);
-       if (!iwdev->hmc_info_mem) {
-               i40iw_pr_err("memory alloc fail\n");
+       if (!iwdev->hmc_info_mem)
                return I40IW_ERR_NO_MEMORY;
-       }
+
        iwdev->pble_rsrc = (struct i40iw_hmc_pble_rsrc *)iwdev->hmc_info_mem;
        dev->hmc_info = &iwdev->hw.hmc;
        dev->hmc_info->hmc_obj = (struct i40iw_hmc_obj_info *)(iwdev->pble_rsrc + 1);
index 6329c97..62e068b 100644 (file)
@@ -2654,7 +2654,6 @@ static struct i40iw_ib_device *i40iw_init_rdma_device(struct i40iw_device *iwdev
        iwibdev->ibdev.iwcm = kzalloc(sizeof(*iwibdev->ibdev.iwcm), GFP_KERNEL);
        if (!iwibdev->ibdev.iwcm) {
                ib_dealloc_device(&iwibdev->ibdev);
-               i40iw_pr_err("iwcm == NULL\n");
                return NULL;
        }