<div dir="ltr"><div>I can remove this part (rte_eth_config), but the new API provides other link information which is not available from the existing API, like supported/advertising/partner_advertising link modes.</div><div>Additionally from what I see, rte_eth_confg is available from rte_eth_dev_conf_get API and it's copy of requested configuration:</div><div>`memcpy(dev_conf, &dev->data->dev_conf, sizeof(struct rte_eth_conf));` </div><div><br></div><div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 3, 2024 at 7:23 PM Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, 3 Apr 2024 16:59:08 +0200<br>
Marek Pazdan <<a href="mailto:mpazdan@arista.com" target="_blank">mpazdan@arista.com</a>> wrote:<br>
<br>
> Hi Thomas,<br>
> <br>
> Information like advertising: speed, pause, autonegotiation etc. and that<br>
> same for link partner advertising.<br>
> I try to mimic ethtool since it's a common, well known and widely used tool<br>
> so adding an API which provides the same set of information in case the<br>
> interface is passthrough to dpdk will be useful I think.<br>
> I've 2 more patches dependent on the one we are discussing now, but I need<br>
> patchwork id to add dependency information. I need moderator approval for<br>
> the current patch.<br>
> <br>
> Regards,<br>
> Marek<br>
<br>
DPDK already had this info under rte_eth_conf and it is supported<br>
on all drivers. Why introduce yet another API and only for some drivers?<br>
</blockquote></div>