On 6/1/22 09:39, Min Hu (Connor) wrote:
The index zero of rte_eth_dev_data::mac_addrs array is as the default MAC
index, and other indexes can't be the same as the address corresponding to
index 0. If we break it, may cause following problems:
1) waste of MAC address spaces.
2) a fake MAC address in the MAC list, isn't in hardware MAC entries.
3) a MAC address is assigned to diffent pool.

Huisong Li (2):
   ethdev: fix one address occupies two indexes in MAC addrs
   ethdev: document default and non-default MAC address
---
v4:
   - fix broken in the patchwork

v3:
   - first explicitly remove the non-default MAC, then set default one.
   - document default and non-default MAC address

v2:
   - fixed commit log.

  lib/ethdev/ethdev_driver.h |  7 ++++++-
  lib/ethdev/rte_ethdev.c    | 39 ++++++++++++++++++++++++++++++++++++--
  2 files changed, 43 insertions(+), 3 deletions(-)


Please, don't forget to add maintainers in Cc when you send patches.
  --cc-cmd ./devtools/get-maintainers.sh
or
  --to-cmd ./devtools/get-maintainers.sh
(I'd prefer To and have dev@dpdk.org in Cc)

Don't worry about it right now. I'll add Thomas and Ferrh in my replies.

Reply via email to