From: Luca Boccassi <bluca@debian.org>
To: "Peng, Yuan" <yuan.peng@intel.com>, "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] FW: 17.11.7-rc1 (LTS) patches review and test
Date: Wed, 18 Sep 2019 10:17:30 +0100 [thread overview]
Message-ID: <e826ef92d4291692d311d73ac80b9823f564ce07.camel@debian.org> (raw)
In-Reply-To: <295acfbe39e7276664681fbf4566e528307a6bef.camel@debian.org>
Hi,
Ping - this is blocking the 17.11.7 release.
Thanks!
On Tue, 2019-09-17 at 10:53 +0100, Luca Boccassi wrote:
> Hello,
>
> Do you have a reference for the patches for bug 1 and 2? Have they
> been
> sent to the mailing list?
>
> On Thu, 2019-09-12 at 08:52 +0000, Peng, Yuan wrote:
> > Update the latest test status here.
> >
> > -----Original Message-----
> > From: Yu, PingX
> > Sent: Thursday, September 12, 2019 2:37 PM
> > To: Yongseok Koh <
> > yskoh@mellanox.com
> >
> > > ; Ananyev, Konstantin <
> >
> > konstantin.ananyev@intel.com
> >
> > > ;
> >
> > asafp@mellanox.com
> >
> >
> > Cc: Xu, Qian Q <
> > qian.q.xu@intel.com
> >
> > > ; Lin, Xueqin <
> >
> > xueqin.lin@intel.com
> >
> > > ; Chen, Zhaoyan <
> >
> > zhaoyan.chen@intel.com
> >
> > > ; O'Hare, Cathal <
> >
> > cathal.ohare@intel.com
> >
> > > ; Yigit, Ferruh <
> >
> > ferruh.yigit@intel.com
> >
> > > ; Ma, LihongX <
> >
> > lihongx.ma@intel.com
> >
> > > ; Li, WenjieX A <
> >
> > wenjiex.a.li@intel.com
> >
> > > ; Wang, FengqinX <
> >
> > fengqinx.wang@intel.com
> >
> > > ; Peng, Yuan <
> >
> > yuan.peng@intel.com
> >
> >
> > Subject: RE: 17.11.7-rc1 (LTS) patches review and test
> >
> > All,
> > Update the latest test status here.
> > The critical bug has been fixed and we have re-tested all Intel key
> > features.
> >
> > Intel(R) Testing result lists here.
> > # Basic Intel(R) NIC testing
> > * PF(i40e): One bug found in 1st round test. 2nd round test pass
> > after fixing the bug. (see Bug1)
> > * PF(ixgbe): the same as PF(i40e).
> > * VF: Pass and no issue.
> > * Build or compile: One bug found in 1st round test. 2nd round test
> > pass after fixing the bug. (see Bug2)
> > * Intel NIC single core/NIC performance: Pass and no issue.
> >
> > #Basic cryptodev and virtio testing
> > * vhost/virtio basic loopback, PVP and performance test: pass.
> > * cryptodev: 3 bugs are found(see bug statistic in below). (See
> > Bug3,4 and 5)
> >
> > Bugs statistic:
> > Bug1: make -C test/ failed. It blocked about 60 test cases for
> > both
> > PF(i40e) and PF(ixgbe). Resolved by adding "#include <inttypes.h>"
> > into test_rwlock.c. Please decide whether submit a patch to DPDK or
> > not.
> > Bug2: compile error on fc30. Patch is provided and validated pass.
> > Need to decide whether merge into DPDK or not.
> > Bug3: L2fwd: test_qat_h_AES_XCBC_MAC_auto failed. No progress yet.
> > commit 39fb9500c42752a537588f7a203433ec4737fa1e
> > Author: Arek Kusztal <
> > arkadiuszx.kusztal@intel.com
> >
> >
> > Date: Wed Dec 12 20:59:02 2018 +0100
> >
> > Bug4: Ipsec-gw_test forwarding failed with null crypto QAT. No
> > progress yet.
> > commit 39fb9500c42752a537588f7a203433ec4737fa1e
> > Author: Arek Kusztal <
> > arkadiuszx.kusztal@intel.com
> >
> >
> > Date: Wed Dec 12 20:59:02 2018 +0100
> >
> > Bug5: Crypto: cryptodev_qat_autotest test failed. No progress yet.
> > 39fb9500c42752a537588f7a203433ec4737fa1e is the first bad commit
> > commit 39fb9500c42752a537588f7a203433ec4737fa1e
> > Author: Arek Kusztal <
> > arkadiuszx.kusztal@intel.com
> >
> >
> > Date: Wed Dec 12 20:59:02 2018 +0100
> >
> > Regards,
> > Yu Ping
> >
> >
> > -----Original Message-----
> > From: Wang, FengqinX
> > Sent: Monday, August 26, 2019 3:12 PM
> > To: Yongseok Koh <
> > yskoh@mellanox.com
> >
> > > ; Ananyev, Konstantin <
> >
> > konstantin.ananyev@intel.com
> >
> > > ;
> >
> > asafp@mellanox.com
> >
> >
> > Cc: Xu, Qian Q <
> > qian.q.xu@intel.com
> >
> > > ; Lin, Xueqin <
> >
> > xueqin.lin@intel.com
> >
> > > ; Chen, Zhaoyan <
> >
> > zhaoyan.chen@intel.com
> >
> > > ; O'Hare, Cathal <
> >
> > cathal.ohare@intel.com
> >
> > > ; Yigit, Ferruh <
> >
> > ferruh.yigit@intel.com
> >
> > > ; Yu, PingX <
> >
> > pingx.yu@intel.com
> >
> > > ; Ma, LihongX <
> >
> > lihongx.ma@intel.com
> >
> > > ; Li, WenjieX A <
> >
> > wenjiex.a.li@intel.com
> >
> >
> > Subject: RE: 17.11.7-rc1 (LTS) patches review and test
> >
> > Add asafp in the loop due to Yongseok is longer working at
> > Mellanox.
> >
> > BRs, Vicky
> >
> > -----Original Message-----
> > From: Wang, FengqinX
> > Sent: Monday, August 26, 2019 3:07 PM
> > To: 'Yongseok Koh' <
> > yskoh@mellanox.com
> >
> > > ; Ananyev, Konstantin <
> >
> > konstantin.ananyev@intel.com
> >
> >
> > Cc: Xu, Qian Q <
> > qian.q.xu@intel.com
> >
> > > ; Lin, Xueqin <
> >
> > xueqin.lin@intel.com
> >
> > > ; Chen, Zhaoyan <
> >
> > zhaoyan.chen@intel.com
> >
> > > ; O'Hare, Cathal <
> >
> > cathal.ohare@intel.com
> >
> > > ; Yigit, Ferruh <
> >
> > ferruh.yigit@intel.com
> >
> > > ; Yu, PingX <
> >
> > pingx.yu@intel.com
> >
> > > ; Ma, LihongX <
> >
> > lihongx.ma@intel.com
> >
> > > ; Li, WenjieX A <
> >
> > wenjiex.a.li@intel.com
> >
> >
> > Subject: RE: 17.11.7-rc1 (LTS) patches review and test
> >
> > Hi Yongseok, Konstantin,
> >
> > There is one critical compile issue found, and below is the bad
> > commit related information, this issue has blocked most of unit
> > tests
> > which have used dpdk/test/test, would you please help to fix this
> > issue ASAP?
> >
> > commit 29e08d740ffc1e0ed9e7a9bd25fb95714fac1070
> > Author: Joyce Kong <
> > joyce.kong@arm.com
> >
> >
> > Date: Thu Jul 25 14:50:21 2019 +0800
> >
> > test/rwlock: benchmark on all available cores
> >
> > [ backported from upstream commit
> > fe252fb695efa9deb95f2e6b7baf6f805996a5b0 ]
> >
> > Add performance test on all available cores to benchmark
> > the scaling up performance of rw_lock.
> >
> > Fixes: af75078fece3 ("first public release")
> >
> > Suggested-by: Gavin Hu <
> > gavin.hu@arm.com
> >
> >
> > Signed-off-by: Joyce Kong <
> > joyce.kong@arm.com
> >
> >
> > Acked-by: Konstantin Ananyev <
> > konstantin.ananyev@intel.com
> >
> >
> >
> > And below is the error message output:
> >
> > test_rwlock.c: In function 'load_loop_fn':
> > test_rwlock.c:126:38: error: expected ')' before 'PRIu64'
> > printf("Core [%u] rwlock_data = %"PRIu64"\n", ^~~~~~
> > )
> > test_rwlock.c:126:19: error: format '%u' expects a matching
> > 'unsigned
> > int' argument [-Werror=format=] printf("Core [%u] rwlock_data =
> > %"PRIu64"\n", ~^
> > test_rwlock.c:126:36: error: spurious trailing '%' in format [-
> > Werror=format=] printf("Core [%u] rwlock_data = %"PRIu64"\n", ^
> > test_rwlock.c: In function 'test_rwlock_perf':
> > test_rwlock.c:160:31: error: expected ')' before 'PRIu64'
> > printf("Core [%u] count = %"PRIu64"\n", i, lock_count[i]); ^~~~~~
> > )
> > test_rwlock.c:160:18: error: format '%u' expects a matching
> > 'unsigned
> > int' argument [-Werror=format=] printf("Core [%u] count =
> > %"PRIu64"\n", i, lock_count[i]); ~^
> > test_rwlock.c:160:29: error: spurious trailing '%' in format [-
> > Werror=format=] printf("Core [%u] count = %"PRIu64"\n", i,
> > lock_count[i]); ^
> > test_rwlock.c:164:26: error: expected ')' before 'PRIu64'
> > printf("Total count = %"PRIu64"\n", total); ^~~~~~
> > )
> > test_rwlock.c:164:24: error: spurious trailing '%' in format [-
> > Werror=format=] printf("Total count = %"PRIu64"\n", total); ^
> > cc1: all warnings being treated as errors
> >
> > BRs, Vicky
> >
> > -----Original Message-----
> > From: Yongseok Koh [mailto:
> > yskoh@mellanox.com
> >
> > ]
> > Sent: Friday, August 16, 2019 2:05 AM
> > To: dpdk stable <
> > stable@dpdk.org
> >
> >
> > Cc:
> > dev@dpdk.org
> >
> > ;
> > pezhang@redhat.com
> >
> > ;
> > siliu@redhat.com
> >
> > ; Wang, FengqinX <
> > fengqinx.wang@intel.com
> >
> > > ; Chen, Zhaoyan <
> >
> > zhaoyan.chen@intel.com
> >
> > > ; Xu, Qian Q <
> >
> > qian.q.xu@intel.com
> >
> > > ;
> >
> > alialnu@mellanox.com
> >
> > ;
> > rasland@mellanox.com
> >
> >
> > Subject: 17.11.7-rc1 (LTS) patches review and test
> >
> > Hi all,
> >
> > Here is a list of patches targeted for LTS release 17.11.7. Please
> > help review and test. The planned date for the final release is Aug
> > 23, Before that, please shout if anyone has objections with these
> > patches being applied.
> >
> > Also for the companies committed to running regression tests,
> > please
> > run the tests and report any issue before the release date.
> >
> > A release candidate tarball can be found at:
> >
> >
> > https://dpdk.org/browse/dpdk-stable/tag/?id=v17.11.7-rc1
> >
> >
> >
> > These patches are located at branch 17.11 of dpdk-stable repo:
> >
> > https://dpdk.org/browse/dpdk-stable/
> >
> >
> >
> >
> > Thanks,
> > Yongseok
> >
> > ---
> >
> > Aaron Conole (1):
> > net/virtio: fix dangling pointer on failure
> >
> > Agalya Babu RadhaKrishnan (1):
> > doc: fix interactive commands in testpmd guide
> >
> > Akhil Goyal (1):
> > crypto/dpaa2_sec: fix session clearing
> >
> > Alejandro Lucero (1):
> > net/nfp: fix RSS query
> >
> > Anatoly Burakov (2):
> > eal: improve musl compatibility of string functions
> > malloc: fix documentation of realloc function
> >
> > Andrew Rybchenko (2):
> > net/sfc: remove control path logging from Rx queue count
> > app/testpmd: set fixed flag for exact link speed
> >
> > Andrius Sirvys (3):
> > net/virtio: fix duplicate naming of include guard
> > bitrate: fix unchecked return value
> > drivers/net: fix shifting 32-bit signed variable 31 times
> >
> > Anoob Joseph (1):
> > cryptodev: fix driver name comparison
> >
> > Beilei Xing (1):
> > net/i40e: fix link speed for X722
> >
> > Bernard Iremonger (1):
> > examples/ipsec-secgw: fix debug logs
> >
> > Bill Hong (1):
> > net: fix Tx VLAN flag for offload emulation
> >
> > Bruce Richardson (9):
> > eal: support strlcat function
> > net/bonding: fix buffer length when printing strings
> > examples/l2fwd-cat: fix build on FreeBSD
> > app/crypto-perf: check range of socket id
> > examples/vhost_scsi: fix null-check for parameter
> > net/i40e: fix dereference before null check in mbuf release
> > net/ixgbe: fix warning with GCC 9
> > bus/fslmc: fix warning with GCC 9
> > eal: tighten permissions on shared memory files
> >
> > Chaitanya Babu Talluri (2):
> > drivers/net: fix possible overflow using strlcat
> > cfgfile: replace strcat with strlcat
> >
> > David Christensen (2):
> > maintainers: update for IBM POWER
> > examples/vm_power_manager: fix PMD specific code
> >
> > David Marchand (7):
> > eal: fix check when retrieving current CPU affinity
> > eal: remove dead code in core list parsing
> > app/testpmd: remove unused field from port struct
> > doc: fix examples in bonding guide
> > net/bonding: fix port id types
> > net/bonding: fix queue index types
> > app/testpmd: remove useless casts on statistics
> >
> > Davide Caratti (1):
> > net/bnxt: support IOVA VA mode
> >
> > Dekel Peled (8):
> > net/mlx5: fix hex dump of error completion
> > net/mlx5: fix sync when handling Tx completions
> > eal/ppc: fix global memory barrier
> > eal: fix typo in comment of vector function
> > doc: fix links to doxygen and sphinx sites
> > app/testpmd: fix typo in comment
> > net/mlx5: fix comments mixing Rx and Tx
> > net/mlx5: fix release of Rx queue object
> >
> > Fan Zhang (1):
> > examples/ipsec-secgw: fix AES-CTR block size
> >
> > Ferruh Yigit (5):
> > kni: fix build with Linux 5.1
> > build: fix crash by disabling AVX512 with binutils 2.31
> > net/ring: fix return value check
> > net/kni: fix return value check
> > app/testpmd: revert fixed flag for exact link speed
> >
> > Gage Eads (1):
> > ring: fix namesize macro documentation block
> >
> > Gavin Hu (3):
> > test/spinlock: remove delay for correct benchmarking
> > test/spinlock: amortize the cost of getting time
> > spinlock: reimplement with atomic one-way barrier
> >
> > Hari Kumar Vemula (1):
> > net/bonding: fix reset active slave
> >
> > Harry van Haaren (1):
> > event/sw: fix enqueue checks in self-test
> >
> > Herakliusz Lipiec (1):
> > doc: fix broken link in LPM guide
> >
> > Hyong Youb Kim (4):
> > net/bonding: avoid warning for invalid port
> > net/enic: fix flow director SCTP matching
> > net/enic: fix SCTP match for flow API
> > net/enic: check for unsupported flow item types
> >
> > Igor Romanov (1):
> > app/testpmd: fix stdout flush after printing stats
> >
> > Ilya Maximets (2):
> > vhost: fix device leak on connection add failure
> > vhost: fix silent queue enabling with legacy guests
> >
> > Ivan Malov (1):
> > net/sfc: log port ID as 16-bit unsigned integer on panic
> >
> > Jerin Jacob (1):
> > mk: fix AVX512 disabled warning on non x86
> >
> > Jianfeng Tan (1):
> > drivers/net: do not use private ethdev data
> >
> > Joyce Kong (1):
> > test/rwlock: benchmark on all available cores
> >
> > Kevin Traynor (3):
> > net/qede: support IOVA VA mode
> > eal/linux: fix log levels for pagemap reading failure
> > power: fix frequency list buffer validation
> >
> > Liang Zhang (1):
> > net/bonding: fix LACP negotiation
> >
> > Luo Gaoliang (1):
> > kni: fix build on RHEL8
> >
> > Marcin Smoczynski (1):
> > examples/ipsec-secgw: fix build error log
> >
> > Natanael Copa (3):
> > bus/fslmc: remove unused include of error.h
> > bus/fslmc: fix build with musl libc
> > app/test: fix build with musl libc
> >
> > Pallantla Poornima (3):
> > app/test: fix sprintf with strlcat
> > test/distributor: replace sprintf with strlcpy
> > test/hash: replace sprintf with snprintf
> >
> > Qi Zhang (1):
> > net/i40e: fix time sync for 25G
> >
> > Radu Nicolau (3):
> > net/null: add MAC address setting fake operation
> > test/virtual_pmd: add MAC address setting fake op
> > test/bonding: assign non-zero MAC to null devices
> >
> > Rahul Lakkireddy (1):
> > net/cxgbe: update Chelsio T5/T6 NIC device ids
> >
> > Rami Rosen (8):
> > net/virtio: remove forward declaration
> > mbuf: fix a typo
> > doc: fix a minor typo in testpmd guide
> > doc: remove reference to rte.doc.mk in programmers guide
> > examples/ethtool: fix two typos
> > doc: fix link in Linux getting started guide
> > app/testpmd: fix a typo in log message
> > ethdev: fix a typo
> >
> > Rasesh Mody (2):
> > net/bnx2x: fix race for periodic flags
> > net/bnx2x: fix optic module verification
> >
> > Shahaf Shuler (1):
> > net/mlx5: fix packet inline on Tx queue wraparound
> >
> > Shahed Shaikh (5):
> > net/bnx2x: fix segfaults due to stale interrupt status
> > net/bnx2x: fix memory leak
> > net/bnx2x: fix ramrod timeout
> > net/bnx2x: fix DMAE timeout
> > net/qede: fix Tx packet prepare for tunnel packets
> >
> > Shreyansh Jain (1):
> > bus/dpaa: fix Rx discard register mask
> >
> > Stephen Hemminger (8):
> > net/octeontx: fix vdev name
> > net/bnxt: fix Rx VLAN offload flags
> > net/virtio: remove useless condition
> > ring: fix an error message
> > net/ring: avoid hard-coded length
> > net/ring: use calloc style where appropriate
> > net/ring: check length of ring name
> > net/virtio: fix buffer leak on VLAN insert
> >
> > Thomas Monjalon (2):
> > mk: fix build of shared library with libbsd
> > bus/vdev: fix debug message on probing
> >
> > Tiwei Bie (4):
> > net/virtio: add barrier in interrupt enable
> > vhost: restore mbuf first when freeing zmbuf
> > vhost: fix potential use-after-free for zero copy mbuf
> > vhost: fix potential use-after-free for memory region
> >
> > Vishal Kulkarni (1):
> > net/cxgbe: fix missing checksum flags and packet type
> >
> > Xiao Wang (1):
> > net/fm10k: fix VLAN strip offload flag
> >
> > Yipeng Wang (1):
> > hash: fix doc about thread/process safety
> >
> > Yongseok Koh (4):
> > net/mlx5: fix instruction hotspot on replenishing Rx buffer
> > net/mlx5: check Tx queue size overflow
> > net/mlx5: fix max number of queues for NEON Tx
> > doc: fix typo in mlx5 guide
> >
> > Yunjian Wang (1):
> > net/ixgbe: fix crash on remove
> >
> >
--
Kind regards,
Luca Boccassi
next prev parent reply other threads:[~2019-09-18 9:17 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-15 18:05 [dpdk-dev] " Yongseok Koh
2019-08-19 6:37 ` Pei Zhang
2019-08-25 10:30 ` Ali Alnubani
2019-08-29 16:10 ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
[not found] ` <4A3FBC67CD7DCF48BECD235A28B050330ADA95BA@SHSMSX101.ccr.corp.intel.com>
2019-08-30 12:58 ` Ferruh Yigit
[not found] ` <4A3FBC67CD7DCF48BECD235A28B050330ADA8312@SHSMSX101.ccr.corp.intel.com>
[not found] ` <FD56CD4A6C7E9841B5CD62946D45F34C01B18CF1@SHSMSX104.ccr.corp.intel.com>
2019-09-12 8:52 ` [dpdk-dev] FW: " Peng, Yuan
2019-09-17 9:53 ` Luca Boccassi
2019-09-18 9:17 ` Luca Boccassi [this message]
2019-09-18 10:03 ` Yu, PingX
2019-09-24 11:07 ` Luca Boccassi
2019-09-24 16:36 ` Ferruh Yigit
2019-09-24 17:15 ` Kevin Traynor
2019-09-24 17:25 ` Luca Boccassi
2019-09-24 20:08 ` Luca Boccassi
2019-09-26 7:34 ` Yu, PingX
2019-09-26 7:38 ` Ferruh Yigit
2019-09-26 8:15 ` Yu, PingX
2019-09-26 8:19 ` Luca Boccassi
2019-09-12 8:59 ` [dpdk-dev] " Peng, Yuan
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=e826ef92d4291692d311d73ac80b9823f564ce07.camel@debian.org \
--to=bluca@debian.org \
--cc=dev@dpdk.org \
--cc=yuan.peng@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).