Re: [dpdk-dev] [PATCH] app/procinfo: add device registers dump

[Stephen Hemminger]

On Sun, 25 Apr 2021 21:02:22 +0800
"Min Hu (Connor)" <humi...@huawei.com> wrote:

> +
> +             memset(&reg_info, 0, sizeof(reg_info));
> +             memset(&dev_info, 0, sizeof(dev_info));

This memset is redundant, rte_eth_dev_info_get already has the same memset.
For the registers, ethdev should be fixed to do it.

> +
> +             ret = rte_eth_dev_info_get(i, &dev_info);
> +             if (ret) {
> +                     printf("Error getting device info: %d\n", ret);
> +                     continue;
> +             }
> +
> +             ret = rte_eth_dev_get_reg_info(i, &reg_info);
> +             if (ret) {
> +                     printf("Error getting device reg info: %d\n", ret);
> +                     continue;
> +             }