]> git.kernelconcepts.de Git - karo-tx-linux.git/commitdiff
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 85637696f6e9600a638e78a48aa9272d18c08883..47d1bbce95614dc79835911e70c19705c2f97ed3 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 ac2f3cd9478c9bd15581a3d488da592edfacf002..a6ad913a30dde395b5ac3be56669190fb792d851 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 6329c971c22fc383330099fa07819b4666ee48b8..62e068b9bf4caa27cd99ab6068957070003dd076 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;
        }