From patchwork Fri Feb 10 01:37:58 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Zhao1, Wei" X-Patchwork-Id: 20357 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 493E92A5E; Fri, 10 Feb 2017 02:43:51 +0100 (CET) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id D79A63B5 for ; Fri, 10 Feb 2017 02:43:48 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga101.fm.intel.com with ESMTP; 09 Feb 2017 17:43:47 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.35,138,1484035200"; d="scan'208";a="42301066" Received: from dpdk1.bj.intel.com ([172.16.182.84]) by orsmga002.jf.intel.com with ESMTP; 09 Feb 2017 17:43:46 -0800 From: Wei Zhao To: dev@dpdk.org Cc: Wei Zhao Date: Fri, 10 Feb 2017 09:37:58 +0800 Message-Id: <1486690678-45460-1-git-send-email-wei.zhao1@intel.com> X-Mailer: git-send-email 2.5.5 Subject: [dpdk-dev] [PATCH] net/ixgbe: fix tci mask check in fdir pasrer X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" It must use big endian when check on the tci mask of vlan and vxlan parser in fdir filter rule pattern parser.Because rte layer send out tci mask using big endian mode. Fixes: 11777435c727 ("net/ixgbe: parse flow director filter") Fixes: cc83320af286 ("net/ixgbe: add tci mask check in fdir parser") Signed-off-by: Wei Zhao --- drivers/net/ixgbe/ixgbe_flow.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ixgbe/ixgbe_flow.c b/drivers/net/ixgbe/ixgbe_flow.c index d64cfd1..5a634d3 100644 --- a/drivers/net/ixgbe/ixgbe_flow.c +++ b/drivers/net/ixgbe/ixgbe_flow.c @@ -1617,7 +1617,7 @@ ixgbe_parse_fdir_filter_normal(const struct rte_flow_attr *attr, return -rte_errno; } rule->mask.vlan_tci_mask = vlan_mask->tci; - rule->mask.vlan_tci_mask &= 0xEFFF; + rule->mask.vlan_tci_mask &= rte_cpu_to_be_16(0xEFFF); /* More than one tags are not supported. */ /** @@ -2361,7 +2361,7 @@ ixgbe_parse_fdir_filter_tunnel(const struct rte_flow_attr *attr, return -rte_errno; } rule->mask.vlan_tci_mask = vlan_mask->tci; - rule->mask.vlan_tci_mask &= 0xEFFF; + rule->mask.vlan_tci_mask &= rte_cpu_to_be_16(0xEFFF); /* More than one tags are not supported. */ /**