[dpdk-dev] net/ixgbe: fix NIC 82599ES port check error

Message ID 1500535541-61376-1-git-send-email-wei.zhao1@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail Compilation issues

Commit Message

Zhao1, Wei July 20, 2017, 7:25 a.m. UTC
  This NIC type port check should use port index
not mask for 82599ES.

Fixes: 16f534e508d ("net/ixgbe: add support 82599ES SCTP packet drop action")

Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
---
 drivers/net/ixgbe/ixgbe_flow.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Comments

Ferruh Yigit July 20, 2017, 11:23 a.m. UTC | #1
On 7/20/2017 8:25 AM, Wei Zhao wrote:
> This NIC type port check should use port index
> not mask for 82599ES.
> 
> Fixes: 16f534e508d ("net/ixgbe: add support 82599ES SCTP packet drop action")
> 
> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>

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

Patch

diff --git a/drivers/net/ixgbe/ixgbe_flow.c b/drivers/net/ixgbe/ixgbe_flow.c
index 2bb5c46..c77bbe2 100644
--- a/drivers/net/ixgbe/ixgbe_flow.c
+++ b/drivers/net/ixgbe/ixgbe_flow.c
@@ -2546,8 +2546,8 @@  ixgbe_parse_fdir_filter(struct rte_eth_dev *dev,
 
 	if (hw->mac.type == ixgbe_mac_82599EB &&
 		rule->fdirflags == IXGBE_FDIRCMD_DROP &&
-		(rule->mask.src_port_mask != 0 ||
-		rule->mask.dst_port_mask != 0))
+		(rule->ixgbe_fdir.formatted.src_port != 0 ||
+		rule->ixgbe_fdir.formatted.dst_port != 0))
 		return -ENOTSUP;
 
 	if (fdir_mode == RTE_FDIR_MODE_NONE ||