From: "Zhang, Qi Z" <qi.z.zhang@intel.com>
To: "Cao, Yahui" <yahui.cao@intel.com>,
"Yang, Qiming" <qiming.yang@intel.com>,
"Lu, Wenzhuo" <wenzhuo.lu@intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>,
"Ye, Xiaolong" <xiaolong.ye@intel.com>,
"Wang, Ying A" <ying.a.wang@intel.com>,
"Su, Simei" <simei.su@intel.com>,
"Sun, Chenmin" <chenmin.sun@intel.com>
Subject: Re: [dpdk-dev] [PATCH v2 1/2] net/ice: fix GTPU extension header renaming
Date: Wed, 20 Nov 2019 09:17:28 +0000 [thread overview]
Message-ID: <039ED4275CED7440929022BC67E7061153DCDD46@SHSMSX105.ccr.corp.intel.com> (raw)
In-Reply-To: <20191120150549.54533-2-yahui.cao@intel.com>
> -----Original Message-----
> From: Cao, Yahui <yahui.cao@intel.com>
> Sent: Wednesday, November 20, 2019 11:06 PM
> To: Yang, Qiming <qiming.yang@intel.com>; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>
> Cc: dev@dpdk.org; Zhang, Qi Z <qi.z.zhang@intel.com>; Cao, Yahui
> <yahui.cao@intel.com>; Ye, Xiaolong <xiaolong.ye@intel.com>; Wang, Ying A
> <ying.a.wang@intel.com>; Su, Simei <simei.su@intel.com>; Sun, Chenmin
> <chenmin.sun@intel.com>
> Subject: [PATCH v2 1/2] net/ice: fix GTPU extension header renaming
How about "fix pattern name of GTPU with extension header"?
>
> GTP-U packet with extension header is GTP-U packet with GTP_PSC header
> inserted between GTP-U header and inner payload.
Pattern name of GTP-U flow with extension header should be xxx_gtp_eh_xxx
so it will not conflict with the pattern name of a GTP-U flow that does not contains extension header
>
> Fixes: 3e4eab9c2192 ("net/ice: add pattern manifest")
> Cc: ying.a.wang@intel.com
>
> Signed-off-by: Yahui Cao <yahui.cao@intel.com>
> ---
> drivers/net/ice/ice_fdir_filter.c | 5 +++--
> drivers/net/ice/ice_generic_flow.c | 8 ++++----
> drivers/net/ice/ice_generic_flow.h | 8 ++++----
> drivers/net/ice/ice_hash.c | 32 +++++++++++++++---------------
> 4 files changed, 27 insertions(+), 26 deletions(-)
>
> diff --git a/drivers/net/ice/ice_fdir_filter.c b/drivers/net/ice/ice_fdir_filter.c
> index 9e251cee5..dc7a97acb 100644
> --- a/drivers/net/ice/ice_fdir_filter.c
> +++ b/drivers/net/ice/ice_fdir_filter.c
> @@ -67,7 +67,7 @@
> ICE_FDIR_INSET_VXLAN_IPV4 | \
> ICE_INSET_TUN_SCTP_SRC_PORT | ICE_INSET_TUN_SCTP_DST_PORT)
>
> -#define ICE_FDIR_INSET_GTPU_IPV4 (\
> +#define ICE_FDIR_INSET_GTPU_EH_IPV4 (\
> ICE_INSET_GTPU_TEID | ICE_INSET_GTPU_QFI)
>
> static struct ice_pattern_match_item ice_fdir_pattern_os[] = { @@ -122,7
> +122,8 @@ static struct ice_pattern_match_item ice_fdir_pattern_comms[] =
> {
> ICE_FDIR_INSET_VXLAN_IPV4_TCP,
> ICE_INSET_NONE},
> {pattern_eth_ipv4_udp_vxlan_eth_ipv4_sctp,
> ICE_FDIR_INSET_VXLAN_IPV4_SCTP,
> ICE_INSET_NONE},
> - {pattern_eth_ipv4_gtpu_ipv4, ICE_FDIR_INSET_GTPU_IPV4,
> ICE_INSET_NONE},
> + {pattern_eth_ipv4_gtpu_eh_ipv4,
> + ICE_FDIR_INSET_GTPU_EH_IPV4,
> ICE_INSET_NONE},
> };
>
> static struct ice_flow_parser ice_fdir_parser_os; diff --git
> a/drivers/net/ice/ice_generic_flow.c b/drivers/net/ice/ice_generic_flow.c
> index 5594f8555..a235337ba 100644
> --- a/drivers/net/ice/ice_generic_flow.c
> +++ b/drivers/net/ice/ice_generic_flow.c
> @@ -1034,7 +1034,7 @@ enum rte_flow_item_type
> pattern_eth_ipv6_nvgre_eth_ipv6_icmp6[] = { };
>
> /* GTPU */
> -enum rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4[] = {
> +enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4[] = {
> RTE_FLOW_ITEM_TYPE_ETH,
> RTE_FLOW_ITEM_TYPE_IPV4,
> RTE_FLOW_ITEM_TYPE_UDP,
> @@ -1043,7 +1043,7 @@ enum rte_flow_item_type
> pattern_eth_ipv4_gtpu_ipv4[] = {
> RTE_FLOW_ITEM_TYPE_IPV4,
> RTE_FLOW_ITEM_TYPE_END,
> };
> -enum rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4_udp[] = {
> +enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4_udp[] = {
> RTE_FLOW_ITEM_TYPE_ETH,
> RTE_FLOW_ITEM_TYPE_IPV4,
> RTE_FLOW_ITEM_TYPE_UDP,
> @@ -1053,7 +1053,7 @@ enum rte_flow_item_type
> pattern_eth_ipv4_gtpu_ipv4_udp[] = {
> RTE_FLOW_ITEM_TYPE_UDP,
> RTE_FLOW_ITEM_TYPE_END,
> };
> -enum rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4_tcp[] = {
> +enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4_tcp[] = {
> RTE_FLOW_ITEM_TYPE_ETH,
> RTE_FLOW_ITEM_TYPE_IPV4,
> RTE_FLOW_ITEM_TYPE_UDP,
> @@ -1064,7 +1064,7 @@ enum rte_flow_item_type
> pattern_eth_ipv4_gtpu_ipv4_tcp[] = {
> RTE_FLOW_ITEM_TYPE_END,
>
> };
> -enum rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4_icmp[] = {
> +enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4_icmp[] = {
> RTE_FLOW_ITEM_TYPE_ETH,
> RTE_FLOW_ITEM_TYPE_IPV4,
> RTE_FLOW_ITEM_TYPE_UDP,
> diff --git a/drivers/net/ice/ice_generic_flow.h
> b/drivers/net/ice/ice_generic_flow.h
> index b6e4ed0e1..a598ceabc 100644
> --- a/drivers/net/ice/ice_generic_flow.h
> +++ b/drivers/net/ice/ice_generic_flow.h
> @@ -336,10 +336,10 @@ extern enum rte_flow_item_type
> pattern_eth_ipv6_nvgre_eth_ipv6_sctp[];
> extern enum rte_flow_item_type pattern_eth_ipv6_nvgre_eth_ipv6_icmp6[];
>
> /* GTPU */
> -extern enum rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4[]; -extern enum
> rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4_udp[]; -extern enum
> rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4_tcp[]; -extern enum
> rte_flow_item_type pattern_eth_ipv4_gtpu_ipv4_icmp[];
> +extern enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4[]; extern
> +enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4_udp[];
> +extern enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4_tcp[];
> +extern enum rte_flow_item_type pattern_eth_ipv4_gtpu_eh_ipv4_icmp[];
>
> /* PPPoE */
> extern enum rte_flow_item_type pattern_eth_pppoed[]; diff --git
> a/drivers/net/ice/ice_hash.c b/drivers/net/ice/ice_hash.c index
> 2e9c1bc67..b145a3f0d 100644
> --- a/drivers/net/ice/ice_hash.c
> +++ b/drivers/net/ice/ice_hash.c
> @@ -120,22 +120,22 @@ static struct ice_pattern_match_item
> ice_hash_pattern_list_os[] = {
>
> /* Supported pattern for comms package. */ static struct
> ice_pattern_match_item ice_hash_pattern_list_comms[] = {
> - {pattern_eth_ipv4, ICE_INSET_NONE, &hint_1},
> - {pattern_eth_ipv4_udp, ICE_INSET_NONE, &hint_2},
> - {pattern_eth_ipv4_tcp, ICE_INSET_NONE, &hint_3},
> - {pattern_eth_ipv4_sctp, ICE_INSET_NONE, &hint_4},
> - {pattern_eth_ipv6, ICE_INSET_NONE, &hint_5},
> - {pattern_eth_ipv6_udp, ICE_INSET_NONE, &hint_6},
> - {pattern_eth_ipv6_tcp, ICE_INSET_NONE, &hint_7},
> - {pattern_eth_ipv6_sctp, ICE_INSET_NONE, &hint_8},
> - {pattern_empty, ICE_INSET_NONE, &hint_0},
> - {pattern_eth_ipv4_gtpu_ipv4, ICE_INSET_NONE, &hint_9},
> - {pattern_eth_ipv4_gtpu_ipv4_udp, ICE_INSET_NONE, &hint_9},
> - {pattern_eth_ipv4_gtpu_ipv4_tcp, ICE_INSET_NONE, &hint_9},
> - {pattern_eth_pppoes_ipv4, ICE_INSET_NONE, &hint_10},
> - {pattern_eth_pppoes_ipv4_udp, ICE_INSET_NONE, &hint_11},
> - {pattern_eth_pppoes_ipv4_tcp, ICE_INSET_NONE, &hint_12},
> - {pattern_eth_pppoes_ipv4_sctp, ICE_INSET_NONE, &hint_13},
> + {pattern_eth_ipv4, ICE_INSET_NONE, &hint_1},
> + {pattern_eth_ipv4_udp, ICE_INSET_NONE, &hint_2},
> + {pattern_eth_ipv4_tcp, ICE_INSET_NONE, &hint_3},
> + {pattern_eth_ipv4_sctp, ICE_INSET_NONE, &hint_4},
> + {pattern_eth_ipv6, ICE_INSET_NONE, &hint_5},
> + {pattern_eth_ipv6_udp, ICE_INSET_NONE, &hint_6},
> + {pattern_eth_ipv6_tcp, ICE_INSET_NONE, &hint_7},
> + {pattern_eth_ipv6_sctp, ICE_INSET_NONE, &hint_8},
> + {pattern_empty, ICE_INSET_NONE, &hint_0},
> + {pattern_eth_ipv4_gtpu_eh_ipv4, ICE_INSET_NONE, &hint_9},
> + {pattern_eth_ipv4_gtpu_eh_ipv4_udp, ICE_INSET_NONE, &hint_9},
> + {pattern_eth_ipv4_gtpu_eh_ipv4_tcp, ICE_INSET_NONE, &hint_9},
> + {pattern_eth_pppoes_ipv4, ICE_INSET_NONE, &hint_10},
> + {pattern_eth_pppoes_ipv4_udp, ICE_INSET_NONE, &hint_11},
> + {pattern_eth_pppoes_ipv4_tcp, ICE_INSET_NONE, &hint_12},
> + {pattern_eth_pppoes_ipv4_sctp, ICE_INSET_NONE, &hint_13},
> };
>
> /**
> --
> 2.17.1
next prev parent reply other threads:[~2019-11-20 9:17 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-20 13:28 [dpdk-dev] [PATCH 0/2] fix FDIR support for non-extension-header GTPU Yahui Cao
2019-11-20 13:28 ` [dpdk-dev] [PATCH 1/2] net/ice: fix GTPU and GTPU extension header mix Yahui Cao
2019-11-20 13:28 ` [dpdk-dev] [PATCH 2/2] net/ice: fix FDIR support for GTPU without extension header Yahui Cao
2019-11-20 15:05 ` [dpdk-dev] [PATCH v2 0/2] fix FDIR support for GTPU Yahui Cao
2019-11-20 15:05 ` [dpdk-dev] [PATCH v2 1/2] net/ice: fix GTPU extension header renaming Yahui Cao
2019-11-20 9:17 ` Zhang, Qi Z [this message]
2019-11-20 15:05 ` [dpdk-dev] [PATCH v2 2/2] net/ice: fix FDIR support for GTPU without extension header Yahui Cao
2019-11-20 9:18 ` Zhang, Qi Z
2019-11-21 11:20 ` [dpdk-dev] [PATCH v3 0/2] fix FDIR support for GTPU Yahui Cao
2019-11-21 3:58 ` Zhang, Qi Z
2019-11-21 5:27 ` Ye Xiaolong
2019-11-21 11:20 ` [dpdk-dev] [PATCH v3 1/2] net/ice: fix pattern name of GTPU with extension header Yahui Cao
2019-11-21 11:20 ` [dpdk-dev] [PATCH v3 2/2] net/ice: add missing FDIR support for GTPU without " Yahui Cao
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=039ED4275CED7440929022BC67E7061153DCDD46@SHSMSX105.ccr.corp.intel.com \
--to=qi.z.zhang@intel.com \
--cc=chenmin.sun@intel.com \
--cc=dev@dpdk.org \
--cc=qiming.yang@intel.com \
--cc=simei.su@intel.com \
--cc=wenzhuo.lu@intel.com \
--cc=xiaolong.ye@intel.com \
--cc=yahui.cao@intel.com \
--cc=ying.a.wang@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).