[dpdk-dev] FW: 17.11.7-rc1 (LTS) patches review and test
Luca Boccassi
bluca at debian.org
Tue Sep 17 11:53:04 CEST 2019
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 at mellanox.com
> >; Ananyev, Konstantin <
> konstantin.ananyev at intel.com
> >;
> asafp at mellanox.com
>
> Cc: Xu, Qian Q <
> qian.q.xu at intel.com
> >; Lin, Xueqin <
> xueqin.lin at intel.com
> >; Chen, Zhaoyan <
> zhaoyan.chen at intel.com
> >; O'Hare, Cathal <
> cathal.ohare at intel.com
> >; Yigit, Ferruh <
> ferruh.yigit at intel.com
> >; Ma, LihongX <
> lihongx.ma at intel.com
> >; Li, WenjieX A <
> wenjiex.a.li at intel.com
> >; Wang, FengqinX <
> fengqinx.wang at intel.com
> >; Peng, Yuan <
> yuan.peng at 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 at 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 at 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 at 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 at mellanox.com
> >; Ananyev, Konstantin <
> konstantin.ananyev at intel.com
> >;
> asafp at mellanox.com
>
> Cc: Xu, Qian Q <
> qian.q.xu at intel.com
> >; Lin, Xueqin <
> xueqin.lin at intel.com
> >; Chen, Zhaoyan <
> zhaoyan.chen at intel.com
> >; O'Hare, Cathal <
> cathal.ohare at intel.com
> >; Yigit, Ferruh <
> ferruh.yigit at intel.com
> >; Yu, PingX <
> pingx.yu at intel.com
> >; Ma, LihongX <
> lihongx.ma at intel.com
> >; Li, WenjieX A <
> wenjiex.a.li at 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 at mellanox.com
> >; Ananyev, Konstantin <
> konstantin.ananyev at intel.com
> >
> Cc: Xu, Qian Q <
> qian.q.xu at intel.com
> >; Lin, Xueqin <
> xueqin.lin at intel.com
> >; Chen, Zhaoyan <
> zhaoyan.chen at intel.com
> >; O'Hare, Cathal <
> cathal.ohare at intel.com
> >; Yigit, Ferruh <
> ferruh.yigit at intel.com
> >; Yu, PingX <
> pingx.yu at intel.com
> >; Ma, LihongX <
> lihongx.ma at intel.com
> >; Li, WenjieX A <
> wenjiex.a.li at 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 at 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 at arm.com
> >
> Signed-off-by: Joyce Kong <
> joyce.kong at arm.com
> >
> Acked-by: Konstantin Ananyev <
> konstantin.ananyev at 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 at mellanox.com
> ]
> Sent: Friday, August 16, 2019 2:05 AM
> To: dpdk stable <
> stable at dpdk.org
> >
> Cc:
> dev at dpdk.org
> ;
> pezhang at redhat.com
> ;
> siliu at redhat.com
> ; Wang, FengqinX <
> fengqinx.wang at intel.com
> >; Chen, Zhaoyan <
> zhaoyan.chen at intel.com
> >; Xu, Qian Q <
> qian.q.xu at intel.com
> >;
> alialnu at mellanox.com
> ;
> rasland at 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
More information about the dev
mailing list