19/02/2025 19:45, Stephen Hemminger:
> On Tue, 18 Feb 2025 12:58:28 +0100
> Thierry Herbelot <thierry.herbe...@6wind.com> wrote:
>
> > 'width' and 'offset' are input parameters when dumping the register
> > info of an Ethernet device. They should be copied in the new request
> > before calling the device callback function.
> >
> > Fixes: 083db2ed9e9 ('ethdev: add report of register names and filter')
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: Thierry Herbelot <thierry.herbe...@6wind.com>
>
> Why does the ethdev code create an on stack temporary variable.
> Looks like it only wants to make sure that names element is NULL.
>
> Really should be one function and when extended fields were added
> should have used API versioning.
> Probably too late now, although rte_eth_dev_get_reg_info_ext()
> is an experimental API.
If it is experimental, the function can be dropped in favour of a better
versioned function.
