From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1E723A04B6; Mon, 7 Sep 2020 16:25:20 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 673911BF8A; Mon, 7 Sep 2020 16:25:19 +0200 (CEST) Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) by dpdk.org (Postfix) with ESMTP id A170B1BE85; Mon, 7 Sep 2020 16:25:17 +0200 (CEST) Received: by mail-wr1-f46.google.com with SMTP id j2so16009336wrx.7; Mon, 07 Sep 2020 07:25:17 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:content-transfer-encoding:user-agent:mime-version; bh=mEdV+Rd7CDb2nbFTpEVMo+M7m2/+C1XVcs57VQfIOAw=; b=nYXeGKm0oYCjtRVDaYEAXaqtasjGJJUF+wsX4wBO2ZCd7g7nve8yTyOZZWJ7ym16ua 4OVH/L5ZnbcoNJ9yKFX0EaY/N3ha0zSObMTqEZBMwIi+a4ohf5rOSRvtzr9d1WLzfreC yjfzJyf/+xb7IJ1r+SLwi6mQdO37ynumJ2PpAHGJ68lEZ9Vx8OM+WZy5k1uVqaSDIyV0 KQe7+wdJWLSY5CBBnKsg9+HGaOhxmmfSRbmduR3KlP47pkpt/tlwgkBXZoOT9R063dhI y/5L6p1ZBlid0a5XnqQIsx2kxVhQiJthx+9HW01Yx1WilWbf36rVT38tIQ3qgkyJPi9X WfEQ== X-Gm-Message-State: AOAM532CuaCIjj3y6KEYeFT/dNbZZ07rxDdjp94n1xf0w5SKrgV85mN8 //ZJM3z2MzVdhmjqER56I2E= X-Google-Smtp-Source: ABdhPJwvHYvuG32w2sQGo15ZdfgBgjU6Py3uRsSrsL/Z1Xj5ieEfAqG+sex/dOgMzXOh7xtNixqKrA== X-Received: by 2002:a5d:514e:: with SMTP id u14mr21643585wrt.211.1599488717271; Mon, 07 Sep 2020 07:25:17 -0700 (PDT) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id p11sm25956824wma.11.2020.09.07.07.25.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Sep 2020 07:25:16 -0700 (PDT) Message-ID: <624c602f9cb313153f62359ed25f19ac44827871.camel@debian.org> From: Luca Boccassi To: "Stokes, Ian" , "Pai G, Sunil" , "Richardson, Bruce" , Christian Ehrhardt Cc: "stable@dpdk.org" , dev Date: Mon, 07 Sep 2020 15:25:16 +0100 In-Reply-To: References: <20200818181222.8462-1-bluca@debian.org> <20200901124747.GB1047@bricha3-MOBL.ger.corp.intel.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.30.5-1.1 MIME-Version: 1.0 Subject: Re: [dpdk-dev] [dpdk-stable] 19.11.4 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, 2020-09-01 at 15:10 +0000, Stokes, Ian wrote: > > Hi, > >=20 > > Yes , OVS was using pkg-config even before these patches were rolled ou= t. > > But it always used to pick the DPDK shared libs by default for OVS even= on using > > the -Bstatic/-Bshared flags. > > These patches from Bruce simplify the process from DPDK side without ha= ving > > the user to specify them. > > Moreover, with these patches , the problem of shared DPDK libs always b= eing > > picked instead of static was not seen any more with a bit of changes fr= om the > > OVS side as well. > > http://patchwork.ozlabs.org/project/openvswitch/patch/20200707141126.71= 41 > > 4-1-sunil.pai.g@intel.com/ . > > The patches for ovs-master are ready as well and will them out soon. > >=20 > > Thanks and Regards, > > Pai G, Sunil > > Sunil >=20 > Hi Folks, >=20 > From the OVS side there was a question WRT why we couldn't use meson in o= ur Travis tests to build DPDK for OVS supporting 19.11.x, in this case it w= as related to the changes described by Sunil above and hence why the backpo= rt was requested.=20 >=20 > It doesn't have a huge impact on OVS, we can still use 19.11.4 if the pat= ches are removed, however I suspect these changes are also on DPDK main bra= nch at this point and will be part of 20.11? So eventually when we move to = 20.11 the same issues may be encountered, maybe that preferable if we are c= hange LTS versions. >=20 > From the OVS side we can look to upstream meson support in the dpdk-lates= t branch and then integrate it when 20.11 is released, but it would be nice= to support it before then if possible. >=20 > Regards > Ian I'll proceed with a revert of those 5 patches for now, given 19.11.4 is already a week late - Christian confirms the build issue is fixed after doing so. We can then figure out what the underlinking issue is without urgency in time for 20.11, and then I'll add the patches again. --=20 Kind regards, Luca Boccassi