mlx5: don't duplicate kvfree()
authorAl Viro <viro@ZenIV.linux.org.uk>
Thu, 20 Nov 2014 08:13:57 +0000 (08:13 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 21 Nov 2014 19:58:18 +0000 (14:58 -0500)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Acked-by: Eli Cohen <eli@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/infiniband/hw/mlx5/cq.c
drivers/infiniband/hw/mlx5/mr.c
drivers/infiniband/hw/mlx5/qp.c
drivers/infiniband/hw/mlx5/srq.c
drivers/net/ethernet/mellanox/mlx5/core/eq.c
drivers/net/ethernet/mellanox/mlx5/core/pagealloc.c
drivers/net/ethernet/mellanox/mlx5/core/port.c
include/linux/mlx5/driver.h

index 10cfce5119a9d2f4ef89c68967236285e7a8730e..c463e7bba5f453f0303a74b88ec96275c796ee0f 100644 (file)
@@ -805,14 +805,14 @@ struct ib_cq *mlx5_ib_create_cq(struct ib_device *ibdev, int entries,
                }
 
 
-       mlx5_vfree(cqb);
+       kvfree(cqb);
        return &cq->ibcq;
 
 err_cmd:
        mlx5_core_destroy_cq(dev->mdev, &cq->mcq);
 
 err_cqb:
-       mlx5_vfree(cqb);
+       kvfree(cqb);
        if (context)
                destroy_cq_user(cq, context);
        else
@@ -1159,11 +1159,11 @@ int mlx5_ib_resize_cq(struct ib_cq *ibcq, int entries, struct ib_udata *udata)
        }
        mutex_unlock(&cq->resize_mutex);
 
-       mlx5_vfree(in);
+       kvfree(in);
        return 0;
 
 ex_alloc:
-       mlx5_vfree(in);
+       kvfree(in);
 
 ex_resize:
        if (udata)
index 8ee7cb46e0590668bd79277bc672a9d41f90f02d..4c89b64aa9cfb1528a1ba9ad34a43bc07ea3ea97 100644 (file)
@@ -853,14 +853,14 @@ static struct mlx5_ib_mr *reg_create(struct ib_pd *pd, u64 virt_addr,
                goto err_2;
        }
        mr->umem = umem;
-       mlx5_vfree(in);
+       kvfree(in);
 
        mlx5_ib_dbg(dev, "mkey = 0x%x\n", mr->mmr.key);
 
        return mr;
 
 err_2:
-       mlx5_vfree(in);
+       kvfree(in);
 
 err_1:
        kfree(mr);
index e261a53f9a02a5edf18725cc75daaef3661fc64a..0e2ef9fe0e2921175f29756c485ef9980767cbbe 100644 (file)
@@ -647,7 +647,7 @@ err_unmap:
        mlx5_ib_db_unmap_user(context, &qp->db);
 
 err_free:
-       mlx5_vfree(*in);
+       kvfree(*in);
 
 err_umem:
        if (qp->umem)
@@ -761,7 +761,7 @@ err_wrid:
        kfree(qp->rq.wrid);
 
 err_free:
-       mlx5_vfree(*in);
+       kvfree(*in);
 
 err_buf:
        mlx5_buf_free(dev->mdev, &qp->buf);
@@ -971,7 +971,7 @@ static int create_qp_common(struct mlx5_ib_dev *dev, struct ib_pd *pd,
                goto err_create;
        }
 
-       mlx5_vfree(in);
+       kvfree(in);
        /* Hardware wants QPN written in big-endian order (after
         * shifting) for send doorbell.  Precompute this value to save
         * a little bit when posting sends.
@@ -988,7 +988,7 @@ err_create:
        else if (qp->create_type == MLX5_QP_KERNEL)
                destroy_qp_kernel(dev, qp);
 
-       mlx5_vfree(in);
+       kvfree(in);
        return err;
 }
 
index 97cc1baaa8e3e8d1a3ea2c8b66dd344b14dc470b..41fec66217dd3b2b7f61fcff7068f01a06493933 100644 (file)
@@ -141,7 +141,7 @@ static int create_srq_user(struct ib_pd *pd, struct mlx5_ib_srq *srq,
        return 0;
 
 err_in:
-       mlx5_vfree(*in);
+       kvfree(*in);
 
 err_umem:
        ib_umem_release(srq->umem);
@@ -209,7 +209,7 @@ static int create_srq_kernel(struct mlx5_ib_dev *dev, struct mlx5_ib_srq *srq,
        return 0;
 
 err_in:
-       mlx5_vfree(*in);
+       kvfree(*in);
 
 err_buf:
        mlx5_buf_free(dev->mdev, &srq->buf);
@@ -306,7 +306,7 @@ struct ib_srq *mlx5_ib_create_srq(struct ib_pd *pd,
        in->ctx.pd = cpu_to_be32(to_mpd(pd)->pdn);
        in->ctx.db_record = cpu_to_be64(srq->db.dma);
        err = mlx5_core_create_srq(dev->mdev, &srq->msrq, in, inlen);
-       mlx5_vfree(in);
+       kvfree(in);
        if (err) {
                mlx5_ib_dbg(dev, "create SRQ failed, err %d\n", err);
                goto err_usr_kern_srq;
index ad2c96a02a5352caec2d8649c714c3616be98ea8..dfd3ad0a39c16e3d26e7b1feed467a6abbce4235 100644 (file)
@@ -390,7 +390,7 @@ int mlx5_create_map_eq(struct mlx5_core_dev *dev, struct mlx5_eq *eq, u8 vecidx,
         */
        eq_update_ci(eq, 1);
 
-       mlx5_vfree(in);
+       kvfree(in);
        return 0;
 
 err_irq:
@@ -400,7 +400,7 @@ err_eq:
        mlx5_cmd_destroy_eq(dev, eq->eqn);
 
 err_in:
-       mlx5_vfree(in);
+       kvfree(in);
 
 err_buf:
        mlx5_buf_free(dev, &eq->buf);
index d476918ef269662eb1d77da8d81b99deb3d7617c..4fdaae9b54d99f56f21a78706776805303ba5ed9 100644 (file)
@@ -349,7 +349,7 @@ out_4k:
        for (i--; i >= 0; i--)
                free_4k(dev, be64_to_cpu(in->pas[i]));
 out_free:
-       mlx5_vfree(in);
+       kvfree(in);
        return err;
 }
 
@@ -400,7 +400,7 @@ static int reclaim_pages(struct mlx5_core_dev *dev, u32 func_id, int npages,
        }
 
 out_free:
-       mlx5_vfree(out);
+       kvfree(out);
        return err;
 }
 
index 313965853e10c14a603e71f6207b267c63aafb45..72c2d002c3b8f65e78b89733c87c44d3ba702089 100644 (file)
@@ -68,9 +68,9 @@ int mlx5_core_access_reg(struct mlx5_core_dev *dev, void *data_in,
                memcpy(data_out, out->data, size_out);
 
 ex2:
-       mlx5_vfree(out);
+       kvfree(out);
 ex1:
-       mlx5_vfree(in);
+       kvfree(in);
        return err;
 }
 EXPORT_SYMBOL_GPL(mlx5_core_access_reg);
index 246310dc8bef9a8f8604941ee40353868ebd3965..b1bf41556b3245fc9970174f5be872202a58eec5 100644 (file)
@@ -633,14 +633,6 @@ static inline void *mlx5_vzalloc(unsigned long size)
        return rtn;
 }
 
-static inline void mlx5_vfree(const void *addr)
-{
-       if (addr && is_vmalloc_addr(addr))
-               vfree(addr);
-       else
-               kfree(addr);
-}
-
 static inline u32 mlx5_base_mkey(const u32 key)
 {
        return key & 0xffffff00u;