If the register values of multiple modules are obtained at a time,
the register values are incorrect because the data field to be filled
pointing to a wrong address. Update the pointer offset of data to
get the correct address.
Fixes: dd4b8bba785f ("net/hns3: support reporting names of registers")
Cc: sta...@dpdk.org

Signed-off-by: Jie Hai <haij...@huawei.com>
---
 drivers/net/hns3/hns3_regs.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/hns3/hns3_regs.c b/drivers/net/hns3/hns3_regs.c
index be6929738942..fef904a296c0 100644
--- a/drivers/net/hns3/hns3_regs.c
+++ b/drivers/net/hns3/hns3_regs.c
@@ -1274,6 +1274,7 @@ hns3_get_dfx_regs(struct hns3_hw *hw, struct 
rte_dev_reg_info *regs, uint32_t mo
        if (cmd_descs == NULL)
                return -ENOMEM;
 
+       data += regs->length;
        for (i = 0; i < opcode_num; i++) {
                opcode = hns3_dfx_reg_opcode_list[i];
                bd_num = bd_num_list[i];
@@ -1285,7 +1286,6 @@ hns3_get_dfx_regs(struct hns3_hw *hw, struct 
rte_dev_reg_info *regs, uint32_t mo
                if (ret)
                        break;
 
-               data += regs->length;
                regs_num = hns3_dfx_reg_fetch_data(cmd_descs, bd_num, data);
                if (regs_num !=  hns3_reg_lists[i].entry_num) {
                        hns3_err(hw, "Query register number differ from the 
list for module %s!",
@@ -1294,6 +1294,7 @@ hns3_get_dfx_regs(struct hns3_hw *hw, struct 
rte_dev_reg_info *regs, uint32_t mo
                }
                hns3_fill_dfx_regs_name(hw, regs, hns3_reg_lists[i].reg_list, 
regs_num);
                regs->length += regs_num;
+               data += regs_num;
        }
        rte_free(cmd_descs);
 
-- 
2.22.0

Reply via email to