Commit bb4874af authored by Yixing Liu's avatar Yixing Liu Committed by Jason Gunthorpe
Browse files

RDMA/hns: Remove redundant 'attr_mask' in modify_qp_init_to_init() 

The attr_mask variable is not used in the function,
so remove it.

Link: https://lore.kernel.org/r/20220922123315.3732205-5-xuhaoyue1@hisilicon.com


Signed-off-by: default avatarYixing Liu <liuyixing1@huawei.com>
Signed-off-by: default avatarHaoyue Xu <xuhaoyue1@hisilicon.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 064fd299

drivers/infiniband/hw/hns/hns_roce_hw_v2.c

+3 −6
Original line number Diff line number Diff line
@@ -4300,7 +4300,6 @@ static inline int get_pdn(struct ib_pd *ib_pd) 


static void modify_qp_reset_to_init(struct ib_qp *ibqp,

				    const struct ib_qp_attr *attr,

				    int attr_mask,

				    struct hns_roce_v2_qp_context *context,

				    struct hns_roce_v2_qp_context *qpc_mask)

{
@@ -4364,7 +4363,7 @@ static void modify_qp_reset_to_init(struct ib_qp *ibqp, 
}



static void modify_qp_init_to_init(struct ib_qp *ibqp,

				   const struct ib_qp_attr *attr, int attr_mask,

				   const struct ib_qp_attr *attr,

				   struct hns_roce_v2_qp_context *context,

				   struct hns_roce_v2_qp_context *qpc_mask)

{
@@ -5015,11 +5014,9 @@ static int hns_roce_v2_set_abs_fields(struct ib_qp *ibqp, 


	if (cur_state == IB_QPS_RESET && new_state == IB_QPS_INIT) {

		memset(qpc_mask, 0, hr_dev->caps.qpc_sz);

		modify_qp_reset_to_init(ibqp, attr, attr_mask, context,

					qpc_mask);

		modify_qp_reset_to_init(ibqp, attr, context, qpc_mask);

	} else if (cur_state == IB_QPS_INIT && new_state == IB_QPS_INIT) {

		modify_qp_init_to_init(ibqp, attr, attr_mask, context,

				       qpc_mask);

		modify_qp_init_to_init(ibqp, attr, context, qpc_mask);

	} else if (cur_state == IB_QPS_INIT && new_state == IB_QPS_RTR) {

		ret = modify_qp_init_to_rtr(ibqp, attr, attr_mask, context,

					    qpc_mask);