patches for DPDK stable branches
 help / color / mirror / Atom feed
From: Luca Boccassi <bluca@debian.org>
To: "Stokes, Ian" <ian.stokes@intel.com>,
	"Pai G, Sunil" <sunil.pai.g@intel.com>,
	 "Richardson, Bruce" <bruce.richardson@intel.com>,
	Christian Ehrhardt <christian.ehrhardt@canonical.com>
Cc: "stable@dpdk.org" <stable@dpdk.org>, dev <dev@dpdk.org>
Subject: Re: [dpdk-stable] [dpdk-dev] 19.11.4 patches review and test
Date: Mon, 07 Sep 2020 15:25:16 +0100	[thread overview]
Message-ID: <624c602f9cb313153f62359ed25f19ac44827871.camel@debian.org> (raw)
In-Reply-To: <b6273d8593864e8c8e0026c1481621b7@intel.com>

On Tue, 2020-09-01 at 15:10 +0000, Stokes, Ian wrote:
> > Hi,
> > 
> > Yes , OVS was using pkg-config even before these patches were rolled out.
> > 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 having
> > the user to specify them.
> > Moreover, with these patches , the problem of shared DPDK libs always being
> > picked instead of static was not seen any more with a bit of changes from the
> > OVS side as well.
> > http://patchwork.ozlabs.org/project/openvswitch/patch/20200707141126.7141
> > 4-1-sunil.pai.g@intel.com/ .
> > The patches for ovs-master are ready as well and will them out soon.
> > 
> > Thanks and Regards,
> > Pai G, Sunil
> > Sunil
> 
> Hi Folks,
> 
> From the OVS side there was a question WRT why we couldn't use meson in our Travis tests to build DPDK for OVS supporting 19.11.x, in this case it was related to the changes described by Sunil above and hence why the backport was requested. 
> 
> It doesn't have a huge impact on OVS, we can still use 19.11.4 if the patches are removed, however I suspect these changes are also on DPDK main branch 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 change LTS versions.
> 
> From the OVS side we can look to upstream meson support in the dpdk-latest branch and then integrate it when 20.11 is released, but it would be nice to support it before then if possible.
> 
> 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.

-- 
Kind regards,
Luca Boccassi

  reply	other threads:[~2020-09-07 14:25 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-18 18:12 [dpdk-stable] " Luca Boccassi
2020-08-24 13:22 ` [dpdk-stable] [dpdk-dev] " Christian Ehrhardt
2020-08-24 13:25   ` Luca Boccassi
2020-08-25  9:13 ` [dpdk-stable] " Pei Zhang
2020-08-25 10:08   ` [dpdk-stable] [dpdk-dev] " Luca Boccassi
2020-08-26  2:30 ` Chen, BoX C
2020-08-26  9:50   ` Luca Boccassi
2020-08-27  9:47     ` Burakov, Anatoly
2020-08-27 10:37       ` Luca Boccassi
2020-08-27 10:57         ` Burakov, Anatoly
2020-08-27 13:23           ` Luca Boccassi
2020-08-28  4:03     ` Wang, ShougangX
2020-08-28  7:52       ` Luca Boccassi
2020-08-28 14:34 ` Govindharajan, Hariprasad
2020-08-28 15:23   ` Luca Boccassi
2020-08-30 14:36 ` Ali Alnubani
2020-08-31  9:13   ` Luca Boccassi
2020-09-01  8:30 ` [dpdk-stable] " Luca Boccassi
2020-09-01 12:32   ` [dpdk-stable] [dpdk-dev] " Christian Ehrhardt
2020-09-01 12:47     ` Bruce Richardson
2020-09-01 13:22       ` Pai G, Sunil
2020-09-01 15:10         ` Stokes, Ian
2020-09-07 14:25           ` Luca Boccassi [this message]
2021-03-18 11:54         ` Christian Ehrhardt
2021-03-18 13:36           ` Pai G, Sunil
2021-03-18 14:48             ` Ilya Maximets
2021-03-18 18:24               ` Pai G, Sunil
2021-03-22  9:49                 ` Christian Ehrhardt
2021-03-22 11:41                   ` Bruce Richardson
2021-03-22 11:59                     ` Luca Boccassi
2021-03-22 12:25                       ` Thomas Monjalon
2021-03-22 14:27                         ` Christian Ehrhardt
2021-03-23 18:17                           ` Thomas Monjalon
2021-03-23 18:51                             ` Ilya Maximets
2021-03-24  7:44                               ` Christian Ehrhardt
2021-03-24 10:28                                 ` Pai G, Sunil
2021-03-24 13:02                                   ` Christian Ehrhardt
2020-09-01 12:49     ` Luca Boccassi
2020-09-01 13:01       ` Bruce Richardson
2020-09-01 13:28         ` Bruce Richardson
2020-09-01 18:04 ` [dpdk-stable] [EXTERNAL] " Abhishek Marathe
2020-09-02 10:47   ` Luca Boccassi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=624c602f9cb313153f62359ed25f19ac44827871.camel@debian.org \
    --to=bluca@debian.org \
    --cc=bruce.richardson@intel.com \
    --cc=christian.ehrhardt@canonical.com \
    --cc=dev@dpdk.org \
    --cc=ian.stokes@intel.com \
    --cc=stable@dpdk.org \
    --cc=sunil.pai.g@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).