From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 921DFA00BE; Mon, 13 Dec 2021 14:32:59 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E342C410F7; Mon, 13 Dec 2021 14:32:58 +0100 (CET) Received: from smtp-relay-internal-0.canonical.com (smtp-relay-internal-0.canonical.com [185.125.188.122]) by mails.dpdk.org (Postfix) with ESMTP id 817A9406A2 for ; Mon, 13 Dec 2021 14:32:57 +0100 (CET) Received: from mail-qt1-f198.google.com (mail-qt1-f198.google.com [209.85.160.198]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-internal-0.canonical.com (Postfix) with ESMTPS id 3568B4070E for ; Mon, 13 Dec 2021 13:32:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1639402377; bh=XJchF2WoT3CawCocpl+nXkEv/LwCD/GHjJ//6xQpIkk=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=OeROaI2EDGuDqF/IKnNaEEyha19vLjo+VLRCNM2HXmqhBjrYn5D2b0gOtUdbJTcwb FlOKd6MyPPnR3oFEk3LuIT8pfWbFIfcPswshARhrSlAaiJ9dc5/aqG28J78XTJVyAb UO5qoogrVtSz07R360ovOrG/l3FSdI7NeNqmjVSt/da400pIkBaf35iUd84c3KzH2S +urRjiyQoTKxNHuYiVodeER+TLagbfHWyR67rlMbER+A2pQA7LygOO//K5cVGtZD4W TZzF/hZPlU/RsRI2d2A2uficRQtW0/eqGa3mMRRDtxS4FyonYvVPU2IqmS2wwFtn3s B9cIxfF9PsxTA== Received: by mail-qt1-f198.google.com with SMTP id f21-20020ac87f15000000b002be401bd1c6so3155579qtk.1 for ; Mon, 13 Dec 2021 05:32:57 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=XJchF2WoT3CawCocpl+nXkEv/LwCD/GHjJ//6xQpIkk=; b=6qiJC0zj++QY9Q6ZUVwKNpUqy8+DWbaSZXWwSeyL8UiNKwzuxbfbF38en+wbzzqpma 6e/8FFygOVDp2fqEyx8KyjKvu+Z7DFxKk/F9unt+EWHIpG0iGD0Hcwd3zk8kxEKsYqcE etSemSUsO0GpErVfDepzLUW5hNlJekZ0i/VAr4+wLi+a/gsPy2zNIHBFySOvvFdad2X0 nhLBUfrN3xS5oSMvoqd6qZi9FPopQsd0kUUK0VTOrrWa6K0ZzNURzG/YLjHK1Iuv/vzz ZvzMoOOxs9CMnhHb4lcGoQXVN7+xp4DdRT2VmVk9xejW723sbmnJgPrMC+YDgHGvdn1L kYwA== X-Gm-Message-State: AOAM530O2n4GWZfkl2LvuHIZUXy1u3w+UeBo3W7yYEjJWmTjgl9akYfD JSkbDHSOt3qugcJ/OtX06ndVu6iH9qnm1N6XwjJ+AbHChdYVmYokUy9oZj69Oyu7epacAqpGJH9 2bUT4GDZEcBQ+Sl2VtRYjc9Rbz0AMsCEdet8K X-Received: by 2002:a05:6214:21ed:: with SMTP id p13mr42968626qvj.111.1639402375526; Mon, 13 Dec 2021 05:32:55 -0800 (PST) X-Google-Smtp-Source: ABdhPJxt0SqZYg03nSrEI1nUBE3pWrmjjIWL4ItFFNM7xdVVJKnJhrALOY2/TbOOPPvJBOK3SRPGFubV8OyBEnREku4= X-Received: by 2002:a05:6214:21ed:: with SMTP id p13mr42968581qvj.111.1639402375145; Mon, 13 Dec 2021 05:32:55 -0800 (PST) MIME-Version: 1.0 References: <20211209144315.3424225-1-christian.ehrhardt@canonical.com> <7bf4583f-c2de-0aa7-fb00-bf3e9ff8d99b@intel.com> <34e0a9a8-327e-0150-b18a-dc3bfdca7d11@intel.com> In-Reply-To: <34e0a9a8-327e-0150-b18a-dc3bfdca7d11@intel.com> From: Christian Ehrhardt Date: Mon, 13 Dec 2021 14:32:29 +0100 Message-ID: Subject: Re: 19.11.11 patches review and test To: Ferruh Yigit Cc: Kalesh Anakkur Purayil , dpdk stable , dpdk-dev , Abhishek Marathe , Akhil Goyal , Ali Alnubani , benjamin.walker@intel.com, David Christensen , Hariprasad Govindharajan , Hemant Agrawal , Ian Stokes , Jerin Jacob , John McNamara , Ju-Hyoung Lee , Kevin Traynor , Luca Boccassi , Pei Zhang , pingx.yu@intel.com, qian.q.xu@intel.com, Raslan Darawsheh , Thomas Monjalon , yuan.peng@intel.com, zhaoyan.chen@intel.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Mon, Dec 13, 2021 at 1:59 PM Ferruh Yigit wrote= : > > On 12/13/2021 12:39 PM, Christian Ehrhardt wrote: > > On Mon, Dec 13, 2021 at 12:06 PM Ferruh Yigit = wrote: > >> > >> On 12/13/2021 4:14 AM, Kalesh Anakkur Purayil wrote: > >>> Hi Christian, > >>> > >>> We still see KNI build failure on SLES15 SP3 with dpdk-stable-19.11.1= 1-rc1. Is there a plan to address that on 19.11.11 LTS? > >>> > >> > >> Hi Kalesh, > >> > >> I have sent a backport patch for 20.11, can you please check if it fix= es > >> the build for the v19.11 too? > >> > >> http://inbox.dpdk.org/stable/BYAPR11MB27110B5509D1C5B999BB3AA1FE709@BY= APR11MB2711.namprd11.prod.outlook.com/T/#m3a451169c23b73d9f421fa0d7cf8da8c9= f9e496f > > > > This patch is already applied in 19.11.11 as [1], thereby at least > > without backport adaptations it surely does not fix the problem. > > > > It is indeed. > > @Kalesh, can you please share more details related to the build error? > > > > Comparing the upstream, the 20.11 backport and the 19.11 kernel_dir > > the difference is that 19.11 currently has it inside the check for > > kernel_dir. > > But both get initialized from ` get_option('kernel_dir')` so either > > both or none would trigger and therefore it should be ok as-is. > > > > I'm happy to consider a fix if someone looks deeper into it and let me > > know what adaptations to try. > > I can also test build SLES15SP3 (before this discussion I knew it was > > "still broken"). > > > > The above fix should fix the build for SLES15SP3, are you aware of any > build error after that patch? As I said my SLES15SP3 test build kept failing [1] and it looked like the very same issue as before. [ 351s] CC /home/abuild/rpmbuild/BUILD/dpdk-1638972763.ee4ce5a57/x86_64-default-linux-= gcc/build/kernel/linux/igb_uio/igb_uio.mod.o [ 351s] /home/abuild/rpmbuild/BUILD/dpdk-1638972763.ee4ce5a57/x86_64-defau= lt-linux-gcc/build/kernel/linux/kni/kni_net.c:791:20: error: initialization from incompatible pointer type [-Werror=3Dincompatible-pointer-types] [ 351s] .ndo_tx_timeout =3D kni_net_tx_timeout, [ 351s] ^~~~~~~~~~~~~~~~~~ [1]: https://build.opensuse.org/package/live_build_log/home:cpaelzer:branch= es:home:bluca:dpdk/dpdk-19.11/SLE_15/x86_64 > > [1]: https://github.com/cpaelzer/dpdk-stable-queue/commit/6c8f6b72f28c7= 13a7017d0db9ce95b2377ce4366 > > > >>> Regards, > >>> Kalesh > >>> > >>> On Thu, Dec 9, 2021 at 8:13 PM > wrote: > >>> > >>> Hi all, > >>> > >>> Here is a list of patches targeted for stable release 19.11.11. > >>> > >>> The planned date for the final release is 7th January 2021. > >>> > >>> Please help with testing and validation of your use cases and re= port > >>> any issues/results with reply-all to this mail. For the final re= lease > >>> the fixes and reported validations will be added to the release = notes. > >>> > >>> A release candidate tarball can be found at: > >>> > >>> https://dpdk.org/browse/dpdk-stable/tag/?id=3Dv19.11.11-rc1 > >>> > >>> These patches are located at branch 19.11 of dpdk-stable repo: > >>> https://dpdk.org/browse/dpdk-stable/ > >>> > >>> Thanks. > >>> > >>> Christian Ehrhardt > > >>> > >>> --- > >>> Ajit Khaparde (3): > >>> net/bnxt: fix Tx queue startup state > >>> net/bnxt: fix memzone free for Tx and Rx rings > >>> net/bnxt: fix tunnel port accounting > >>> > >>> Alexander Bechikov (1): > >>> mbuf: fix dump of dynamic fields and flags > >>> > >>> Alexander Kozyrev (2): > >>> net/mlx5: fix GENEVE and VXLAN-GPE flow item matching > >>> net/mlx5: fix GRE flow item matching > >>> > >>> Alvin Zhang (1): > >>> net/i40e: fix Rx packet statistics > >>> > >>> Aman Singh (1): > >>> kni: fix build for SLES15-SP3 > >>> > >>> Anatoly Burakov (2): > >>> vfio: fix FreeBSD clear group stub > >>> vfio: fix FreeBSD documentation > >>> > >>> Anoob Joseph (2): > >>> test/crypto: skip plain text compare for null cipher > >>> common/cpt: fix KASUMI input length > >>> > >>> Arek Kusztal (2): > >>> crypto/qat: fix status in RSA decryption > >>> crypto/qat: fix uncleared cookies after operation > >>> > >>> Ben Pfaff (1): > >>> doc: fix numbers power of 2 in LPM6 guide > >>> > >>> Bing Zhao (2): > >>> net/mlx5: fix flow tables double release > >>> net/mlx5: fix RETA update without stopping device > >>> > >>> Bruce Richardson (3): > >>> eal/freebsd: lock memory device to prevent conflicts > >>> test/mem: fix memory autotests on FreeBSD > >>> eal/freebsd: ignore in-memory option > >>> > >>> Chengchang Tang (2): > >>> net/bonding: fix dedicated queue mode in vector burst > >>> net/bonding: fix RSS key length > >>> > >>> Chengfeng Ye (1): > >>> net/axgbe: fix unreleased lock in I2C transfer > >>> > >>> Chengwen Feng (1): > >>> kni: check error code of allmulticast mode switch > >>> > >>> Cian Ferriter (1): > >>> ring: fix Doxygen comment of internal function > >>> > >>> Ciara Loftus (1): > >>> net/af_xdp: disable secondary process support > >>> > >>> Ciara Power (3): > >>> crypto/openssl: fix CCM processing 0 length source > >>> examples/fips_validation: remove unused allocation > >>> test/crypto: fix unnecessary stats retrieval > >>> > >>> Conor Walsh (8): > >>> net/hinic/base: remove some unused variables > >>> bus/fslmc: remove unused device count > >>> event/sw: remove unused inflight events count > >>> net/liquidio: remove unused counter > >>> net/qede/base: remove unused message size > >>> net/vmxnet3: fix build with clang 13 > >>> test/distributor: remove unused counter > >>> examples/performance-thread: remove unused hits count > >>> > >>> Dapeng Yu (1): > >>> net/ice: save rule on switch filter creation > >>> > >>> Dariusz Sosnowski (2): > >>> net/mlx5: fix MPLS tunnel outer layer overwrite > >>> doc: fix typo in coding style > >>> > >>> David Christensen (3): > >>> eal/ppc: ignore GCC 10 stringop-overflow warnings > >>> config/ppc: ignore GCC 11 psabi warnings > >>> test/atomic: fix 128-bit atomic test with many cores > >>> > >>> David Hunt (1): > >>> test/power: fix CPU frequency when turbo enabled > >>> > >>> David Marchand (10): > >>> net/virtio: avoid unneeded link interrupt configuration > >>> net/virtio-user: fix Rx interrupts with multi-queue > >>> bus/vmbus: fix leak on device scan > >>> test/latency: fix loop boundary > >>> eal/x86: fix some CPU extended features definitions > >>> mbuf: enforce no option for dynamic fields and flags > >>> doc: fix default mempool option in guides > >>> eal/linux: fix uevent message parsing > >>> test/red: fix typo in test description > >>> examples/ntb: fix build dependency > >>> > >>> Eli Britstein (4): > >>> net: avoid cast-align warning in VLAN insert function > >>> mbuf: avoid cast-align warning in data offset macro > >>> eal/x86: avoid cast-align warning in memcpy functions > >>> app/testpmd: fix packet burst spreading stats > >>> > >>> Eugenio P=C3=A9rez (1): > >>> vhost: clean IOTLB cache on vring stop > >>> > >>> Fan Zhang (1): > >>> examples/fips_validation: fix device start > >>> > >>> Feifei Wang (2): > >>> drivers/net: fix typo in vector Rx comment > >>> drivers/net: fix vector Rx comments > >>> > >>> Ferruh Yigit (3): > >>> net/softnic: fix useless address check > >>> net/i40e: fix 32-bit build > >>> ethdev: fix crash on owner delete > >>> > >>> Gagandeep Singh (3): > >>> common/dpaax/caamflib: fix IV for short MAC-I in SNOW3G > >>> common/dpaax: fix physical address conversion > >>> drivers/crypto: fix IPsec TTL decrement option > >>> > >>> Gaoxiang Liu (1): > >>> vhost: log socket path on adding connection > >>> > >>> Georg Sauthoff (1): > >>> net: fix aliasing in checksum computation > >>> > >>> Gregory Etelson (6): > >>> examples/multi_process: fix Rx packets distribution > >>> net/mlx5: fix RSS expansion scheme for GRE header > >>> app/testpmd: fix hexadecimal parser with odd length > >>> net/mlx5: fix VXLAN-GPE next protocol translation > >>> net/mlx5: fix GENEVE protocol type translation > >>> net/mlx5: fix GRE protocol type translation > >>> > >>> Haiyue Wang (1): > >>> net/ice: fix memzone leak after device init failure > >>> > >>> Hanumanth Reddy Pothula (1): > >>> net/octeontx2: fix MTU when PTP is enabled > >>> > >>> Harneet Singh (1): > >>> examples/l3fwd-power: fix early shutdown > >>> > >>> Harry van Haaren (1): > >>> app/eventdev: fix terminal colour after control-c exit > >>> > >>> Heinrich Kuhn (1): > >>> net/nfp: fix minimum descriptor sizes > >>> > >>> Honnappa Nagarahalli (1): > >>> eal: reset lcore task callback and argument > >>> > >>> Huisong Li (5): > >>> net/hns3: fix input parameters of MAC functions > >>> app/testpmd: retain all original dev conf when config DCB > >>> ethdev: fix PCI device release in secondary process > >>> net/hns3: simplify queue DMA address arithmetic > >>> app/testpmd: remove unused header file > >>> > >>> Hyong Youb Kim (2): > >>> net/enic: fix filter mode detection > >>> net/enic: avoid error message when no advanced filtering > >>> > >>> Ivan Ilchenko (2): > >>> net/virtio: fix mbuf count on Rx queue setup > >>> ethdev: fix xstats by ID API documentation > >>> > >>> Jerin Jacob (1): > >>> examples/performance-thread: fix build with clang 12.0.1 > >>> > >>> Jiawei Wang (1): > >>> net/mlx5: fix metadata and meter split shared tag > >>> > >>> Jie Wang (2): > >>> net/i40e: fix forward outer IPv6 VXLAN > >>> app/testpmd: fix DCB in VT configuration > >>> > >>> Jim Harris (2): > >>> eal/linux: remove unused variable for socket memory > >>> power: fix build with clang 13 > >>> > >>> Joyce Kong (2): > >>> test/service: fix some comment > >>> test/func_reentrancy: free memzones after test > >>> > >>> Julien Meunier (2): > >>> stack: fix reload head when pop fails > >>> net/ixgbe: fix queue release > >>> > >>> Junxiao Shi (1): > >>> net/memif: fix chained mbuf determination > >>> > >>> Kai Ji (1): > >>> test/crypto: fix data lengths > >>> > >>> Kalesh AP (5): > >>> net/bnxt: fix ring group free > >>> net/bnxt: fix double allocation of ring groups > >>> net/bnxt: fix function driver register/unregister > >>> net/bnxt: check FW capability for VLAN offloads > >>> net/bnxt: fix VLAN indication in Rx mbuf > >>> > >>> Kevin Traynor (2): > >>> bitrate: fix registration to match API description > >>> bitrate: fix calculation to match API description > >>> > >>> Konstantin Ananyev (2): > >>> test/bpf: fix undefined behavior with clang > >>> test: fix ring PMD initialisation > >>> > >>> Kumara Parameshwaran (1): > >>> net/failsafe: fix secondary process probe > >>> > >>> Lance Richardson (1): > >>> net: fix checksum API documentation > >>> > >>> Li Feng (1): > >>> vhost: add sanity check on inflight last index > >>> > >>> Long Li (1): > >>> bus/vmbus: fix ring buffer mapping in secondary process > >>> > >>> Maciej Szwed (1): > >>> interrupt: fix request notifier interrupt processing > >>> > >>> Maxime Coquelin (3): > >>> app/testpmd: fix RSS key length > >>> app/testpmd: fix RSS type display > >>> net/mlx5: fix RSS RETA update > >>> > >>> Michal Krawczyk (4): > >>> net/ena: fix offload capabilities verification > >>> net/ena: fix per-queue offload capabilities > >>> net/ena: advertise scattered Rx capability > >>> Revert "net/ena: trigger reset on Tx prepare failure" > >>> > >>> Naga Harish K S V (2): > >>> eventdev/eth_rx: fix WRR buffer overrun > >>> eventdev/eth_tx: fix queue delete logic > >>> > >>> Nithin Dabilpuram (2): > >>> app/testpmd: fix check without outer checksum > >>> app/testpmd: fix dump of Tx offload flags > >>> > >>> Olivier Matz (1): > >>> mem: fix dynamic hugepage mapping in container > >>> > >>> Pablo de Lara (1): > >>> efd: allow more CPU sockets in table creation > >>> > >>> Przemyslaw Zegan (2): > >>> app/crypto-perf: fix AAD template copy overrun > >>> common/qat: fix queue pairs number > >>> > >>> Qi Zhang (1): > >>> net/ice/base: fix typo in comment > >>> > >>> Qiming Chen (12): > >>> net/ixgbe: fix Rx multicast statistics after reset > >>> net/pcap: fix resource leakage on port probe > >>> net/ixgbe: fix hash handle leak > >>> net/ixgbe: fix queue resource leak > >>> net/ixgbe: fix MAC resource leak > >>> net/ixgbe: fix mbuf leak > >>> net/i40e: fix mbuf leak > >>> net/i40e: fix device startup resource release > >>> net/iavf: fix mbuf leak > >>> net/i40e/base: fix resource leakage > >>> net/iavf: fix Rx queue buffer size alignment > >>> common/iavf: fix ARQ resource leak > >>> > >>> Rebecca Troy (1): > >>> test/compress: fix buffer overflow > >>> > >>> Robin Zhang (7): > >>> net/i40e/base: fix PHY identifiers for 2.5G and 5G adapte= rs > >>> net/i40e/base: fix PF reset > >>> net/i40e/base: fix update link data for X722 > >>> net/i40e/base: fix AOC media type > >>> net/i40e/base: fix function name in comments > >>> net/i40e/base: fix potentially uninitialized variables > >>> net/i40e/base: fix using checksum before check > >>> > >>> Ruifeng Wang (3): > >>> examples/service_cores: fix lcore count check > >>> net/i40e: fix risk in descriptor read in NEON Rx > >>> net/i40e: fix risk in descriptor read in scalar Rx > >>> > >>> Sean Morrissey (3): > >>> remove repeated 'the' in the code > >>> fix PMD wording > >>> doc: capitalise PMD > >>> > >>> Shijith Thotton (2): > >>> test/event_crypto: fix event crypto metadata write > >>> test/event: fix timer adapter creation test > >>> > >>> Somnath Kotur (1): > >>> net/bnxt: fix Rx next consumer index in mbuf alloc fail > >>> > >>> Stephen Hemminger (4): > >>> mbuf: fix typo in comment > >>> bpf: allow self-xor operation > >>> net/bnxt: fix firmware version query > >>> fix spelling in comments and doxygen > >>> > >>> Sunil Kumar Kori (1): > >>> app/testpmd: fix access to DSCP table entries > >>> > >>> Thomas Monjalon (2): > >>> net/qede: fix minsize build > >>> doc: remove repeated repeated words > >>> > >>> Timothy Redaelli (1): > >>> doc: strip build artefacts for examples file list > >>> > >>> Tudor Cornea (3): > >>> net/iavf: fix overflow in maximum packet length config > >>> net/af_packet: fix ignoring full ring on Tx > >>> net/ixgbe: fix port initialization if MTU config fails > >>> > >>> Vanshika Shukla (1): > >>> examples/ptpclient: fix delay request message > >>> > >>> Viacheslav Ovsiienko (2): > >>> app/testpmd: fix hex string parser in flow commands > >>> doc: describe timestamp limitations for mlx5 > >>> > >>> Vladimir Medvedkin (3): > >>> hash: fix Doxygen comment of Toeplitz file > >>> lpm6: fix buffer overflow > >>> rib: fix IPv6 depth mask > >>> > >>> William Tu (1): > >>> doc: fix emulated device names in e1000 guide > >>> > >>> Xiaoyu Min (1): > >>> net/mlx4: fix empty Ethernet spec with VLAN > >>> > >>> Xuan Ding (1): > >>> net/virtio: fix indirect descriptor reconnection > >>> > >>> Xueming Li (2): > >>> eal: fix device iterator when no bus is selected > >>> net/mlx5: fix Altivec Rx > >>> > >>> Yunjian Wang (1): > >>> net/ice: fix memzone leak on queue re-configure > >>> > >>> Yury Kylulin (1): > >>> net/i40e: support 25G AOC/ACC cables > >>> > >>> Zhihong Peng (1): > >>> examples/performance-thread: fix build with ASan > >>> > >>> Zhihong Wang (1): > >>> app/testpmd: fix Tx retry in flowgen engine > >>> > >>> > >>> > >>> -- > >>> Regards, > >>> Kalesh A P > >> > > > > > --=20 Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd