From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3116CA034F; Mon, 22 Mar 2021 13:25:42 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A60A640040; Mon, 22 Mar 2021 13:25:41 +0100 (CET) Received: from wout3-smtp.messagingengine.com (wout3-smtp.messagingengine.com [64.147.123.19]) by mails.dpdk.org (Postfix) with ESMTP id C19584003D; Mon, 22 Mar 2021 13:25:39 +0100 (CET) Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.west.internal (Postfix) with ESMTP id E35111615; Mon, 22 Mar 2021 08:25:37 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Mon, 22 Mar 2021 08:25:38 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm3; bh= wF0LFrWkF8fwOe+g6bWCIOqOCiFhG9PlMR1BR2t1JQI=; b=jT4aRXydF7E5xzKP QLeoT78bSFMYhgYghHugtvSUPt4+sJwOvXiJ9GlmU5YXuYuhkl6xkIWCVn8Y24YN m/8GljlE9DNGUgXVMl3Tt5AfmaV/vrf8geD9wNCzKBuISE6/i3T8GsO7hCgMGkPR iPGPLurziSF3YNaEms7MTzCpW2g7N6abGiOyLUqPn3tR1IGyJtbV8NXbW4j2yM4+ E4Ga0Syd9pi7+Xx+ZEbtd6FGKS4zD3qAp6ZZ6trKT4MhPVj+DEnwF5Pj38/cBdKC ltXtzFO/utneVhWFZ75AVoILbRTB5ppEKEhX86xqC8h4ya94qzdBJpKsnnAuU6b5 FXSo6g== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; bh=wF0LFrWkF8fwOe+g6bWCIOqOCiFhG9PlMR1BR2t1J QI=; b=uwBJHyCaq+CTuMunyah6/4XjgitIinmYmdfPM68yv4ZibilwRfpGLZdf8 eh1qGPv36CBALnJuiPeshCTFmwnyMag09MmIHGOv6QlRz2daKfyQJtNV+KiDY/y/ ZyaM4MaY7GkEGM+uXgCh+Y75McWDUxAFvEWZEIfwVgOBxcfK2ZCpscNu5biZ2T/B PJ7Ih25cpG2pvlgI8WcyiQlkjWLayUIQex/pfdIdVlIAufFsrCJsf1QxGCVas341 JLGGnyQmUIctx8To67AS3TdotRJA/WI48OpXl2Hyats9TWaTI4En7WbfqGxPUtq7 LdGvs2aVZ5fyg7zZ2Q1yJQOGnmUkQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledrudeggedgfeelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkfgjfhgggfgtsehtqhertddttdejnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepteelhedtheetkeelteduhfdutedvgfdukefhfffgffetvdetvefg leejteetffefnecuffhomhgrihhnpehoiihlrggsshdrohhrghenucfkphepjeejrddufe egrddvtdefrddukeegnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghi lhhfrhhomhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvth X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 1153F1080067; Mon, 22 Mar 2021 08:25:34 -0400 (EDT) From: Thomas Monjalon To: Luca Boccassi Cc: Bruce Richardson , Christian Ehrhardt , "Pai G, Sunil" , Ilya Maximets , "Stokes, Ian" , "Govindharajan, Hariprasad" , "stable@dpdk.org" , dev , James Page Date: Mon, 22 Mar 2021 13:25:33 +0100 Message-ID: <11715925.4IKFeQ5fnV@thomas> In-Reply-To: <7eb39330834de50d2f3ee603adcd7f5501be9a83.camel@debian.org> References: <20200818181222.8462-1-bluca@debian.org> <20210322114101.GB1440@bricha3-MOBL.ger.corp.intel.com> <7eb39330834de50d2f3ee603adcd7f5501be9a83.camel@debian.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] 19.11.4 patches review and test X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 22/03/2021 12:59, Luca Boccassi: > On Mon, 2021-03-22 at 11:41 +0000, Bruce Richardson wrote: > > On Mon, Mar 22, 2021 at 10:49:54AM +0100, Christian Ehrhardt wrote: > > > On Thu, Mar 18, 2021 at 7:25 PM Pai G, Sunil = wrote: > > > > Hi Christian, Ilya > > > > From: Ilya Maximets > > > > > On 3/18/21 2:36 PM, Pai G, Sunil wrote: > > > > > > Hey Christian, > > > > > >=20 > > > > > > > > > > > >=20 > > > > > > > back in 19.11.4 these DPDK changes were not picked up as the= y have > > > > > > > broken builds as discussed here. > > > > > > > Later on the communication was that all this works fine now a= nd > > > > > > > thereby Luca has "reverted the reverts" in 19.11.6 [1]. > > > > > > >=20 > > > > > > > But today we were made aware that still no OVS 2.13 builds ag= ainst a > > > > > > > DPDK that has those changes. > > > > > > > Not 2.13.1 as we have it in Ubuntu nor (if it needs some OVS = changes > > > > > > > backported) the recent 2.13.3 does build. > > > > > > > They still fail with the very same issue I reported [2] back = then. > > > > > > >=20 > > > > > > > Unfortunately I have just released 19.11.7 so I can't revert = them > > > > > > > there - but OTOH reverting and counter reverting every other = release > > > > > > > seems wrong anyway. > > > > >=20 > > > > > It is wrong indeed, but the main question here is why these patch= es was > > > > > backported to stable release in a first place? > > > > >=20 > > > > > Looking at these patches, they are not actual bug fixes but more = like "nice to > > > > > have" features that additionally breaks the way application links= with DPDK. > > > > > Stuff like that should not be acceptable to the stable release wi= thout a strong > > > > > justification or, at least, testing with actual applications. > > >=20 > > > I agree, but TBH IIRC these changes were initially by OVS people :-) > > > One could chase down the old talks between Luca and the requesters, b= ut I don't > > > think that gains us that much. > > >=20 > > > > > Since we already have a revert of revert, revert of revert of rev= ert doesn't > > > > > seem so bad. > > >=20 > > > As long as we don't extend this series, yeah > > >=20 > > > > > > > I wanted to ask if there is a set of patches that OVS would n= eed to > > > > > > > backport to 2.13.x to make this work? > > > > > > > If they could be identified and prepared Distros could use th= em on > > > > > > > 2.13.3 asap and 2.13.4 could officially release them for OVS = later on. > > > > > > >=20 > > > > > > > But for that we'd need a hint which OVS changes that would ne= ed to be. > > > > > > > All I know atm is from the testing reports on DPDK it seems t= hat OVS > > > > > > > 2.14.3 and 2.15 are happy with the new DPDK code. > > > > > > > Do you have pointers on what 2.13.3 would need to get backpor= ted to > > > > > > > work again in regard to this build issue. > > > > > >=20 > > > > > > You would need to use partial contents from patch : > > > > > > http://patchwork.ozlabs.org/project/openvswitch/patch/160814236= 5- > > > > > 26215 > > > > > > -1-git-send-email-ian.stokes@intel.com/ > > > > > >=20 > > > > > > If you'd like me to send patches which would work with 2.13, 2.= 14, I'm > > > > > > ok with that too.[keeping only those parts from patch which fix= es the issue > > > > > you see.] But we must ensure it doesn=E2=80=99t cause problems fo= r OVS too. > > > > > > Your thoughts Ilya ? > > > > >=20 > > > > > We had more fixes on top of this particular patch and I'd like to= not cherry- > > > > > pick and re-check all of this again. > > > >=20 > > > > I agree, we had more fixes on top of this. It would be risky to che= rry-pick. > > > > So it might be a better option to revert. > > >=20 > > > I agree, as far as I assessed the situation it would mean the revert > > > of the following list. > > > And since that is a lot of "reverts" in the string, to be clear it me= ans that > > > those original changes would not be present anymore in 19.11.x. > > >=20 > > > f49248a990 Revert "Revert "build/pkg-config: prevent overlinking"" > > > 39586a4cf0 Revert "Revert "build/pkg-config: improve static linking f= lags"" > > > 906e935a1f Revert "Revert "build/pkg-config: output drivers first for > > > static build"" > > > deebf95239 Revert "Revert "build/pkg-config: move pkg-config file cre= ation"" > > > a3bd9a34bf Revert "Revert "build: always link whole DPDK static libra= ries"" > > > d4bc124438 Revert "Revert "devtools: test static linkage with pkg-con= fig"" > > >=20 > > > But to avoid going back&forth I'd prefer to have a signed-off on that > > > approach from: > > > - Luca (for 19.11.6 which has added the changes) > > > - Bruce (for being involved in the old&new case in general) > > > - Thomas (for general master maintainer thoughts) > > >=20 > >=20 > > If this is what is needed to ensure OVS can continue to use this release > > series, then I am absolutely fine with it. >=20 > This was requested by OVS, so if they don't need it anymore it's fine > by me as well I am not sure to understand the full story, but I am a bit worried that our release is dictated by a single "user" (project using DPDK). Please do you have links of discussion history?