[dpdk-dev] [dpdk-stable] [PATCH] net/sfc: fix main MAC address handling

Ferruh Yigit ferruh.yigit at intel.com
Thu Dec 21 20:30:38 CET 2017


On 12/20/2017 1:52 AM, Andrew Rybchenko wrote:
> From: Ivan Malov <ivan.malov at oktetlabs.ru>
> 
> There is a school of thought that rte_eth_dev_default_mac_addr_set()
> must call the PMD callback first and then save the new MAC address
> in dev->data->mac_addrs[0]. 

:)

> If this concept gets approved, it will
> break the current approach used in sfc driver as the latter relies
> on the assumption that the new MAC address is already contained in
> dev->data->mac_addrs[0], and, if adapter restart is needed to make
> the HW apply the new address, the outdated value will be retrieved
> from dev->data. In order to preclude any possible bugs, this patch
> adds device private storage for the up-to-date copy of the address.
Thank you for taking care of this.

> 
> Fixes: c100fd464bb7 ("net/sfc: support main MAC address change")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Ivan Malov <ivan.malov at oktetlabs.ru>
> Signed-off-by: Andrew Rybchenko <arybchenko at solarflare.com>

Applied to dpdk-next-net/master, thanks.


More information about the dev mailing list