From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adrien Mazarguil Subject: Re: [PATCH 3/3] net/mlx5: do not invalidate title CQE Date: Thu, 17 Nov 2016 11:38:30 +0100 Message-ID: <20161117103830.GC4729@6wind.com> References: <47daa16c78c14e3e11904308b39e086d495ca1d0.1479376117.git.nelio.laranjeiro@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Nelio Laranjeiro Return-path: Received: from mail-wm0-f51.google.com (mail-wm0-f51.google.com [74.125.82.51]) by dpdk.org (Postfix) with ESMTP id 36EEC56A8 for ; Thu, 17 Nov 2016 11:38:39 +0100 (CET) Received: by mail-wm0-f51.google.com with SMTP id t79so135836622wmt.0 for ; Thu, 17 Nov 2016 02:38:39 -0800 (PST) Content-Disposition: inline In-Reply-To: <47daa16c78c14e3e11904308b39e086d495ca1d0.1479376117.git.nelio.laranjeiro@6wind.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, Nov 17, 2016 at 10:49:56AM +0100, Nelio Laranjeiro wrote: > We can leave the title completion queue entry untouched since its contents > are not modified. > > Reported-by: Liming Sun > Signed-off-by: Nelio Laranjeiro > --- > drivers/net/mlx5/mlx5_rxtx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c > index 04860bb..ffd09ac 100644 > --- a/drivers/net/mlx5/mlx5_rxtx.c > +++ b/drivers/net/mlx5/mlx5_rxtx.c > @@ -1162,7 +1162,7 @@ mlx5_rx_poll_len(struct rxq *rxq, volatile struct mlx5_cqe *cqe, > zip->na += 8; > } > if (unlikely(rxq->zip.ai == rxq->zip.cqe_cnt)) { > - uint16_t idx = rxq->cq_ci; > + uint16_t idx = rxq->cq_ci + 1; > uint16_t end = zip->cq_ci; > > while (idx != end) { > -- > 2.1.4 Acked-by: Adrien Mazarguil -- Adrien Mazarguil 6WIND