DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] rte_eth_dev_config_restore problem
@ 2016-10-26 17:33 Igor Ryzhov
  0 siblings, 0 replies; only message in thread
From: Igor Ryzhov @ 2016-10-26 17:33 UTC (permalink / raw)
  To: dev; +Cc: Thomas Monjalon

Hello everyone,

I think there is a bug in rte_eth_dev_config_restore function.
During restoration of MAC address configuration, all MAC addresses are
restored with mac_addr_add function, but as I think MAC address with index
0 shouldn't be restored in such way, because it is a default MAC address.

This problem can be solved in two ways:
1. Just call mac_addr_set instead of mac_addr_add for index 0.
2. Don't restore address with index 0 at all and let driver do it.

I think the second option is the right one, because:
1. Some drivers don't support mac_addr_set at all, it means that we must
not touch it.
2. Some drivers already support restoration of default MAC address. For
example, look at the ixgbe "ixgbe_init_rx_addrs_generic" function. It
restores default MAC address if it was overridden by user. All that we have
to do is to rewrite hw->mac.addr in mac_addr_set function.

Best regards,
Igor

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2016-10-26 17:33 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-26 17:33 [dpdk-dev] rte_eth_dev_config_restore problem Igor Ryzhov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).