* [dpdk-dev] [PATCH] net/ice: fix tunnel type get error @ 2020-07-15 2:07 Wei Zhao 2020-07-15 2:45 ` Lu, Nannan ` (2 more replies) 0 siblings, 3 replies; 6+ messages in thread From: Wei Zhao @ 2020-07-15 2:07 UTC (permalink / raw) To: dev; +Cc: qi.z.zhang, nannan.lu, Wei Zhao When get tunnel type for ICE_NON_TUN, there need some upodate for gtp related pctype. Fixes: 418d2563d10b ("net/ice/base: get tunnel type for recipe") Signed-off-by: Wei Zhao <wei.zhao1@intel.com> --- drivers/net/ice/base/ice_switch.c | 4 ++-- drivers/net/ice/base/ice_switch.h | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/ice/base/ice_switch.c b/drivers/net/ice/base/ice_switch.c index 296aa5484..ebf405f7a 100644 --- a/drivers/net/ice/base/ice_switch.c +++ b/drivers/net/ice/base/ice_switch.c @@ -1069,7 +1069,7 @@ static enum ice_sw_tunnel_type ice_get_tun_type_for_recipe(u8 rid) non_tun_valid = true; } - if (j >= ICE_PROFID_IPV4_GTPC_TEID && + if (j >= ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER && j <= ICE_PROFID_IPV6_GTPU_IPV6_OTHER) gtp_valid = true; @@ -1136,7 +1136,7 @@ static enum ice_sw_tunnel_type ice_get_tun_type_for_recipe(u8 rid) tun_type = ICE_SW_TUN_PPPOE_IPV6_TCP; break; case ICE_PROFID_PPPOE_IPV6_UDP: - tun_type = ICE_SW_TUN_PPPOE_IPV4_UDP; + tun_type = ICE_SW_TUN_PPPOE_IPV6_UDP; break; case ICE_PROFID_PPPOE_IPV6_OTHER: tun_type = ICE_SW_TUN_PPPOE_IPV6; diff --git a/drivers/net/ice/base/ice_switch.h b/drivers/net/ice/base/ice_switch.h index cc3d2702e..77c70d3b2 100644 --- a/drivers/net/ice/base/ice_switch.h +++ b/drivers/net/ice/base/ice_switch.h @@ -28,6 +28,7 @@ #define ICE_PROFID_PPPOE_IPV6_UDP 39 #define ICE_PROFID_PPPOE_IPV6_OTHER 40 #define ICE_PROFID_IPV4_GTPC_TEID 41 +#define ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER 47 #define ICE_PROFID_IPV6_GTPU_IPV6_OTHER 70 #define ICE_PROFID_IPV4_ESP 71 #define ICE_PROFID_IPV6_ESP 72 -- 2.19.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] net/ice: fix tunnel type get error 2020-07-15 2:07 [dpdk-dev] [PATCH] net/ice: fix tunnel type get error Wei Zhao @ 2020-07-15 2:45 ` Lu, Nannan 2020-07-15 14:02 ` Zhang, Qi Z 2020-07-16 1:42 ` [dpdk-dev] [PATCH v2] " Wei Zhao 2 siblings, 0 replies; 6+ messages in thread From: Lu, Nannan @ 2020-07-15 2:45 UTC (permalink / raw) To: Zhao1, Wei, dev; +Cc: Zhang, Qi Z > -----Original Message----- > From: Zhao1, Wei <wei.zhao1@intel.com> > Sent: Wednesday, July 15, 2020 10:08 AM > To: dev@dpdk.org > Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Lu, Nannan <nannan.lu@intel.com>; > Zhao1, Wei <wei.zhao1@intel.com> > Subject: [PATCH] net/ice: fix tunnel type get error > > When get tunnel type for ICE_NON_TUN, there need some upodate for gtp > related pctype. > > Fixes: 418d2563d10b ("net/ice/base: get tunnel type for recipe") > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com> > --- > drivers/net/ice/base/ice_switch.c | 4 ++-- drivers/net/ice/base/ice_switch.h | > 1 + > 2 files changed, 3 insertions(+), 2 deletions(-) > Tested-by: Nannan Lu <nannan.lu@intel.com> > -- > 2.19.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] net/ice: fix tunnel type get error 2020-07-15 2:07 [dpdk-dev] [PATCH] net/ice: fix tunnel type get error Wei Zhao 2020-07-15 2:45 ` Lu, Nannan @ 2020-07-15 14:02 ` Zhang, Qi Z 2020-07-16 1:49 ` Zhao1, Wei 2020-07-16 1:42 ` [dpdk-dev] [PATCH v2] " Wei Zhao 2 siblings, 1 reply; 6+ messages in thread From: Zhang, Qi Z @ 2020-07-15 14:02 UTC (permalink / raw) To: Zhao1, Wei, dev; +Cc: Lu, Nannan > -----Original Message----- > From: Zhao1, Wei <wei.zhao1@intel.com> > Sent: Wednesday, July 15, 2020 10:08 AM > To: dev@dpdk.org > Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Lu, Nannan <nannan.lu@intel.com>; > Zhao1, Wei <wei.zhao1@intel.com> > Subject: [PATCH] net/ice: fix tunnel type get error > > When get tunnel type for ICE_NON_TUN, there need some upodate for gtp > related pctype. please reword the commit log, what do you mean "get tunnel type for ICE_NON_TUN"? also we don't have pctype in ice, please use profile. > > Fixes: 418d2563d10b ("net/ice/base: get tunnel type for recipe") > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com> > --- > drivers/net/ice/base/ice_switch.c | 4 ++-- drivers/net/ice/base/ice_switch.h > | 1 + > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/ice/base/ice_switch.c > b/drivers/net/ice/base/ice_switch.c > index 296aa5484..ebf405f7a 100644 > --- a/drivers/net/ice/base/ice_switch.c > +++ b/drivers/net/ice/base/ice_switch.c > @@ -1069,7 +1069,7 @@ static enum ice_sw_tunnel_type > ice_get_tun_type_for_recipe(u8 rid) > non_tun_valid = true; > } > > - if (j >= ICE_PROFID_IPV4_GTPC_TEID && > + if (j >= ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER && > j <= ICE_PROFID_IPV6_GTPU_IPV6_OTHER) > gtp_valid = true; > > @@ -1136,7 +1136,7 @@ static enum ice_sw_tunnel_type > ice_get_tun_type_for_recipe(u8 rid) > tun_type = ICE_SW_TUN_PPPOE_IPV6_TCP; > break; > case ICE_PROFID_PPPOE_IPV6_UDP: > - tun_type = ICE_SW_TUN_PPPOE_IPV4_UDP; > + tun_type = ICE_SW_TUN_PPPOE_IPV6_UDP; > break; > case ICE_PROFID_PPPOE_IPV6_OTHER: > tun_type = ICE_SW_TUN_PPPOE_IPV6; > diff --git a/drivers/net/ice/base/ice_switch.h > b/drivers/net/ice/base/ice_switch.h > index cc3d2702e..77c70d3b2 100644 > --- a/drivers/net/ice/base/ice_switch.h > +++ b/drivers/net/ice/base/ice_switch.h > @@ -28,6 +28,7 @@ > #define ICE_PROFID_PPPOE_IPV6_UDP 39 > #define ICE_PROFID_PPPOE_IPV6_OTHER 40 > #define ICE_PROFID_IPV4_GTPC_TEID 41 > +#define ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER 47 > #define ICE_PROFID_IPV6_GTPU_IPV6_OTHER 70 > #define ICE_PROFID_IPV4_ESP 71 > #define ICE_PROFID_IPV6_ESP 72 > -- > 2.19.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] net/ice: fix tunnel type get error 2020-07-15 14:02 ` Zhang, Qi Z @ 2020-07-16 1:49 ` Zhao1, Wei 0 siblings, 0 replies; 6+ messages in thread From: Zhao1, Wei @ 2020-07-16 1:49 UTC (permalink / raw) To: Zhang, Qi Z, dev; +Cc: Lu, Nannan H, qi > -----Original Message----- > From: Zhang, Qi Z <qi.z.zhang@intel.com> > Sent: Wednesday, July 15, 2020 10:02 PM > To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org > Cc: Lu, Nannan <nannan.lu@intel.com> > Subject: RE: [PATCH] net/ice: fix tunnel type get error > > > > > -----Original Message----- > > From: Zhao1, Wei <wei.zhao1@intel.com> > > Sent: Wednesday, July 15, 2020 10:08 AM > > To: dev@dpdk.org > > Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Lu, Nannan > > <nannan.lu@intel.com>; Zhao1, Wei <wei.zhao1@intel.com> > > Subject: [PATCH] net/ice: fix tunnel type get error > > > > When get tunnel type for ICE_NON_TUN, there need some upodate for gtp > > related pctype. > > please reword the commit log, what do you mean "get tunnel type for > ICE_NON_TUN"? > also we don't have pctype in ice, please use profile. Yes, ice use ptype. > > > > Fixes: 418d2563d10b ("net/ice/base: get tunnel type for recipe") > > > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com> > > --- > > drivers/net/ice/base/ice_switch.c | 4 ++-- > > drivers/net/ice/base/ice_switch.h > > | 1 + > > 2 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/net/ice/base/ice_switch.c > > b/drivers/net/ice/base/ice_switch.c > > index 296aa5484..ebf405f7a 100644 > > --- a/drivers/net/ice/base/ice_switch.c > > +++ b/drivers/net/ice/base/ice_switch.c > > @@ -1069,7 +1069,7 @@ static enum ice_sw_tunnel_type > > ice_get_tun_type_for_recipe(u8 rid) > > non_tun_valid = true; > > } > > > > - if (j >= ICE_PROFID_IPV4_GTPC_TEID && > > + if (j >= ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER && > > j <= ICE_PROFID_IPV6_GTPU_IPV6_OTHER) > > gtp_valid = true; > > > > @@ -1136,7 +1136,7 @@ static enum ice_sw_tunnel_type > > ice_get_tun_type_for_recipe(u8 rid) > > tun_type = ICE_SW_TUN_PPPOE_IPV6_TCP; > > break; > > case ICE_PROFID_PPPOE_IPV6_UDP: > > - tun_type = ICE_SW_TUN_PPPOE_IPV4_UDP; > > + tun_type = ICE_SW_TUN_PPPOE_IPV6_UDP; > > break; > > case ICE_PROFID_PPPOE_IPV6_OTHER: > > tun_type = ICE_SW_TUN_PPPOE_IPV6; diff --git > > a/drivers/net/ice/base/ice_switch.h > > b/drivers/net/ice/base/ice_switch.h > > index cc3d2702e..77c70d3b2 100644 > > --- a/drivers/net/ice/base/ice_switch.h > > +++ b/drivers/net/ice/base/ice_switch.h > > @@ -28,6 +28,7 @@ > > #define ICE_PROFID_PPPOE_IPV6_UDP 39 > > #define ICE_PROFID_PPPOE_IPV6_OTHER 40 > > #define ICE_PROFID_IPV4_GTPC_TEID 41 > > +#define ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER 47 > > #define ICE_PROFID_IPV6_GTPU_IPV6_OTHER 70 > > #define ICE_PROFID_IPV4_ESP 71 > > #define ICE_PROFID_IPV6_ESP 72 > > -- > > 2.19.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-dev] [PATCH v2] net/ice: fix tunnel type get error 2020-07-15 2:07 [dpdk-dev] [PATCH] net/ice: fix tunnel type get error Wei Zhao 2020-07-15 2:45 ` Lu, Nannan 2020-07-15 14:02 ` Zhang, Qi Z @ 2020-07-16 1:42 ` Wei Zhao 2020-07-16 3:26 ` Zhang, Qi Z 2 siblings, 1 reply; 6+ messages in thread From: Wei Zhao @ 2020-07-16 1:42 UTC (permalink / raw) To: dev; +Cc: qi.z.zhang, Wei Zhao When function ice_get_tun_type_for_recipe() get tunnel type, for ICE_NON_TUN we need to include gtp-c and some gtp-u ptype with no payload, as they do not have tunnel packet as paylod. Fixes: 418d2563d10b ("net/ice/base: get tunnel type for recipe") Signed-off-by: Wei Zhao <wei.zhao1@intel.com> Tested-by: Nannan Lu <nannan.lu@intel.com> --- drivers/net/ice/base/ice_switch.c | 4 ++-- drivers/net/ice/base/ice_switch.h | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/ice/base/ice_switch.c b/drivers/net/ice/base/ice_switch.c index 296aa5484..ebf405f7a 100644 --- a/drivers/net/ice/base/ice_switch.c +++ b/drivers/net/ice/base/ice_switch.c @@ -1069,7 +1069,7 @@ static enum ice_sw_tunnel_type ice_get_tun_type_for_recipe(u8 rid) non_tun_valid = true; } - if (j >= ICE_PROFID_IPV4_GTPC_TEID && + if (j >= ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER && j <= ICE_PROFID_IPV6_GTPU_IPV6_OTHER) gtp_valid = true; @@ -1136,7 +1136,7 @@ static enum ice_sw_tunnel_type ice_get_tun_type_for_recipe(u8 rid) tun_type = ICE_SW_TUN_PPPOE_IPV6_TCP; break; case ICE_PROFID_PPPOE_IPV6_UDP: - tun_type = ICE_SW_TUN_PPPOE_IPV4_UDP; + tun_type = ICE_SW_TUN_PPPOE_IPV6_UDP; break; case ICE_PROFID_PPPOE_IPV6_OTHER: tun_type = ICE_SW_TUN_PPPOE_IPV6; diff --git a/drivers/net/ice/base/ice_switch.h b/drivers/net/ice/base/ice_switch.h index cc3d2702e..77c70d3b2 100644 --- a/drivers/net/ice/base/ice_switch.h +++ b/drivers/net/ice/base/ice_switch.h @@ -28,6 +28,7 @@ #define ICE_PROFID_PPPOE_IPV6_UDP 39 #define ICE_PROFID_PPPOE_IPV6_OTHER 40 #define ICE_PROFID_IPV4_GTPC_TEID 41 +#define ICE_PROFID_IPV4_GTPU_EH_IPV4_OTHER 47 #define ICE_PROFID_IPV6_GTPU_IPV6_OTHER 70 #define ICE_PROFID_IPV4_ESP 71 #define ICE_PROFID_IPV6_ESP 72 -- 2.19.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH v2] net/ice: fix tunnel type get error 2020-07-16 1:42 ` [dpdk-dev] [PATCH v2] " Wei Zhao @ 2020-07-16 3:26 ` Zhang, Qi Z 0 siblings, 0 replies; 6+ messages in thread From: Zhang, Qi Z @ 2020-07-16 3:26 UTC (permalink / raw) To: Zhao1, Wei, dev > -----Original Message----- > From: Zhao1, Wei <wei.zhao1@intel.com> > Sent: Thursday, July 16, 2020 9:42 AM > To: dev@dpdk.org > Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Zhao1, Wei <wei.zhao1@intel.com> > Subject: [PATCH v2] net/ice: fix tunnel type get error > > When function ice_get_tun_type_for_recipe() get tunnel type, for > ICE_NON_TUN we need to include gtp-c and some gtp-u ptype with no payload, > as they do not have tunnel packet as paylod. > > Fixes: 418d2563d10b ("net/ice/base: get tunnel type for recipe") > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com> > Tested-by: Nannan Lu <nannan.lu@intel.com> Acked-by: Qi Zhang <qi.z.zhang@intel.com> Applied to dpdk-next-net-intel. Thanks Qi ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-07-16 3:27 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-07-15 2:07 [dpdk-dev] [PATCH] net/ice: fix tunnel type get error Wei Zhao 2020-07-15 2:45 ` Lu, Nannan 2020-07-15 14:02 ` Zhang, Qi Z 2020-07-16 1:49 ` Zhao1, Wei 2020-07-16 1:42 ` [dpdk-dev] [PATCH v2] " Wei Zhao 2020-07-16 3:26 ` Zhang, Qi Z
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).