On Tue, Apr 10, 2018 at 09:34:10PM +0800, Xueming Li wrote: > Tunnel w/o tunnel id pattern could match any non-tunneled packet, > this patch allowed tunnel w/o tunnel id pattern after proper outer spec. > > Signed-off-by: Xueming Li <xuemi...@mellanox.com>
Acked-by: Nelio Laranjeiro <nelio.laranje...@6wind.com> > --- > drivers/net/mlx5/mlx5_flow.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c > index 857b8b716..58d437308 100644 > --- a/drivers/net/mlx5/mlx5_flow.c > +++ b/drivers/net/mlx5/mlx5_flow.c > @@ -1803,7 +1803,8 @@ mlx5_flow_create_vxlan(const struct rte_flow_item *item, > * before will also match this rule. > * To avoid such situation, VNI 0 is currently refused. > */ > - if (!vxlan.val.tunnel_id) > + /* Only allow tunnel w/o tunnel id pattern after proper outer spec. */ > + if (parser->out_layer == HASH_RXQ_ETH && !vxlan.val.tunnel_id) > return rte_flow_error_set(data->error, EINVAL, > RTE_FLOW_ERROR_TYPE_ITEM, > item, > -- > 2.13.3 > -- Nélio Laranjeiro 6WIND
