From: David Harton <dharton@cpp-rtpbld-31.cpprtplab>
To: beilei.xing@intel.com, jingjing.wu@intel.com
Cc: dev@dpdk.org, David Harton <dharton@cisco.com>
Subject: [PATCH v3] i40e: fix i40evf_add_mac_addr to permit multicast addresses
Date: Tue, 12 Sep 2017 23:21:10 -0400 [thread overview]
Message-ID: <20170913032110.39994-1-dharton@cpp-rtpbld-31.cpprtplab> (raw)
In-Reply-To: <20170912130210.18288-1-dharton@cpp-rtpbld-31.cpprtplab>
From: David Harton <dharton@cisco.com>
The i40e maintains a single MAC filter table for both
unicast and multicast addresses. The i40e_validate_mac_addr
function was preventing multicast addresses from being added
to the table via i40evf_add_mac_addr. Fixed the issue by
adjusting the check in i40evf_add_mac_addr.
Fixes: 4861cde46116 ("i40e: new poll mode driver")
Fixes: 97ac72aa71a9 ("i40e: support setting VF MAC address")
Signed-off-by: David Harton <dharton@cisco.com>
---
v3
* Modified diff to use more conventional logic.
v2
* Removed multicast check in i40evf_add_mac_addr.
v1
* Removed multicast check in i40e_validate_mac_addr.
drivers/net/i40e/i40e_ethdev_vf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/i40e/i40e_ethdev_vf.c b/drivers/net/i40e/i40e_ethdev_vf.c
index f6d8293..08afd9e 100644
--- a/drivers/net/i40e/i40e_ethdev_vf.c
+++ b/drivers/net/i40e/i40e_ethdev_vf.c
@@ -888,7 +888,7 @@ i40evf_add_mac_addr(struct rte_eth_dev *dev,
int err;
struct vf_cmd_info args;
- if (i40e_validate_mac_addr(addr->addr_bytes) != I40E_SUCCESS) {
+ if (is_zero_ether_addr(addr)) {
PMD_DRV_LOG(ERR, "Invalid mac:%x:%x:%x:%x:%x:%x",
addr->addr_bytes[0], addr->addr_bytes[1],
addr->addr_bytes[2], addr->addr_bytes[3],
--
2.10.3.dirty
next prev parent reply other threads:[~2017-09-13 3:21 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-22 22:21 [PATCH] i40e: fix i40e_validate_mac_addr to permit multicast addresses David Harton
2017-08-31 15:53 ` Ferruh Yigit
2017-08-31 16:04 ` David Harton (dharton)
2017-09-08 12:51 ` David Harton (dharton)
2017-09-08 12:56 ` Ferruh Yigit
2017-09-11 5:41 ` Xing, Beilei
2017-09-11 17:22 ` David Harton (dharton)
2017-09-12 3:00 ` Xing, Beilei
2017-09-12 13:02 ` [PATCH v2] i40e: fix i40evf_add_mac_addr " David Harton
2017-09-13 2:20 ` Xing, Beilei
2017-09-13 2:38 ` David Harton (dharton)
2017-09-13 2:43 ` Xing, Beilei
2017-09-13 3:21 ` David Harton [this message]
2017-09-13 3:42 ` [PATCH v3] " Xing, Beilei
2017-09-14 18:55 ` Ferruh Yigit
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170913032110.39994-1-dharton@cpp-rtpbld-31.cpprtplab \
--to=dharton@cpp-rtpbld-31.cpprtplab \
--cc=beilei.xing@intel.com \
--cc=dev@dpdk.org \
--cc=dharton@cisco.com \
--cc=jingjing.wu@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.