All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Ehrhardt <christian.ehrhardt@canonical.com>
To: Ali Alnubani <alialnu@nvidia.com>
Cc: "stable@dpdk.org" <stable@dpdk.org>,
	"dev@dpdk.org" <dev@dpdk.org>,
	 Abhishek Marathe <Abhishek.Marathe@microsoft.com>,
	Akhil Goyal <akhil.goyal@nxp.com>,
	 "benjamin.walker@intel.com" <benjamin.walker@intel.com>,
	David Christensen <drc@linux.vnet.ibm.com>,
	 "hariprasad.govindharajan@intel.com"
	<hariprasad.govindharajan@intel.com>,
	 Hemant Agrawal <hemant.agrawal@nxp.com>,
	Ian Stokes <ian.stokes@intel.com>,
	 Jerin Jacob <jerinj@marvell.com>,
	John McNamara <john.mcnamara@intel.com>,
	 Ju-Hyoung Lee <juhlee@microsoft.com>,
	Kevin Traynor <ktraynor@redhat.com>,
	 Luca Boccassi <bluca@debian.org>, Pei Zhang <pezhang@redhat.com>,
	 "pingx.yu@intel.com" <pingx.yu@intel.com>,
	"qian.q.xu@intel.com" <qian.q.xu@intel.com>,
	 Raslan Darawsheh <rasland@nvidia.com>,
	NBU-Contact-Thomas Monjalon <thomas@monjalon.net>,
	 "yuan.peng@intel.com" <yuan.peng@intel.com>,
	"zhaoyan.chen@intel.com" <zhaoyan.chen@intel.com>,
	 Asaf Penso <asafp@nvidia.com>,
	Slava Ovsiienko <viacheslavo@nvidia.com>
Subject: Re: [dpdk-dev] 19.11.9 patches review and test
Date: Mon, 5 Jul 2021 13:18:45 +0200	[thread overview]
Message-ID: <CAATJJ0J6bDo1wLh73EkjQrTsSuP3+a71CrGyzLnxVTny_wz_uQ@mail.gmail.com> (raw)
In-Reply-To: <DM4PR12MB516727A7A526DD6ABAAF9ACCDA1C9@DM4PR12MB5167.namprd12.prod.outlook.com>

On Mon, Jul 5, 2021 at 12:44 PM Ali Alnubani <alialnu@nvidia.com> wrote:
>
> > -----Original Message-----
> > From: Ali Alnubani
> > Sent: Monday, July 5, 2021 9:35 AM
> > To: Christian Ehrhardt <christian.ehrhardt@canonical.com>
> > Cc: stable@dpdk.org; dev@dpdk.org; Abhishek Marathe
> > <Abhishek.Marathe@microsoft.com>; Akhil Goyal <akhil.goyal@nxp.com>;
> > benjamin.walker@intel.com; David Christensen <drc@linux.vnet.ibm.com>;
> > hariprasad.govindharajan@intel.com; Hemant Agrawal
> > <hemant.agrawal@nxp.com>; Ian Stokes <ian.stokes@intel.com>; Jerin
> > Jacob <jerinj@marvell.com>; John McNamara <john.mcnamara@intel.com>;
> > Ju-Hyoung Lee <juhlee@microsoft.com>; Kevin Traynor
> > <ktraynor@redhat.com>; Luca Boccassi <bluca@debian.org>; Pei Zhang
> > <pezhang@redhat.com>; pingx.yu@intel.com; qian.q.xu@intel.com; Raslan
> > Darawsheh <rasland@nvidia.com>; NBU-Contact-Thomas Monjalon
> > <thomas@monjalon.net>; yuan.peng@intel.com; zhaoyan.chen@intel.com;
> > Asaf Penso <asafp@nvidia.com>
> > Subject: RE: 19.11.9 patches review and test
> >
> > Hi Christian,
> > Sorry for the delay.
> >
> > > -----Original Message-----
> > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com>
> > > Sent: Friday, July 2, 2021 10:42 AM
> > > To: Ali Alnubani <alialnu@nvidia.com>
> > > Cc: stable@dpdk.org; dev@dpdk.org; Abhishek Marathe
> > > <Abhishek.Marathe@microsoft.com>; Akhil Goyal <akhil.goyal@nxp.com>;
> > > benjamin.walker@intel.com; David Christensen
> > <drc@linux.vnet.ibm.com>;
> > > hariprasad.govindharajan@intel.com; Hemant Agrawal
> > > <hemant.agrawal@nxp.com>; Ian Stokes <ian.stokes@intel.com>; Jerin
> > > Jacob <jerinj@marvell.com>; John McNamara
> > <john.mcnamara@intel.com>;
> > > Ju-Hyoung Lee <juhlee@microsoft.com>; Kevin Traynor
> > > <ktraynor@redhat.com>; Luca Boccassi <bluca@debian.org>; Pei Zhang
> > > <pezhang@redhat.com>; pingx.yu@intel.com; qian.q.xu@intel.com;
> > Raslan
> > > Darawsheh <rasland@nvidia.com>; NBU-Contact-Thomas Monjalon
> > > <thomas@monjalon.net>; yuan.peng@intel.com;
> > zhaoyan.chen@intel.com
> > > Subject: Re: 19.11.9 patches review and test
> > >
> > > On Thu, Jul 1, 2021 at 5:50 PM Ali Alnubani <alialnu@nvidia.com> wrote:
> > > >
> > > > Hi,
> > > >
> > > > > -----Original Message-----
> > > > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com>
> > > > > Sent: Thursday, June 17, 2021 9:38 AM
> > > > > To: stable@dpdk.org
> > > > > Cc: dev@dpdk.org; Abhishek Marathe
> > > <Abhishek.Marathe@microsoft.com>;
> > > > > Akhil Goyal <akhil.goyal@nxp.com>; Ali Alnubani
> > > > > <alialnu@nvidia.com>; benjamin.walker@intel.com; David Christensen
> > > > > <drc@linux.vnet.ibm.com>; hariprasad.govindharajan@intel.com;
> > > Hemant
> > > > > Agrawal <hemant.agrawal@nxp.com>; Ian Stokes
> > > <ian.stokes@intel.com>;
> > > > > Jerin Jacob <jerinj@marvell.com>; John McNamara
> > > > > <john.mcnamara@intel.com>; Ju-Hyoung Lee
> > <juhlee@microsoft.com>;
> > > > > Kevin Traynor <ktraynor@redhat.com>; Luca Boccassi
> > > > > <bluca@debian.org>; Pei Zhang <pezhang@redhat.com>;
> > > > > pingx.yu@intel.com; qian.q.xu@intel.com; Raslan Darawsheh
> > > > > <rasland@nvidia.com>; NBU-Contact-Thomas Monjalon
> > > > > <thomas@monjalon.net>; yuan.peng@intel.com;
> > > zhaoyan.chen@intel.com
> > > > > Subject: 19.11.9 patches review and test
> > > > >
> > > > > Hi all,
> > > > >
> > > > > Here is a list of patches targeted for stable release 19.11.9.
> > > > >
> > > > > The planned date for the final release is 2nd July.
> > > > >
> > > > > Please help with testing and validation of your use cases and
> > > > > report any issues/results with reply-all to this mail. For the
> > > > > final release the fixes and reported validations will be added to
> > > > > the release
> > > notes.
> > > > >
> > > > > A renewed release candidate tarball can be found at:
> > > > >
> > > > >     https://dpdk.org/browse/dpdk-stable/tag/?id=v19.11.9-rc3
> > > > >
> > > > > These patches are located at branch 19.11 of dpdk-stable repo:
> > > > >     https://dpdk.org/browse/dpdk-stable/
> > > > >
> > > > > Thanks.
> > > > >
> > > > > Christian Ehrhardt <christian.ehrhardt@canonical.com>
> > > > >
> > > >
> > > > The following covers the functional tests that we ran on Mellanox
> > > hardware for this release:
> > > > - Basic functionality:
> > > >   Send and receive multiple types of traffic.
> > > > - testpmd xstats counter test.
> > > > - testpmd timestamp test.
> > > > - Changing/checking link status through testpmd.
> > > > - RTE flow tests:
> > > >   Items: eth / vlan / ipv4 / ipv6 / tcp / udp / icmp / gre / nvgre /
> > > > vxlan / ip in
> > > ip / mplsoudp / mplsogre
> > > >   Actions: drop / queue / rss / mark / flag / jump / count /
> > > > raw_encap / raw_decap / vxlan_encap / vxlan_decap / NAT / dec_ttl
> > > > - Some RSS tests.
> > > > - VLAN filtering, stripping and insertion tests.
> > > > - Checksum and TSO tests.
> > > > - ptype tests.
> > > > - link_status_interrupt example application tests.
> > > > - l3fwd-power example application tests.
> > > > - Multi-process example applications tests.
> > > >
> > > > Functional tests ran on:
> > > > - NIC: ConnectX-4 Lx / OS: Ubuntu 20.04 LTS / Driver:
> > > > MLNX_OFED_LINUX-5.3-1.0.0.1 / Firmware: 14.30.1004
> > > > - NIC: ConnectX-5 / OS: Ubuntu 20.04 LTS / Driver:
> > > > MLNX_OFED_LINUX-5.3-1.0.0.1 / Firmware: 16.30.1004
> > > >
> > > > We discovered 2 new issues due to environment changes:
> > > > - can't create some rules with count action, fixed by
> > > https://inbox.dpdk.org/stable/20210621145105.963179-1-
> > > lmargalit@nvidia.com/T/#u.
> > >
> > > As discussed in the release meeting yesterday, that is a regression
> > > and needs to be added.
> > > While I feel slightly tired of new respins this is what we do the validation
> > for.
> > > So thanks you (all!) for the testing!
> > >
> > > I have now applied the referred fix to 19.11.9
> >
> > Thank you.
> >
> > >
> > > FYI: I see this came up ~5 days into the -rc3 testing, but I missed it
> > > so I checked why.
> > > It was breaking the usual subject syntax (double colon) and not
> > > flagged for
> > > 19.11.9 (as requested in backport mails and the common
> > > style) in subject or body.
> > >
> > > > - rte_flow hit counter doesn't increment, still discussing this internally.
> > >
> > > For this issue please tell me until Monday what you expect.
> > > Does it look close to being resolved (then I'll hold -rc4 back a bit
> > > longer) or should I go on planning to release 19.11.9 without it (then
> > > I'd tag -
> > > rc4 on Monday).
> >
> > We are checking if we can resolve this in the next few hours. Hope that's ok
> > with you.
>
> Should be resolved by https://inbox.dpdk.org/stable/20210705085856.13971-1-viacheslavo@nvidia.com/T/#u.

Thanks - got it, applies and is mlx-only.
That should be fine.

I'm doing cross-arch/cross-distro build tests before tagging -rc4 based on this.

> >
> > >
> > > Generally on -rc4 I think it would be sufficient if you would re-run
> > > your testing as your tests cover the only two things we have touched
> > > in -rc4 (clang
> > > 12 and the flow issue).
> > > I'm not "against" more tests, just saying that those should be
> > > sufficient in case no others come back then.
> > > Because other than the issues reported in here all other tests already
> > > came back fully green for -rc3.
> > >
> > > FYI the usual non-official WIP repo on
> > > https://github.com/cpaelzer/dpdk-stable-queue/commits/19.11 holds
> > what
> > > 19.11.9-rc4 would become if you want to pre-test anything.
> > >
> > > > Compilation tests with multiple configurations in the following
> > > > OS/driver
> > > combinations are also passing:
> > > > - Ubuntu 20.04.2 with MLNX_OFED_LINUX-5.3-1.0.0.1.
> > > > - Ubuntu 20.04.2 with rdma-core master (a66e2a5).
> > > > - Ubuntu 20.04.2 with rdma-core v28.0.
> > > > - Ubuntu 18.04.5 with rdma-core v17.1.
> > > > - Ubuntu 18.04.5 with rdma-core master (a66e2a5) (i386).
> > > > - Ubuntu 16.04.7 with rdma-core v22.7.
> > > > - Fedora 34 with rdma-core v35.0 (only with gcc).
> > > > - Fedora 35 (Rawhide) with rdma-core v35.0 (only with gcc).
> > > > - CentOS 7 7.9.2009 with rdma-core master (a66e2a5).
> > > > - CentOS 7 7.9.2009 with MLNX_OFED_LINUX-5.3-1.0.0.1.
> > > > - CentOS 8 8.3.2011 with rdma-core master (7f2d460).
> > > > - OpenSUSE Leap 15.3 with rdma-core v31.0.
> > > >
> > > > Note that clang builds are failing in Fedora 34 and 35 due to the
> > > > following
> > > issues:
> > > > - https://bugs.dpdk.org/show_bug.cgi?id=745
> > >
> > > This is still totally unknown and not specific to 19.11 series.
> > > I guess this clang issue has to wait for the next stable release then.
> > >
> > > > - https://bugs.dpdk.org/show_bug.cgi?id=733
> > >
> > > We've identified the patch that needs to be backported for this one.
> > > I have done so and built across all platforms - LGTM.
> > > If your tests could also confirm this once we have -rc4 tagged that
> > > would be great.
> >
> > Sure, I'll retest once rc4 is out.
> >
> > >
> > > > We don't see any other critical issues blocking this release.
> > > >
> > > > Thanks,
> > > > Ali
> > >
> > >
> > >
> > > --
> > > Christian Ehrhardt
> > > Staff Engineer, Ubuntu Server
> > > Canonical Ltd
> >
> > Thanks again Christian,
> >
> > Ali
>
> Thanks,
> Ali



-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd

  reply	other threads:[~2021-07-05 11:19 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-17  6:38 [dpdk-dev] 19.11.9 patches review and test Christian Ehrhardt
2021-06-17 17:46 ` Stokes, Ian
2021-06-24 11:05 ` Yu, PingX
2021-07-01 15:50 ` Ali Alnubani
2021-07-02  6:19   ` Christian Ehrhardt
2021-07-02  7:42   ` Christian Ehrhardt
2021-07-05  6:35     ` Ali Alnubani
2021-07-05 10:44       ` Ali Alnubani
2021-07-05 11:18         ` Christian Ehrhardt [this message]
2021-07-05  4:14 ` Pei Zhang
2021-07-05  5:24   ` Christian Ehrhardt
  -- strict thread matches above, loose matches on Subject: below --
2021-07-05 13:23 Christian Ehrhardt
2021-07-06  2:45 ` Yu, PingX
2021-07-14  7:12 ` Jiang, YuX
2021-07-15  9:13 ` Ali Alnubani
2021-06-02 15:00 Christian Ehrhardt
2021-06-03  5:48 ` Christian Ehrhardt

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=CAATJJ0J6bDo1wLh73EkjQrTsSuP3+a71CrGyzLnxVTny_wz_uQ@mail.gmail.com \
    --to=christian.ehrhardt@canonical.com \
    --cc=Abhishek.Marathe@microsoft.com \
    --cc=akhil.goyal@nxp.com \
    --cc=alialnu@nvidia.com \
    --cc=asafp@nvidia.com \
    --cc=benjamin.walker@intel.com \
    --cc=bluca@debian.org \
    --cc=dev@dpdk.org \
    --cc=drc@linux.vnet.ibm.com \
    --cc=hariprasad.govindharajan@intel.com \
    --cc=hemant.agrawal@nxp.com \
    --cc=ian.stokes@intel.com \
    --cc=jerinj@marvell.com \
    --cc=john.mcnamara@intel.com \
    --cc=juhlee@microsoft.com \
    --cc=ktraynor@redhat.com \
    --cc=pezhang@redhat.com \
    --cc=pingx.yu@intel.com \
    --cc=qian.q.xu@intel.com \
    --cc=rasland@nvidia.com \
    --cc=stable@dpdk.org \
    --cc=thomas@monjalon.net \
    --cc=viacheslavo@nvidia.com \
    --cc=yuan.peng@intel.com \
    --cc=zhaoyan.chen@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.