[dpdk-dev] [PATCH v4 03/11] net/mlx5: support L3 VXLAN flow

[Xueming(Steven) Li]

> -----Original Message-----
> From: Nélio Laranjeiro <nelio.laranjeiro at 6wind.com>
> Sent: Wednesday, April 18, 2018 11:09 PM
> To: Xueming(Steven) Li <xuemingl at mellanox.com>
> Cc: Shahaf Shuler <shahafs at mellanox.com>; dev at dpdk.org
> Subject: Re: [PATCH v4 03/11] net/mlx5: support L3 VXLAN flow
> 
> On Wed, Apr 18, 2018 at 02:43:30PM +0000, Xueming(Steven) Li wrote:
> >
> >
> > > -----Original Message-----
> > > From: Nélio Laranjeiro <nelio.laranjeiro at 6wind.com>
> > > Sent: Wednesday, April 18, 2018 2:49 PM
> > > To: Xueming(Steven) Li <xuemingl at mellanox.com>
> > > Cc: Shahaf Shuler <shahafs at mellanox.com>; dev at dpdk.org
> > > Subject: Re: [PATCH v4 03/11] net/mlx5: support L3 VXLAN flow
> > >
> > > On Tue, Apr 17, 2018 at 11:14:28PM +0800, Xueming Li wrote:
> > > > This patch support L3 VXLAN, no inner L2 header comparing to
> > > > standard VXLAN protocol. L3 VXLAN using specific overlay UDP
> > > > destination port to discriminate against standard VXLAN, FW has to
> > > > be configured to support
> > > > it:
> > > >   sudo mlxconfig -d <device> -y s IP_OVER_VXLAN_EN=1
> > > >   sudo mlxconfig -d <device> -y s IP_OVER_VXLAN_PORT=<port>
> > > >
> > > > Signed-off-by: Xueming Li <xuemingl at mellanox.com>
> > > > ---
> > > >  drivers/net/mlx5/mlx5_flow.c | 4 +++-
> > > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/drivers/net/mlx5/mlx5_flow.c
> > > > b/drivers/net/mlx5/mlx5_flow.c index 771d5f14d..d7a921dff 100644
> > > > --- a/drivers/net/mlx5/mlx5_flow.c
> > > > +++ b/drivers/net/mlx5/mlx5_flow.c
> > > > @@ -413,7 +413,9 @@ static const struct mlx5_flow_items mlx5_flow_items[] = {
> > > >  		.dst_sz = sizeof(struct ibv_flow_spec_tunnel),
> > > >  	},
> > > >  	[RTE_FLOW_ITEM_TYPE_VXLAN] = {
> > > > -		.items = ITEMS(RTE_FLOW_ITEM_TYPE_ETH),
> > > > +		.items = ITEMS(RTE_FLOW_ITEM_TYPE_ETH,
> > > > +			       RTE_FLOW_ITEM_TYPE_IPV4, /* For L3 VXLAN. */
> > > > +			       RTE_FLOW_ITEM_TYPE_IPV6), /* For L3 VXLAN. */
> > > >  		.actions = valid_actions,
> > > >  		.mask = &(const struct rte_flow_item_vxlan){
> > > >  			.vni = "\xff\xff\xff",
> > > > --
> > > > 2.13.3
> > >
> > > Such support must be under device parameter has it depends on the
> > > configuration of the firmware.  If the firmware is not correctly configured the PMD must refuse
> such rule.
> > >
> > > Thanks,
> > >
> > > --
> > > Nélio Laranjeiro
> > > 6WIND
> >
> > Are you suggesting Verbs parameter? I'm afraid we can't have it in
> > short time, need new patch in later release when Verbs ready.
> 
> Take a look at [1], this is what I mean.

Enabling a new device parameter can't make L3 VXLAN packet get received if fw configuration not set.
On the other hand, if fw continuation enabled and device parameter not set, packet could be received
but failed to create rule. I'm afraid that a device parameter will introduce complexity of using 
this feature w/o real benefits.

> 
> Regards,
> 
> [1]
> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdpdk.org%2Fdoc%2Fguides%2Fnics%2Fml
> x5.html%23run-time-
> configuration&data=02%7C01%7Cxuemingl%40mellanox.com%7Ce3b4395f998a447f054308d5a53e3afc%7Ca652971c7d2e
> 4d9ba6a4d149256f461b%7C0%7C0%7C636596609057090801&sdata=VrTSE0favcGEhpyURZ8thHjBxdlR8f80%2BOmDy1a1vZU%
> 3D&reserved=0
> 
> --
> Nélio Laranjeiro
> 6WIND