On Mon, Dec 05, 2016 at 05:18:27PM -0800, Bart Van Assche wrote: > This has been detected by building the mlx5 driver with W=1. > > Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx> > Cc: Eli Cohen <eli@xxxxxxxxxxxx> Thanks, Fixes: 1a412fb1caa2 ('net/mlx5: Fixes: 1a412fb1caa2 (IB/mlx5: Modify QP commands via mlx5 ifc') Acked-by: Leon Romanovsky <leonro@xxxxxxxxxxxx> > --- > drivers/infiniband/hw/mlx5/qp.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/drivers/infiniband/hw/mlx5/qp.c b/drivers/infiniband/hw/mlx5/qp.c > index d1e921816bfe..b2c6c13d44ac 100644 > --- a/drivers/infiniband/hw/mlx5/qp.c > +++ b/drivers/infiniband/hw/mlx5/qp.c > @@ -2577,7 +2577,6 @@ static int __mlx5_ib_modify_qp(struct ib_qp *ibqp, > struct mlx5_ib_port *mibport = NULL; > enum mlx5_qp_state mlx5_cur, mlx5_new; > enum mlx5_qp_optpar optpar; > - int sqd_event; > int mlx5_st; > int err; > u16 op; > @@ -2724,12 +2723,6 @@ static int __mlx5_ib_modify_qp(struct ib_qp *ibqp, > if (qp->rq.wqe_cnt && cur_state == IB_QPS_RESET && new_state == IB_QPS_INIT) > context->db_rec_addr = cpu_to_be64(qp->db.dma); > > - if (cur_state == IB_QPS_RTS && new_state == IB_QPS_SQD && > - attr_mask & IB_QP_EN_SQD_ASYNC_NOTIFY && attr->en_sqd_async_notify) > - sqd_event = 1; > - else > - sqd_event = 0; > - > if (cur_state == IB_QPS_RESET && new_state == IB_QPS_INIT) { > u8 port_num = (attr_mask & IB_QP_PORT ? attr->port_num : > qp->port) - 1; > -- > 2.11.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html