From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3BAFAA2EFC for ; Wed, 18 Sep 2019 11:17:37 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B007E1C1B6; Wed, 18 Sep 2019 11:17:35 +0200 (CEST) Received: from mail-wm1-f66.google.com (mail-wm1-f66.google.com [209.85.128.66]) by dpdk.org (Postfix) with ESMTP id DA9031C1B0 for ; Wed, 18 Sep 2019 11:17:34 +0200 (CEST) Received: by mail-wm1-f66.google.com with SMTP id p7so1670755wmp.4 for ; Wed, 18 Sep 2019 02:17:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:date:in-reply-to :references:content-transfer-encoding:user-agent:mime-version; bh=PpYcdrJ+Njrh9t71BeIm8cy60OD+N/MBmjDli1BXk0E=; b=mCJkt2um9PA6NSU8QvP6wdIV0fqQdtzgVjUYSyId1Sjy/oJhNsULz+jDSTnAdazJY5 VrOeAfpjEKa5Wen8+wLhoTIL7KyqFVYaSfhVlCGZtPligcKbbVnUj5YTsrC6VUNDmRjK LMxYdoBeiXF6D1qJmi5s/wS1NFT4Yf/K50PgJX5x+rmOx61h8oZggexVj/zv0n+lJTq/ cgA/LhQjqPtxcHoLrLHF50QehjzRwZkQVh8uNMYBE4g3VRkoIcpJQOfUrGLqb3BevOFY mB+mYY3HRHRVWLm6Zz/Gmna098mnuhy/aPlVGyaveWbzXilpQIrl8Ynd2/Oj/Kbb8VS0 +yFA== X-Gm-Message-State: APjAAAWohYyVsXmCcK+c41vjv8GHVVZoL0JBHc3WDb+tqOeukd8/vh34 NOdKeZMog5BbgCg+ZSzpQ4WekB6q9cc= X-Google-Smtp-Source: APXvYqwfcmpneraeVlNuC0ctN1LJsg7Sx4BCDEMH2KV4ocifVB0mRgIXFRzcFJVINgkgkBtfTVJnwQ== X-Received: by 2002:a1c:1a4b:: with SMTP id a72mr1958375wma.44.1568798254162; Wed, 18 Sep 2019 02:17:34 -0700 (PDT) Received: from localhost ([176.12.107.132]) by smtp.gmail.com with ESMTPSA id u10sm10131470wrg.55.2019.09.18.02.17.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Sep 2019 02:17:33 -0700 (PDT) Message-ID: From: Luca Boccassi To: "Peng, Yuan" , "dev@dpdk.org" Date: Wed, 18 Sep 2019 10:17:30 +0100 In-Reply-To: <295acfbe39e7276664681fbf4566e528307a6bef.camel@debian.org> References: <20190815180508.14043-1-yskoh@mellanox.com> <4A3FBC67CD7DCF48BECD235A28B050330ADA8312@SHSMSX101.ccr.corp.intel.com> <67D543A150B29E4CAAE53918F64EDAEA3775D71C@SHSMSX103.ccr.corp.intel.com> <295acfbe39e7276664681fbf4566e528307a6bef.camel@debian.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.30.5-1.1 MIME-Version: 1.0 Subject: Re: [dpdk-dev] FW: 17.11.7-rc1 (LTS) patches review and test X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, Ping - this is blocking the 17.11.7 release. Thanks! On Tue, 2019-09-17 at 10:53 +0100, Luca Boccassi wrote: > Hello, >=20 > Do you have a reference for the patches for bug 1 and 2? Have they > been > sent to the mailing list? >=20 > On Thu, 2019-09-12 at 08:52 +0000, Peng, Yuan wrote: > > Update the latest test status here. > >=20 > > -----Original Message----- > > From: Yu, PingX=20 > > Sent: Thursday, September 12, 2019 2:37 PM > > To: Yongseok Koh < > > yskoh@mellanox.com > >=20 > > > ; Ananyev, Konstantin < > >=20 > > konstantin.ananyev@intel.com > >=20 > > > ;=20 > >=20 > > asafp@mellanox.com > >=20 > >=20 > > Cc: Xu, Qian Q < > > qian.q.xu@intel.com > >=20 > > > ; Lin, Xueqin < > >=20 > > xueqin.lin@intel.com > >=20 > > > ; Chen, Zhaoyan < > >=20 > > zhaoyan.chen@intel.com > >=20 > > > ; O'Hare, Cathal < > >=20 > > cathal.ohare@intel.com > >=20 > > > ; Yigit, Ferruh < > >=20 > > ferruh.yigit@intel.com > >=20 > > > ; Ma, LihongX < > >=20 > > lihongx.ma@intel.com > >=20 > > > ; Li, WenjieX A < > >=20 > > wenjiex.a.li@intel.com > >=20 > > > ; Wang, FengqinX < > >=20 > > fengqinx.wang@intel.com > >=20 > > > ; Peng, Yuan < > >=20 > > yuan.peng@intel.com > >=20 > >=20 > > Subject: RE: 17.11.7-rc1 (LTS) patches review and test > >=20 > > All, > > Update the latest test status here. > > The critical bug has been fixed and we have re-tested all Intel key > > features.=20 > >=20 > > 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. > >=20 > > #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) > >=20 > > Bugs statistic: > > Bug1: make -C test/ failed. It blocked about 60 test cases for > > both > > PF(i40e) and PF(ixgbe). Resolved by adding "#include " > > 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 > >=20 > >=20 > > Date: Wed Dec 12 20:59:02 2018 +0100 > >=20 > > Bug4: Ipsec-gw_test forwarding failed with null crypto QAT. No > > progress yet. > > commit 39fb9500c42752a537588f7a203433ec4737fa1e > > Author: Arek Kusztal < > > arkadiuszx.kusztal@intel.com > >=20 > >=20 > > Date: Wed Dec 12 20:59:02 2018 +0100 > >=20 > > 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 > >=20 > >=20 > > Date: Wed Dec 12 20:59:02 2018 +0100 > >=20 > > Regards, > > Yu Ping > >=20 > >=20 > > -----Original Message----- > > From: Wang, FengqinX=20 > > Sent: Monday, August 26, 2019 3:12 PM > > To: Yongseok Koh < > > yskoh@mellanox.com > >=20 > > > ; Ananyev, Konstantin < > >=20 > > konstantin.ananyev@intel.com > >=20 > > > ;=20 > >=20 > > asafp@mellanox.com > >=20 > >=20 > > Cc: Xu, Qian Q < > > qian.q.xu@intel.com > >=20 > > > ; Lin, Xueqin < > >=20 > > xueqin.lin@intel.com > >=20 > > > ; Chen, Zhaoyan < > >=20 > > zhaoyan.chen@intel.com > >=20 > > > ; O'Hare, Cathal < > >=20 > > cathal.ohare@intel.com > >=20 > > > ; Yigit, Ferruh < > >=20 > > ferruh.yigit@intel.com > >=20 > > > ; Yu, PingX < > >=20 > > pingx.yu@intel.com > >=20 > > > ; Ma, LihongX < > >=20 > > lihongx.ma@intel.com > >=20 > > > ; Li, WenjieX A < > >=20 > > wenjiex.a.li@intel.com > >=20 > >=20 > > Subject: RE: 17.11.7-rc1 (LTS) patches review and test > >=20 > > Add asafp in the loop due to Yongseok is longer working at > > Mellanox. > >=20 > > BRs, Vicky > >=20 > > -----Original Message----- > > From: Wang, FengqinX=20 > > Sent: Monday, August 26, 2019 3:07 PM > > To: 'Yongseok Koh' < > > yskoh@mellanox.com > >=20 > > > ; Ananyev, Konstantin < > >=20 > > konstantin.ananyev@intel.com > >=20 > >=20 > > Cc: Xu, Qian Q < > > qian.q.xu@intel.com > >=20 > > > ; Lin, Xueqin < > >=20 > > xueqin.lin@intel.com > >=20 > > > ; Chen, Zhaoyan < > >=20 > > zhaoyan.chen@intel.com > >=20 > > > ; O'Hare, Cathal < > >=20 > > cathal.ohare@intel.com > >=20 > > > ; Yigit, Ferruh < > >=20 > > ferruh.yigit@intel.com > >=20 > > > ; Yu, PingX < > >=20 > > pingx.yu@intel.com > >=20 > > > ; Ma, LihongX < > >=20 > > lihongx.ma@intel.com > >=20 > > > ; Li, WenjieX A < > >=20 > > wenjiex.a.li@intel.com > >=20 > >=20 > > Subject: RE: 17.11.7-rc1 (LTS) patches review and test > >=20 > > Hi Yongseok, Konstantin, > >=20 > > 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? > >=20 > > commit 29e08d740ffc1e0ed9e7a9bd25fb95714fac1070 > > Author: Joyce Kong < > > joyce.kong@arm.com > >=20 > >=20 > > Date: Thu Jul 25 14:50:21 2019 +0800 > >=20 > > test/rwlock: benchmark on all available cores > >=20 > > [ backported from upstream commit > > fe252fb695efa9deb95f2e6b7baf6f805996a5b0 ] > >=20 > > Add performance test on all available cores to benchmark > > the scaling up performance of rw_lock. > >=20 > > Fixes: af75078fece3 ("first public release") > >=20 > > Suggested-by: Gavin Hu < > > gavin.hu@arm.com > >=20 > >=20 > > Signed-off-by: Joyce Kong < > > joyce.kong@arm.com > >=20 > >=20 > > Acked-by: Konstantin Ananyev < > > konstantin.ananyev@intel.com > >=20 > >=20 > >=20 > > And below is the error message output: > >=20 > > test_rwlock.c: In function 'load_loop_fn': > > test_rwlock.c:126:38: error: expected ')' before 'PRIu64' > > printf("Core [%u] rwlock_data =3D %"PRIu64"\n", ^~~~~~ > > ) > > test_rwlock.c:126:19: error: format '%u' expects a matching > > 'unsigned > > int' argument [-Werror=3Dformat=3D] printf("Core [%u] rwlock_data =3D > > %"PRIu64"\n", ~^ > > test_rwlock.c:126:36: error: spurious trailing '%' in format [- > > Werror=3Dformat=3D] printf("Core [%u] rwlock_data =3D %"PRIu64"\n", ^ > > test_rwlock.c: In function 'test_rwlock_perf': > > test_rwlock.c:160:31: error: expected ')' before 'PRIu64' > > printf("Core [%u] count =3D %"PRIu64"\n", i, lock_count[i]); ^~~~~~ > > ) > > test_rwlock.c:160:18: error: format '%u' expects a matching > > 'unsigned > > int' argument [-Werror=3Dformat=3D] printf("Core [%u] count =3D > > %"PRIu64"\n", i, lock_count[i]); ~^ > > test_rwlock.c:160:29: error: spurious trailing '%' in format [- > > Werror=3Dformat=3D] printf("Core [%u] count =3D %"PRIu64"\n", i, > > lock_count[i]); ^ > > test_rwlock.c:164:26: error: expected ')' before 'PRIu64' > > printf("Total count =3D %"PRIu64"\n", total); ^~~~~~ > > ) > > test_rwlock.c:164:24: error: spurious trailing '%' in format [- > > Werror=3Dformat=3D] printf("Total count =3D %"PRIu64"\n", total); ^ > > cc1: all warnings being treated as errors > >=20 > > BRs, Vicky > >=20 > > -----Original Message----- > > From: Yongseok Koh [mailto: > > yskoh@mellanox.com > >=20 > > ] > > Sent: Friday, August 16, 2019 2:05 AM > > To: dpdk stable < > > stable@dpdk.org > >=20 > >=20 > > Cc:=20 > > dev@dpdk.org > >=20 > > ;=20 > > pezhang@redhat.com > >=20 > > ;=20 > > siliu@redhat.com > >=20 > > ; Wang, FengqinX < > > fengqinx.wang@intel.com > >=20 > > > ; Chen, Zhaoyan < > >=20 > > zhaoyan.chen@intel.com > >=20 > > > ; Xu, Qian Q < > >=20 > > qian.q.xu@intel.com > >=20 > > > ;=20 > >=20 > > alialnu@mellanox.com > >=20 > > ;=20 > > rasland@mellanox.com > >=20 > >=20 > > Subject: 17.11.7-rc1 (LTS) patches review and test > >=20 > > Hi all, > >=20 > > 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. > >=20 > > Also for the companies committed to running regression tests, > > please > > run the tests and report any issue before the release date. > >=20 > > A release candidate tarball can be found at: > >=20 > > =20 > > https://dpdk.org/browse/dpdk-stable/tag/?id=3Dv17.11.7-rc1 > >=20 > >=20 > >=20 > > These patches are located at branch 17.11 of dpdk-stable repo: > > =20 > > https://dpdk.org/browse/dpdk-stable/ > >=20 > >=20 > >=20 > >=20 > > Thanks, > > Yongseok > >=20 > > --- > >=20 > > Aaron Conole (1): > > net/virtio: fix dangling pointer on failure > >=20 > > Agalya Babu RadhaKrishnan (1): > > doc: fix interactive commands in testpmd guide > >=20 > > Akhil Goyal (1): > > crypto/dpaa2_sec: fix session clearing > >=20 > > Alejandro Lucero (1): > > net/nfp: fix RSS query > >=20 > > Anatoly Burakov (2): > > eal: improve musl compatibility of string functions > > malloc: fix documentation of realloc function > >=20 > > Andrew Rybchenko (2): > > net/sfc: remove control path logging from Rx queue count > > app/testpmd: set fixed flag for exact link speed > >=20 > > 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 > >=20 > > Anoob Joseph (1): > > cryptodev: fix driver name comparison > >=20 > > Beilei Xing (1): > > net/i40e: fix link speed for X722 > >=20 > > Bernard Iremonger (1): > > examples/ipsec-secgw: fix debug logs > >=20 > > Bill Hong (1): > > net: fix Tx VLAN flag for offload emulation > >=20 > > 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 > >=20 > > Chaitanya Babu Talluri (2): > > drivers/net: fix possible overflow using strlcat > > cfgfile: replace strcat with strlcat > >=20 > > David Christensen (2): > > maintainers: update for IBM POWER > > examples/vm_power_manager: fix PMD specific code > >=20 > > 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 > >=20 > > Davide Caratti (1): > > net/bnxt: support IOVA VA mode > >=20 > > 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 > >=20 > > Fan Zhang (1): > > examples/ipsec-secgw: fix AES-CTR block size > >=20 > > 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 > >=20 > > Gage Eads (1): > > ring: fix namesize macro documentation block > >=20 > > 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 > >=20 > > Hari Kumar Vemula (1): > > net/bonding: fix reset active slave > >=20 > > Harry van Haaren (1): > > event/sw: fix enqueue checks in self-test > >=20 > > Herakliusz Lipiec (1): > > doc: fix broken link in LPM guide > >=20 > > 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 > >=20 > > Igor Romanov (1): > > app/testpmd: fix stdout flush after printing stats > >=20 > > Ilya Maximets (2): > > vhost: fix device leak on connection add failure > > vhost: fix silent queue enabling with legacy guests > >=20 > > Ivan Malov (1): > > net/sfc: log port ID as 16-bit unsigned integer on panic > >=20 > > Jerin Jacob (1): > > mk: fix AVX512 disabled warning on non x86 > >=20 > > Jianfeng Tan (1): > > drivers/net: do not use private ethdev data > >=20 > > Joyce Kong (1): > > test/rwlock: benchmark on all available cores > >=20 > > Kevin Traynor (3): > > net/qede: support IOVA VA mode > > eal/linux: fix log levels for pagemap reading failure > > power: fix frequency list buffer validation > >=20 > > Liang Zhang (1): > > net/bonding: fix LACP negotiation > >=20 > > Luo Gaoliang (1): > > kni: fix build on RHEL8 > >=20 > > Marcin Smoczynski (1): > > examples/ipsec-secgw: fix build error log > >=20 > > 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 > >=20 > > Pallantla Poornima (3): > > app/test: fix sprintf with strlcat > > test/distributor: replace sprintf with strlcpy > > test/hash: replace sprintf with snprintf > >=20 > > Qi Zhang (1): > > net/i40e: fix time sync for 25G > >=20 > > 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 > >=20 > > Rahul Lakkireddy (1): > > net/cxgbe: update Chelsio T5/T6 NIC device ids > >=20 > > 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 > >=20 > > Rasesh Mody (2): > > net/bnx2x: fix race for periodic flags > > net/bnx2x: fix optic module verification > >=20 > > Shahaf Shuler (1): > > net/mlx5: fix packet inline on Tx queue wraparound > >=20 > > 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 > >=20 > > Shreyansh Jain (1): > > bus/dpaa: fix Rx discard register mask > >=20 > > 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 > >=20 > > Thomas Monjalon (2): > > mk: fix build of shared library with libbsd > > bus/vdev: fix debug message on probing > >=20 > > 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 > >=20 > > Vishal Kulkarni (1): > > net/cxgbe: fix missing checksum flags and packet type > >=20 > > Xiao Wang (1): > > net/fm10k: fix VLAN strip offload flag > >=20 > > Yipeng Wang (1): > > hash: fix doc about thread/process safety > >=20 > > 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 > >=20 > > Yunjian Wang (1): > > net/ixgbe: fix crash on remove > >=20 > >=20 --=20 Kind regards, Luca Boccassi