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@xxxxxxxxxx>
> ---
> 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
