Commit 45baad7d authored by Yixing Liu's avatar Yixing Liu Committed by Leon Romanovsky
Browse files

RDMA/hns: Remove the num_qpc_timer variable 

The bt number of qpc_timer of HIP09 increases compared with that of HIP08.
Therefore, qpc_timer_bt_num and num_qpc_timer do not match. As a result,
the driver may fail to allocate qpc_timer. So the driver needs to uniquely
uses qpc_timer_bt_num to represent the bt number of qpc_timer.

Fixes: 0e40dc2f ("RDMA/hns: Add timer allocation support for hip08")
Link: https://lore.kernel.org/r/20220829105021.1427804-4-liangwenpeng@huawei.com


Signed-off-by: default avatarYixing Liu <liuyixing1@huawei.com>
Signed-off-by: default avatarWenpeng Liang <liangwenpeng@huawei.com>
Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent 0c8b5d62

drivers/infiniband/hw/hns/hns_roce_device.h

+0 −1
Original line number Diff line number Diff line
@@ -730,7 +730,6 @@ struct hns_roce_caps { 
	u32		num_qps;

	u32		num_pi_qps;

	u32		reserved_qps;

	int		num_qpc_timer;

	u32		num_srqs;

	u32		max_wqes;

	u32		max_srq_wrs;

drivers/infiniband/hw/hns/hns_roce_hw_v2.c

+1 −2
Original line number Diff line number Diff line
@@ -1977,7 +1977,7 @@ static void set_default_caps(struct hns_roce_dev *hr_dev) 


	caps->num_mtpts		= HNS_ROCE_V2_MAX_MTPT_NUM;

	caps->num_pds		= HNS_ROCE_V2_MAX_PD_NUM;

	caps->num_qpc_timer	= HNS_ROCE_V2_MAX_QPC_TIMER_NUM;

	caps->qpc_timer_bt_num	= HNS_ROCE_V2_MAX_QPC_TIMER_BT_NUM;

	caps->cqc_timer_bt_num	= HNS_ROCE_V2_MAX_CQC_TIMER_BT_NUM;



	caps->max_qp_init_rdma	= HNS_ROCE_V2_MAX_QP_INIT_RDMA;
@@ -2273,7 +2273,6 @@ static int hns_roce_query_pf_caps(struct hns_roce_dev *hr_dev) 
	caps->max_rq_sg		     = le16_to_cpu(resp_a->max_rq_sg);

	caps->max_rq_sg = roundup_pow_of_two(caps->max_rq_sg);

	caps->max_extend_sg	     = le32_to_cpu(resp_a->max_extend_sg);

	caps->num_qpc_timer	     = le16_to_cpu(resp_a->num_qpc_timer);

	caps->max_srq_sges	     = le16_to_cpu(resp_a->max_srq_sges);

	caps->max_srq_sges = roundup_pow_of_two(caps->max_srq_sges);

	caps->num_aeq_vectors	     = resp_a->num_aeq_vectors;

drivers/infiniband/hw/hns/hns_roce_hw_v2.h

+1 −1
Original line number Diff line number Diff line
@@ -36,11 +36,11 @@ 
#include <linux/bitops.h>



#define HNS_ROCE_V2_MAX_QP_NUM			0x1000

#define HNS_ROCE_V2_MAX_QPC_TIMER_NUM		0x200

#define HNS_ROCE_V2_MAX_WQE_NUM			0x8000

#define HNS_ROCE_V2_MAX_SRQ_WR			0x8000

#define HNS_ROCE_V2_MAX_SRQ_SGE			64

#define HNS_ROCE_V2_MAX_CQ_NUM			0x100000

#define HNS_ROCE_V2_MAX_QPC_TIMER_BT_NUM	0x100

#define HNS_ROCE_V2_MAX_CQC_TIMER_BT_NUM	0x100

#define HNS_ROCE_V2_MAX_SRQ_NUM			0x100000

#define HNS_ROCE_V2_MAX_CQE_NUM			0x400000

drivers/infiniband/hw/hns/hns_roce_main.c

+1 −1
Original line number Diff line number Diff line
@@ -725,7 +725,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
		ret = hns_roce_init_hem_table(hr_dev, &hr_dev->qpc_timer_table,

					      HEM_TYPE_QPC_TIMER,

					      hr_dev->caps.qpc_timer_entry_sz,

					      hr_dev->caps.num_qpc_timer, 1);

					      hr_dev->caps.qpc_timer_bt_num, 1);

		if (ret) {

			dev_err(dev,

				"Failed to init QPC timer memory, aborting.\n");