From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f48.google.com (mail-wm0-f48.google.com [74.125.82.48]) by dpdk.org (Postfix) with ESMTP id 6A7B82B91 for ; Thu, 5 Jan 2017 19:12:20 +0100 (CET) Received: by mail-wm0-f48.google.com with SMTP id a197so442004083wmd.0 for ; Thu, 05 Jan 2017 10:12:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=GbeP+1G/DfRFtYLKheevMFK1ucmhIMxs+uW3XixGOKg=; b=Oj2G3mGaL2PJkP+0S8pUxc3RTKIO6IuwwmNWj+f4I2OgQCr29iUA3DH8poRnDusB5w GrTqKcijXIsDgAENSFk3le02bgQfg6TaPHzWssubaP7S28Kvqe5ifpcb+OxHU91JnUoO gvFxBWk8C2oG5DAki7cG7gnvgST7u9lpGBs1w31xD3oDSIi2oooQIQbUxX3GjA7Gbm24 jhQNwIFbPCQNhfYPmYDKFXsYgdpmwF5sqF1+bs/ngtligXXd/pmtmfKaxg09x+hduyE1 RVVkdo8ZVlmSFTk80uXMINvpoG/F553Aaq1CcLs2Xm/LgY5HafLSvuIDeIj9VioqC6mn JCKQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=GbeP+1G/DfRFtYLKheevMFK1ucmhIMxs+uW3XixGOKg=; b=QODkyKck4+lSlYRh74+isMMUv2CQL7Jvn6ka4ms2wsiC/vVICAliCsWvUG9pYNdjoQ db1VP9uq7bOt08G6VZL6TgVzIovYoTNql10rQgXjOK1hvm0o1ljEHyPoBADbqNIK7PMh ffQPrZqexXLbOqq6ZLhoeBbrtlWHRpGgaqgioFO2sBaCx/pD5rw1pmbp34c4aYtLMAg7 +f4fh+aMRm0euLgn8prA5EksOvkNAjANlkxGrLCEObUYrnnI7O6vGEYGnz6TVwZn06pJ 8PaGq06VhIfAfNCvmjkeEHSta9tEjShuvn/abyCEHrRjJbYSU07qH6dotVJ4hQTtqImN 1tSA== X-Gm-Message-State: AIkVDXKRXj0lBXukdYz6zhygPav1Mh2pwBoenV3MBXIfZ3WJCB9azVWbuAj+VjT9luw0ozPB X-Received: by 10.28.52.76 with SMTP id b73mr68131861wma.8.1483639940064; Thu, 05 Jan 2017 10:12:20 -0800 (PST) Received: from 6wind.com (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id e188sm101801286wma.21.2017.01.05.10.12.18 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 05 Jan 2017 10:12:18 -0800 (PST) Date: Thu, 5 Jan 2017 19:12:12 +0100 From: Adrien Mazarguil To: Ferruh Yigit Cc: Nelio Laranjeiro , dev@dpdk.org Message-ID: <20170105181212.GR12822@6wind.com> References: <1481290078-4530-1-git-send-email-nelio.laranjeiro@6wind.com> <2da35386-f774-a1ca-c4f2-cc77f74f6e2c@intel.com> <20170105163226.GP12822@6wind.com> <20170105165223.GQ12822@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [dpdk-dev] [PATCH] net/mlx5: fix Tx doorbell X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 05 Jan 2017 18:12:20 -0000 On Thu, Jan 05, 2017 at 05:01:27PM +0000, Ferruh Yigit wrote: > On 1/5/2017 4:52 PM, Adrien Mazarguil wrote: > > On Thu, Jan 05, 2017 at 05:32:26PM +0100, Adrien Mazarguil wrote: > >> Hi Ferruh, > >> > >> On Thu, Jan 05, 2017 at 03:19:35PM +0000, Ferruh Yigit wrote: > >>> On 12/9/2016 1:27 PM, Nelio Laranjeiro wrote: > >>>> Too much data is uselessly written to the Tx doorbell. > >>>> > >>>> Fixes: 1d88ba171942 ("net/mlx5: refactor Tx data path") > >>>> > >>>> Signed-off-by: Nelio Laranjeiro > >>>> Acked-by: Adrien Mazarguil > >>>> > >>> > >>> Applied to dpdk-next-net/master, thanks. > >>> > >>> Is not CC'ing stable intentional, since this patch depends on a patch > >>> introduced in this release? If not intentional, please CC stable. > >> > >> I intended to update the commit message for this patch as in the meantime we > >> discovered it addresses a significant regression introduced in v16.11. > >> > >> CC'ing stable now. > >> > >> If possible, can you amend the commit log with: > >> > >> --- > >> > >> net/mlx5: fix Tx doorbell > >> > >> Too much data is uselessly written to the Tx doorbell, which since v16.11 > >> may also cause Tx queues to behave erratically and crash applications. > >> > >> This regression was seen on VF devices when the BlueFlame buffer size is > >> zero (txq->cqe_n == 0) due to the following change: > >> > >> - cqe = &(*txq->cqes)[ci & (txq->cqe_n - 1)].cqe64; > >> + cqe = &(*txq->cqes)[ci & ((1 << txq->cqe_n) - 1)]; > >> > >> Fixes: 1d88ba171942 ("net/mlx5: refactor Tx data path") > >> Fixes: e2f116ee3cac ("net/mlx5: reduce memory overhead for CQE handling") > >> > >> Signed-off-by: Nelio Laranjeiro > >> Acked-by: Adrien Mazarguil > >> Cc: stable@dpdk.org > >> > >> --- > > > > I mixed the commit that introduced the regression with a similar looking yet > > harmless one, here is the proper message to use, sorry for the noise: > > > > --- > > > > net/mlx5: fix Tx doorbell > > > > Too much data is uselessly written to the Tx doorbell, which since v16.11 > > may also cause Tx queues to behave erratically and crash applications. > > > > This regression was seen on VF devices when the BlueFlame buffer size is > > zero (txq->bf_buf_size) due to the following change: > > > > - txq->bf_offset ^= txq->bf_buf_size; > > + txq->bf_offset ^= (1 << txq->bf_buf_size); > > > > Fixes: 1d88ba171942 ("net/mlx5: refactor Tx data path") > > Fixes: d5793daefec8 ("net/mlx5: reduce memory overhead for BF handling") > > > > Signed-off-by: Nelio Laranjeiro > > Acked-by: Adrien Mazarguil > > Cc: stable@dpdk.org > > > > --- > > > > Can you please confirm commit in latest next-net? Not sure if the "Cc: stable@dpdk.org" line should have been kept, otherwise it's perfect, thanks Ferruh! -- Adrien Mazarguil 6WIND