Commit bfb3bde9 authored by Wenpeng Liang's avatar Wenpeng Liang Committed by Leon Romanovsky
Browse files

RDMA/hns: Remove redundant member doorbell_qpn of struct hns_roce_qp 

The value of doorbell_qpn is always equal to qpn on current hardware
versions. So remove it.

Link: https://lore.kernel.org/r/20220829105021.1427804-5-liangwenpeng@huawei.com


Signed-off-by: default avatarWenpeng Liang <liangwenpeng@huawei.com>
Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent 637ff8ea

drivers/infiniband/hw/hns/hns_roce_device.h

+0 −1
Original line number Diff line number Diff line
@@ -599,7 +599,6 @@ struct hns_roce_qp { 
	struct hns_roce_db	rdb;

	struct hns_roce_db	sdb;

	unsigned long		en_flags;

	u32			doorbell_qpn;

	enum ib_sig_type	sq_signal_bits;

	struct hns_roce_wq	sq;

drivers/infiniband/hw/hns/hns_roce_hw_v2.c

+1 −1
Original line number Diff line number Diff line
@@ -637,7 +637,7 @@ static inline void update_sq_db(struct hns_roce_dev *hr_dev, 
	} else {

		struct hns_roce_v2_db sq_db = {};



		hr_reg_write(&sq_db, DB_TAG, qp->doorbell_qpn);

		hr_reg_write(&sq_db, DB_TAG, qp->qpn);

		hr_reg_write(&sq_db, DB_CMD, HNS_ROCE_V2_SQ_DB);

		hr_reg_write(&sq_db, DB_PI, qp->sq.head);

		hr_reg_write(&sq_db, DB_SL, qp->sl);

drivers/infiniband/hw/hns/hns_roce_qp.c

+0 −3
Original line number Diff line number Diff line
@@ -218,7 +218,6 @@ static int alloc_qpn(struct hns_roce_dev *hr_dev, struct hns_roce_qp *hr_qp) 


	if (hr_qp->ibqp.qp_type == IB_QPT_GSI) {

		num = 1;

		hr_qp->doorbell_qpn = 1;

	} else {

		mutex_lock(&qp_table->bank_mutex);

		bankid = get_least_load_bankid_for_qp(qp_table->bank);
@@ -234,8 +233,6 @@ static int alloc_qpn(struct hns_roce_dev *hr_dev, struct hns_roce_qp *hr_qp) 


		qp_table->bank[bankid].inuse++;

		mutex_unlock(&qp_table->bank_mutex);



		hr_qp->doorbell_qpn = (u32)num;

	}



	hr_qp->qpn = num;