On 9/24/2019 12:07 PM, Luca Boccassi wrote: > Ping - any updates on these patches? The release of 17.11.7 is > currently on hold waiting for a resolution here.
Hi Luca, Test team provided some internal details: 1- Build issue on f30, A patch has been sent to stable mail list to fix it, not sure if it has been merged: https://mails.dpdk.org/archives/stable/2019-September/017122.html 2- "make -C test/" fails. "Fix looks to be to add "#include <inttypes.h>" into test_rwlock.c" Not sure if there is a patch for this. 3- l2fwd-crypto, ERROR: Test Case test_qat_h_AES_XCBC_MAC_auto Result FAILED Work in progress 4- ipsec-secgw, forwarding failed with null crypto QAT Work in progress 5- Unit test "cryptodev_qat_autotest" failed Issue not analyzed yet. > > Thanks! > > On Wed, 2019-09-18 at 10:03 +0000, Yu, PingX wrote: >> Luca, >> FYI. Currently there isn't reference for bug1 and bug2 and also >> haven't send to mailing list yet. These bugs should be fixed before >> release as they will block the basic PF test. >> >> Regards, >> Yu Ping >> >> >> -----Original Message----- >> From: dev [mailto: >> dev-boun...@dpdk.org >> ] On Behalf Of Luca Boccassi >> Sent: Wednesday, September 18, 2019 5:18 PM >> To: Peng, Yuan < >> yuan.p...@intel.com >>> ; >> dev@dpdk.org >> >> Subject: Re: [dpdk-dev] FW: 17.11.7-rc1 (LTS) patches review and test >> >> 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 < >>>> ys...@mellanox.com >>>> >>>> >>>>> ; Ananyev, Konstantin < >>>> >>>> konstantin.anan...@intel.com >>>> >>>> >>>>> ; >>>> >>>> as...@mellanox.com >>>> >>>> >>>> >>>> Cc: Xu, Qian Q < >>>> qian.q...@intel.com >>>> >>>> >>>>> ; Lin, Xueqin < >>>> >>>> xueqin....@intel.com >>>> >>>> >>>>> ; Chen, Zhaoyan < >>>> >>>> zhaoyan.c...@intel.com >>>> >>>> >>>>> ; O'Hare, Cathal < >>>> >>>> cathal.oh...@intel.com >>>> >>>> >>>>> ; Yigit, Ferruh < >>>> >>>> ferruh.yi...@intel.com >>>> >>>> >>>>> ; Ma, LihongX < >>>> >>>> lihongx...@intel.com >>>> >>>> >>>>> ; Li, WenjieX A < >>>> >>>> wenjiex.a...@intel.com >>>> >>>> >>>>> ; Wang, FengqinX < >>>> >>>> fengqinx.w...@intel.com >>>> >>>> >>>>> ; Peng, Yuan < >>>> >>>> yuan.p...@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.kusz...@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.kusz...@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.kusz...@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 < >>>> ys...@mellanox.com >>>> >>>> >>>>> ; Ananyev, Konstantin < >>>> >>>> konstantin.anan...@intel.com >>>> >>>> >>>>> ; >>>> >>>> as...@mellanox.com >>>> >>>> >>>> >>>> Cc: Xu, Qian Q < >>>> qian.q...@intel.com >>>> >>>> >>>>> ; Lin, Xueqin < >>>> >>>> xueqin....@intel.com >>>> >>>> >>>>> ; Chen, Zhaoyan < >>>> >>>> zhaoyan.c...@intel.com >>>> >>>> >>>>> ; O'Hare, Cathal < >>>> >>>> cathal.oh...@intel.com >>>> >>>> >>>>> ; Yigit, Ferruh < >>>> >>>> ferruh.yi...@intel.com >>>> >>>> >>>>> ; Yu, PingX < >>>> >>>> pingx...@intel.com >>>> >>>> >>>>> ; Ma, LihongX < >>>> >>>> lihongx...@intel.com >>>> >>>> >>>>> ; Li, WenjieX A < >>>> >>>> wenjiex.a...@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' < >>>> ys...@mellanox.com >>>> >>>> >>>>> ; Ananyev, Konstantin < >>>> >>>> konstantin.anan...@intel.com >>>> >>>> >>>> >>>> Cc: Xu, Qian Q < >>>> qian.q...@intel.com >>>> >>>> >>>>> ; Lin, Xueqin < >>>> >>>> xueqin....@intel.com >>>> >>>> >>>>> ; Chen, Zhaoyan < >>>> >>>> zhaoyan.c...@intel.com >>>> >>>> >>>>> ; O'Hare, Cathal < >>>> >>>> cathal.oh...@intel.com >>>> >>>> >>>>> ; Yigit, Ferruh < >>>> >>>> ferruh.yi...@intel.com >>>> >>>> >>>>> ; Yu, PingX < >>>> >>>> pingx...@intel.com >>>> >>>> >>>>> ; Ma, LihongX < >>>> >>>> lihongx...@intel.com >>>> >>>> >>>>> ; Li, WenjieX A < >>>> >>>> wenjiex.a...@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.k...@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...@arm.com >>>> >>>> >>>> >>>> Signed-off-by: Joyce Kong < >>>> joyce.k...@arm.com >>>> >>>> >>>> >>>> Acked-by: Konstantin Ananyev < >>>> konstantin.anan...@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: >>>> ys...@mellanox.com >>>> >>>> >>>> ] >>>> Sent: Friday, August 16, 2019 2:05 AM >>>> To: dpdk stable < >>>> sta...@dpdk.org >>>> >>>> >>>> >>>> Cc: >>>> dev@dpdk.org >>>> >>>> >>>> ; >>>> pezh...@redhat.com >>>> >>>> >>>> ; >>>> si...@redhat.com >>>> >>>> >>>> ; Wang, FengqinX < >>>> fengqinx.w...@intel.com >>>> >>>> >>>>> ; Chen, Zhaoyan < >>>> >>>> zhaoyan.c...@intel.com >>>> >>>> >>>>> ; Xu, Qian Q < >>>> >>>> qian.q...@intel.com >>>> >>>> >>>>> ; >>>> >>>> alia...@mellanox.com >>>> >>>> >>>> ; >>>> rasl...@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 >>
