RDMA/mlx5: Delete not needed GSI QP signal QP type
authorLeon Romanovsky <leonro@nvidia.com>
Sat, 26 Sep 2020 10:24:44 +0000 (13:24 +0300)
committerJason Gunthorpe <jgg@nvidia.com>
Tue, 29 Sep 2020 16:09:49 +0000 (13:09 -0300)
GSI QP doesn't need signal QP type because it is initialized statically to
zero, which is IB_SIGNAL_ALL_WR also wr->send_flags isn't set too.  This
means that the GSI QP signal QP type can be removed.

Link: https://lore.kernel.org/r/20200926102450.2966017-5-leon@kernel.org
Reviewed-by: Maor Gottlieb <maorg@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@nvidia.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/mlx5/gsi.c
drivers/infiniband/hw/mlx5/mlx5_ib.h

index f5aa116..7fcad91 100644 (file)
@@ -35,7 +35,6 @@
 struct mlx5_ib_gsi_wr {
        struct ib_cqe cqe;
        struct ib_wc wc;
-       int send_flags;
        bool completed:1;
 };
 
@@ -59,10 +58,7 @@ static void generate_completions(struct mlx5_ib_qp *mqp)
                if (!wr->completed)
                        break;
 
-               if (gsi->sq_sig_type == IB_SIGNAL_ALL_WR ||
-                   wr->send_flags & IB_SEND_SIGNALED)
-                       WARN_ON_ONCE(mlx5_ib_generate_wc(gsi_cq, &wr->wc));
-
+               WARN_ON_ONCE(mlx5_ib_generate_wc(gsi_cq, &wr->wc));
                wr->completed = false;
        }
 
@@ -132,7 +128,6 @@ int mlx5_ib_create_gsi(struct ib_pd *pd, struct mlx5_ib_qp *mqp,
        spin_lock_init(&gsi->lock);
 
        gsi->cap = attr->cap;
-       gsi->sq_sig_type = attr->sq_sig_type;
        gsi->port_num = port_num;
 
        gsi->cq = ib_alloc_cq(pd->device, gsi, attr->cap.max_send_wr, 0,
@@ -236,7 +231,6 @@ static struct ib_qp *create_gsi_ud_qp(struct mlx5_ib_gsi_qp *gsi)
                        .max_send_sge = gsi->cap.max_send_sge,
                        .max_inline_data = gsi->cap.max_inline_data,
                },
-               .sq_sig_type = gsi->sq_sig_type,
                .qp_type = IB_QPT_UD,
                .create_flags = MLX5_IB_QP_CREATE_SQPN_QP1,
        };
index 3261720..6ab3efb 100644 (file)
@@ -388,7 +388,6 @@ struct mlx5_ib_gsi_qp {
        struct ib_qp *rx_qp;
        u8 port_num;
        struct ib_qp_cap cap;
-       enum ib_sig_type sq_sig_type;
        struct ib_cq *cq;
        struct mlx5_ib_gsi_wr *outstanding_wrs;
        u32 outstanding_pi, outstanding_ci;