Commit eebe580f authored by Leon Romanovsky's avatar Leon Romanovsky Committed by Jason Gunthorpe

RDMA/mlx5: Delete not needed GSI QP signal QP type

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.orgReviewed-by: default avatarMaor Gottlieb <maorg@mellanox.com>
Signed-off-by: default avatarLeon Romanovsky <leonro@nvidia.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 2dc4d672
...@@ -35,7 +35,6 @@ ...@@ -35,7 +35,6 @@
struct mlx5_ib_gsi_wr { struct mlx5_ib_gsi_wr {
struct ib_cqe cqe; struct ib_cqe cqe;
struct ib_wc wc; struct ib_wc wc;
int send_flags;
bool completed:1; bool completed:1;
}; };
...@@ -59,10 +58,7 @@ static void generate_completions(struct mlx5_ib_qp *mqp) ...@@ -59,10 +58,7 @@ static void generate_completions(struct mlx5_ib_qp *mqp)
if (!wr->completed) if (!wr->completed)
break; break;
if (gsi->sq_sig_type == IB_SIGNAL_ALL_WR || WARN_ON_ONCE(mlx5_ib_generate_wc(gsi_cq, &wr->wc));
wr->send_flags & IB_SEND_SIGNALED)
WARN_ON_ONCE(mlx5_ib_generate_wc(gsi_cq, &wr->wc));
wr->completed = false; wr->completed = false;
} }
...@@ -132,7 +128,6 @@ int mlx5_ib_create_gsi(struct ib_pd *pd, struct mlx5_ib_qp *mqp, ...@@ -132,7 +128,6 @@ int mlx5_ib_create_gsi(struct ib_pd *pd, struct mlx5_ib_qp *mqp,
spin_lock_init(&gsi->lock); spin_lock_init(&gsi->lock);
gsi->cap = attr->cap; gsi->cap = attr->cap;
gsi->sq_sig_type = attr->sq_sig_type;
gsi->port_num = port_num; gsi->port_num = port_num;
gsi->cq = ib_alloc_cq(pd->device, gsi, attr->cap.max_send_wr, 0, 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) ...@@ -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_send_sge = gsi->cap.max_send_sge,
.max_inline_data = gsi->cap.max_inline_data, .max_inline_data = gsi->cap.max_inline_data,
}, },
.sq_sig_type = gsi->sq_sig_type,
.qp_type = IB_QPT_UD, .qp_type = IB_QPT_UD,
.create_flags = MLX5_IB_QP_CREATE_SQPN_QP1, .create_flags = MLX5_IB_QP_CREATE_SQPN_QP1,
}; };
......
...@@ -388,7 +388,6 @@ struct mlx5_ib_gsi_qp { ...@@ -388,7 +388,6 @@ struct mlx5_ib_gsi_qp {
struct ib_qp *rx_qp; struct ib_qp *rx_qp;
u8 port_num; u8 port_num;
struct ib_qp_cap cap; struct ib_qp_cap cap;
enum ib_sig_type sq_sig_type;
struct ib_cq *cq; struct ib_cq *cq;
struct mlx5_ib_gsi_wr *outstanding_wrs; struct mlx5_ib_gsi_wr *outstanding_wrs;
u32 outstanding_pi, outstanding_ci; u32 outstanding_pi, outstanding_ci;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment