Re: [PATCH for-next 3/4] RDMA/hns: Add support for QP's restrack attributes

[Leon Romanovsky <leon@kernel.org>]

On Mon, Jan 24, 2022 at 08:46:23PM +0800, Wenpeng Liang wrote:
> The restrack attributes of QP come from the QPC and the queue information
> maintained by the software code.
> 
> Signed-off-by: Wenpeng Liang <liangwenpeng@huawei.com>
> ---
>  drivers/infiniband/hw/hns/hns_roce_device.h   |   2 +
>  drivers/infiniband/hw/hns/hns_roce_hw_v2.c    |  16 +--
>  drivers/infiniband/hw/hns/hns_roce_main.c     |   1 +
>  drivers/infiniband/hw/hns/hns_roce_restrack.c | 116 ++++++++++++++++++
>  4 files changed, 128 insertions(+), 7 deletions(-)

<...>

> +	if (ret) {
> +		dev_err(hr_dev->dev, "QUERY qpc cmd process error.\n");

ibdev_err(...) in all places.

>  		goto out;
> +	}

<...>

> +static int hns_roce_fill_qp(struct hns_roce_qp *hr_qp,
> +			    struct sk_buff *msg,
> +			    struct hns_roce_v2_qp_context *context)
> +{
> +	static struct {
> +		char *name;
> +		u32 mask;
> +		u32 l;
> +	} reg[] = {
> +		{ "tst", HR_REG_CFG(QPC_TST) },
> +		{ "qp_st", HR_REG_CFG(QPC_QP_ST) },
> +		{ "chk_flg", HR_REG_CFG(QPC_CHECK_FLG) },
> +		{ "err_type", HR_REG_CFG(QPC_ERR_TYPE) },
> +		{ "srq_en", HR_REG_CFG(QPC_SRQ_EN) },
> +		{ "srqn", HR_REG_CFG(QPC_SRQN) },
> +		{ "qkey_xrcd", HR_REG_CFG(QPC_QKEY_XRCD) },
> +		{ "tx_cqn", HR_REG_CFG(QPC_TX_CQN) },
> +		{ "rx_cqn", HR_REG_CFG(QPC_RX_CQN) },
> +		{ "sq_pi", HR_REG_CFG(QPC_SQ_PRODUCER_IDX) },
> +		{ "sq_ci", HR_REG_CFG(QPC_SQ_CONSUMER_IDX) },
> +		{ "rq_pi", HR_REG_CFG(QPC_RQ_PRODUCER_IDX) },
> +		{ "rq_ci", HR_REG_CFG(QPC_RQ_CONSUMER_IDX) },
> +		{ "sq_shift", HR_REG_CFG(QPC_SQ_SHIFT) },
> +		{ "rqws", HR_REG_CFG(QPC_RQWS) },
> +		{ "rq_shift", HR_REG_CFG(QPC_RQ_SHIFT) },
> +		{ "sge_shift", HR_REG_CFG(QPC_SGE_SHIFT) },
> +		{ "max_ird", HR_REG_CFG(QPC_SR_MAX) },
> +		{ "max_ord", HR_REG_CFG(QPC_RR_MAX) },
> +		{ "gmv_idx", HR_REG_CFG(QPC_GMV_IDX) },
> +		{ "sq_vlan_en", HR_REG_CFG(QPC_SQ_VLAN_EN) },
> +		{ "rq_vlan_en", HR_REG_CFG(QPC_RQ_VLAN_EN) },
> +		{ "vlan_id", HR_REG_CFG(QPC_VLAN_ID) },
> +		{ "mtu", HR_REG_CFG(QPC_MTU) },
> +		{ "hop_limit", HR_REG_CFG(QPC_HOPLIMIT) },
> +		{ "tc", HR_REG_CFG(QPC_TC) },
> +		{ "fl", HR_REG_CFG(QPC_FL) },
> +		{ "sl", HR_REG_CFG(QPC_SL) },
> +		{ "rre", HR_REG_CFG(QPC_RRE) },
> +		{ "rwe", HR_REG_CFG(QPC_RWE) },
> +		{ "ate", HR_REG_CFG(QPC_ATE) },
> +		{ "ext_ate", HR_REG_CFG(QPC_EXT_ATE) },
> +		{ "fre", HR_REG_CFG(QPC_FRE) },
> +		{ "rmt_e2e", HR_REG_CFG(QPC_RMT_E2E) },
> +		{ "retry_num_init", HR_REG_CFG(QPC_RETRY_NUM_INIT) },
> +		{ "retry_cnt", HR_REG_CFG(QPC_RETRY_CNT) },
> +		{ "flush_idx", HR_REG_CFG(QPC_SQ_FLUSH_IDX) },
> +		{ "sq_max_idx", HR_REG_CFG(QPC_SQ_MAX_IDX) },
> +		{ "sq_tx_err", HR_REG_CFG(QPC_SQ_TX_ERR) },
> +		{ "sq_rx_err", HR_REG_CFG(QPC_SQ_RX_ERR) },
> +		{ "rq_rx_err", HR_REG_CFG(QPC_RQ_RX_ERR) },
> +		{ "rq_tx_err", HR_REG_CFG(QPC_RQ_TX_ERR) },
> +		{ "rq_cqeidx", HR_REG_CFG(QPC_RQ_CQE_IDX) },
> +		{ "rq_rty_tx_err", HR_REG_CFG(QPC_RQ_RTY_TX_ERR) },
> +	};

I have a feeling that this is abuse of our vendor specific attributes.
Why don't you use RDMA_NLDEV_ATTR_RES_RAW for such huge dumps?

Thanks