From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nelio Laranjeiro Subject: Re: [PATCH] net/mlx5: fix flow mark action on port start Date: Wed, 15 Nov 2017 08:58:35 +0100 Message-ID: <20171115075835.kyp4yw3yyc2vtvm5@laranjeiro-vm.dev.6wind.com> References: <20171115070949.95541-1-shahafs@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Cc: yskoh@mellanox.com, adrien.mazarguil@6wind.com, dev@dpdk.org To: Shahaf Shuler Return-path: Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com [74.125.82.65]) by dpdk.org (Postfix) with ESMTP id F31F21B1F7 for ; Wed, 15 Nov 2017 08:58:43 +0100 (CET) Received: by mail-wm0-f65.google.com with SMTP id b189so1039146wmd.5 for ; Tue, 14 Nov 2017 23:58:43 -0800 (PST) Content-Disposition: inline In-Reply-To: <20171115070949.95541-1-shahafs@mellanox.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, Nov 15, 2017 at 09:09:49AM +0200, Shahaf Shuler wrote: > Mark action flag was not set on the flow, causing the rx burst function > after port start to ignore the flow mark. > > Fixes: 8086cf08b2f0 ("net/mlx5: handle RSS hash configuration in RSS flow") > Cc: nelio.laranjeiro@6wind.com > > Signed-off-by: Shahaf Shuler > --- > drivers/net/mlx5/mlx5_flow.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c > index 466d74b5f..f32dfdd3f 100644 > --- a/drivers/net/mlx5/mlx5_flow.c > +++ b/drivers/net/mlx5/mlx5_flow.c > @@ -1930,6 +1930,7 @@ priv_flow_create(struct priv *priv, > flow->queues = (uint16_t (*)[])(flow + 1); > memcpy(flow->queues, parser.queues, parser.queues_n * sizeof(uint16_t)); > flow->queues_n = parser.queues_n; > + flow->mark = parser.mark; > /* Copy RSS configuration. */ > flow->rss_conf = parser.rss_conf; > flow->rss_conf.rss_key = flow->rss_key; > -- > 2.12.0 Acked-by: Nelio Laranjeiro -- Nélio Laranjeiro 6WIND