From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6D2E4C4CECD for ; Tue, 17 Sep 2019 09:53:09 +0000 (UTC) Received: from dpdk.org (dpdk.org [92.243.14.124]) by mail.kernel.org (Postfix) with ESMTP id CA3E8214AF for ; Tue, 17 Sep 2019 09:53:08 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CA3E8214AF Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=debian.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=dev-bounces@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D4A491BF09; Tue, 17 Sep 2019 11:53:07 +0200 (CEST) Received: from mail-wm1-f65.google.com (mail-wm1-f65.google.com [209.85.128.65]) by dpdk.org (Postfix) with ESMTP id 47F621BF05 for ; Tue, 17 Sep 2019 11:53:06 +0200 (CEST) Received: by mail-wm1-f65.google.com with SMTP id x2so2548835wmj.2 for ; Tue, 17 Sep 2019 02:53:06 -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=bwfDkIAMs06M1tt3gugTjZUaJBpu/wxO3a1x32kfRAk=; b=RGnTSlJ+0MA51qvWjAPeRY2kEPF+yUwWZ/OLGfJHKprP8VTtXStwqOpGEJuAXcabJI srcV48nViV81Pf8ygWjWCOIPVgZNjKzjCs7Mh4cDozTs5yopRg70LX6XvqkdT9+UjNB2 +7ctDCkNOixKKss+W8WWtD71U7vZPyQI5hSyD80R7YCZ+P+mE718vRt6pHUNgBPGe1wL S8lRgVIL1l6l4By9zCNxQaGsVdVJH8JltiBtlLJ3v2EzYsRS91N7O+YAGwbQCw9y/2kk UWRdCCwDk9Wu1Gl7l9kYmFQFadnfw+nAp5lhFUfWXAvD91FKUKFkrFXY84OP1NSDO2Of Z4Mg== X-Gm-Message-State: APjAAAV3bIj2g4LiHz+74NGQAuuvNRmJzhqb6BEhoa1QKjGCfTPazpaX VS58/MdQlV7fHZ5x7cJbDw4= X-Google-Smtp-Source: APXvYqxhzlNlmOVJy5Rbr2xJLrOwZXQoxfTZuUBnLM8H9sHT0kXzSBBudVk+oAzt2AorxFVBLUbV6w== X-Received: by 2002:a1c:4846:: with SMTP id v67mr2741146wma.120.1568713985633; Tue, 17 Sep 2019 02:53:05 -0700 (PDT) Received: from localhost ([2a01:4b00:f419:6f00:7a8e:ed70:5c52:ea3]) by smtp.gmail.com with ESMTPSA id o8sm2147047wmc.30.2019.09.17.02.53.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 17 Sep 2019 02:53:04 -0700 (PDT) Message-ID: <295acfbe39e7276664681fbf4566e528307a6bef.camel@debian.org> From: Luca Boccassi To: "Peng, Yuan" , "dev@dpdk.org" Date: Tue, 17 Sep 2019 10:53:04 +0100 In-Reply-To: <67D543A150B29E4CAAE53918F64EDAEA3775D71C@SHSMSX103.ccr.corp.intel.com> References: <20190815180508.14043-1-yskoh@mellanox.com> <4A3FBC67CD7DCF48BECD235A28B050330ADA8312@SHSMSX101.ccr.corp.intel.com> <67D543A150B29E4CAAE53918F64EDAEA3775D71C@SHSMSX103.ccr.corp.intel.com> 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" 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. >=20 > -----Original Message----- > From: Yu, PingX=20 > Sent: Thursday, September 12, 2019 2:37 PM > To: Yongseok Koh < > yskoh@mellanox.com > >; Ananyev, Konstantin < > konstantin.ananyev@intel.com > >;=20 > asafp@mellanox.com >=20 > 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 >=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 > > > 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 > > > 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 > > > 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 > >; Ananyev, Konstantin < > konstantin.ananyev@intel.com > >;=20 > asafp@mellanox.com >=20 > 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 >=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 > >; 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 >=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 > > > 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 > > > Signed-off-by: Joyce Kong < > joyce.kong@arm.com > > > Acked-by: Konstantin Ananyev < > konstantin.ananyev@intel.com > > >=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 > ] > Sent: Friday, August 16, 2019 2:05 AM > To: dpdk stable < > stable@dpdk.org > > > Cc:=20 > dev@dpdk.org > ;=20 > pezhang@redhat.com > ;=20 > siliu@redhat.com > ; Wang, FengqinX < > fengqinx.wang@intel.com > >; Chen, Zhaoyan < > zhaoyan.chen@intel.com > >; Xu, Qian Q < > qian.q.xu@intel.com > >;=20 > alialnu@mellanox.com > ;=20 > rasland@mellanox.com >=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 > These patches are located at branch 17.11 of dpdk-stable repo: > =20 > https://dpdk.org/browse/dpdk-stable/ >=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