From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id D6DEB325A; Mon, 21 Aug 2017 17:25:20 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id ED254356F9; Mon, 21 Aug 2017 15:25:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com ED254356F9 Authentication-Results: ext-mx06.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx06.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=ktraynor@redhat.com Received: from ktraynor.remote.csb (unknown [10.36.118.65]) by smtp.corp.redhat.com (Postfix) with ESMTP id E339F5C54B; Mon, 21 Aug 2017 15:25:18 +0000 (UTC) To: Yuanhan Liu , Ferruh Yigit References: <20170818104545.GN9612@yliu-home> <20170821034011.GO9612@yliu-home> <2d8dbb48-978f-94fb-1282-8b5c4ea30220@intel.com> <20170821092326.GQ9612@yliu-home> Cc: dpdk stable , dev@dpdk.org From: Kevin Traynor Organization: Red Hat Message-ID: <35153d01-fca2-8844-09d0-107199e0a01f@redhat.com> Date: Mon, 21 Aug 2017 16:25:17 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <20170821092326.GQ9612@yliu-home> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.30]); Mon, 21 Aug 2017 15:25:20 +0000 (UTC) Subject: Re: [dpdk-dev] [dpdk-stable] 16.11.3 (LTS) patches review and test 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: Mon, 21 Aug 2017 15:25:21 -0000 On 08/21/2017 10:23 AM, Yuanhan Liu wrote: > On Mon, Aug 21, 2017 at 09:23:24AM +0100, Ferruh Yigit wrote: >>>> I think this one should go in too, as OVS hits this and writes back the >>>> wrong watermark value to the shared register which can cause problems >>>> for other ports. I've applied and tested it with DPDK 16.11. >>>> >>>> commit 0e61ab56e01655f02bfe4e6249e032e864b0f5dd >>>> Author: Qi Zhang >>>> Date: Thu Aug 10 18:48:07 2017 +0800 >>>> >>>> net/i40e: fix flow control watermark mismatch >>>> >>>> Flow control watermark is not read out correctly, >>>> that may cause an application who not intend to change >>>> watermark but does change it with a rte_eth_dev_flow_ctrl_set >>>> call right after rte_eth_dev_flow_ctrl_get. >>> >>> Weird, I couldn't find this commit: >>> >>> $ git show 0e61ab56e01655f02bfe4e6249e032e864b0f5dd >>> fatal: bad object 0e61ab56e01655f02bfe4e6249e032e864b0f5dd >> >> The commit is in next-net. Not sure how to proceed, any idea? > > I see. Thanks. So it's not merged to Thomas tree yet. I normally take > commits from there. Because I add a tag like following every time I > pick a commit: > > [ upstream commit 5b9b65b14e05c106bb8229c0fe0b347315da7d00 ] > oops, I didn't think about that :/ > If I take commits directly from next-* tree, the commit could have > been changed while merging to Thomas tree, for two reasons: > > - the next-* could have done a rebase > - we don't do "git merge" at Thomas tree > > > However, if that commit is urgent, I think I could make it for 16.11.3 > release with following tag: > > [ next-net commit 0e61ab56e01655f02bfe4e6249e032e864b0f5dd ] > > If not, I will postone it to 16.11.4. > > --yliu > Given that it took 9 months for the bug to be found, I think postponing to DPDK 16.11.4 is better than upsetting the dpdk-stable commit log to reference multiple repos. thanks, Kevin.