Re: [dpdk-dev] [PATCH v2 6/7] net/mlx5: fix reception when VLAN is added

2017-10-24 Thread Nélio Laranjeiro
On Tue, Oct 24, 2017 at 06:41:07AM -0700, Yongseok Koh wrote: > On Tue, Oct 24, 2017 at 09:34:34AM +0200, Nélio Laranjeiro wrote: > > On Tue, Oct 24, 2017 at 09:11:42AM +0200, Nélio Laranjeiro wrote: > > > On Mon, Oct 23, 2017 at 12:25:45PM -0700, Yongseok Koh wrote: > > > > On Mon, Oct 23, 2017 at

Re: [dpdk-dev] [PATCH v2 6/7] net/mlx5: fix reception when VLAN is added

2017-10-24 Thread Yongseok Koh
On Tue, Oct 24, 2017 at 09:34:34AM +0200, Nélio Laranjeiro wrote: > On Tue, Oct 24, 2017 at 09:11:42AM +0200, Nélio Laranjeiro wrote: > > On Mon, Oct 23, 2017 at 12:25:45PM -0700, Yongseok Koh wrote: > > > On Mon, Oct 23, 2017 at 04:49:56PM +0200, Nelio Laranjeiro wrote: > > > > @@ -261,75 +284,59

Re: [dpdk-dev] [PATCH v2 6/7] net/mlx5: fix reception when VLAN is added

2017-10-24 Thread Nélio Laranjeiro
On Tue, Oct 24, 2017 at 09:11:42AM +0200, Nélio Laranjeiro wrote: > On Mon, Oct 23, 2017 at 12:25:45PM -0700, Yongseok Koh wrote: > > On Mon, Oct 23, 2017 at 04:49:56PM +0200, Nelio Laranjeiro wrote: > > > @@ -261,75 +284,59 @@ priv_dev_traffic_enable(struct priv *priv, struct > > > rte_eth_dev *d

Re: [dpdk-dev] [PATCH v2 6/7] net/mlx5: fix reception when VLAN is added

2017-10-24 Thread Nélio Laranjeiro
On Mon, Oct 23, 2017 at 12:25:45PM -0700, Yongseok Koh wrote: > On Mon, Oct 23, 2017 at 04:49:56PM +0200, Nelio Laranjeiro wrote: > > @@ -261,75 +284,59 @@ priv_dev_traffic_enable(struct priv *priv, struct > > rte_eth_dev *dev) > > }; > > > > claim_zero(mlx5_ctrl_flow(dev

Re: [dpdk-dev] [PATCH v2 6/7] net/mlx5: fix reception when VLAN is added

2017-10-23 Thread Yongseok Koh
On Mon, Oct 23, 2017 at 04:49:56PM +0200, Nelio Laranjeiro wrote: > @@ -261,75 +284,59 @@ priv_dev_traffic_enable(struct priv *priv, struct > rte_eth_dev *dev) > }; > > claim_zero(mlx5_ctrl_flow(dev, &promisc, &promisc)); > - } else if (dev->data->all_multicast) {

[dpdk-dev] [PATCH v2 6/7] net/mlx5: fix reception when VLAN is added

2017-10-23 Thread Nelio Laranjeiro
When VLAN is enabled in the Rx side, only packets matching this VLAN are expected, this also includes the broadcast and all multicast packets. Fixes: 272733b5ebfd ("net/mlx5: use flow to enable unicast traffic") Fixes: 6a6b6828fe6a ("net/mlx5: use flow to enable all multi mode") Signed-off-by: Ne