Hi Christian/Ferruh, On Mon, 13 Dec 2021 at 7:03 PM, Christian Ehrhardt < christian.ehrhardt@canonical.com> wrote: > 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.11-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 > fixes > > >> the build for the v19.11 too? > > >> > > >> > http://inbox.dpdk.org/stable/BYAPR11MB27110B5509D1C5B999BB3AA1FE709@BYAPR11MB2711.namprd11.prod.outlook.com/T/#m3a451169c23b73d9f421fa0d7cf8da8c9f9e496f > > > > > > 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-default-linux-gcc/build/kernel/linux/kni/kni_net.c:791:20: > error: initialization from incompatible pointer type > [-Werror=incompatible-pointer-types] > [ 351s] .ndo_tx_timeout = kni_net_tx_timeout, > [ 351s] ^~~~~~~~~~~~~~~~~~ > > [1]: > https://build.opensuse.org/package/live_build_log/home:cpaelzer:branches:home:bluca:dpdk/dpdk-19.11/SLE_15/x86_64 [Kalesh] Yes, i am seeing the same error. I used make command to build dpdk, not meson. The back ported commit you mentioned takes care of meson build only I think. Regards, Kalesh > > > > > > [1]: > https://github.com/cpaelzer/dpdk-stable-queue/commit/6c8f6b72f28c713a7017d0db9ce95b2377ce4366 > > > > > >>> 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 > report > > >>> any issues/results with reply-all to this mail. For the final > release > > >>> 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=v19.11.11-rc1 < > https://dpdk.org/browse/dpdk-stable/tag/?id=v19.11.11-rc1> > > >>> > > >>> These patches are located at branch 19.11 of dpdk-stable repo: > > >>> https://dpdk.org/browse/dpdk-stable/ < > https://dpdk.org/browse/dpdk-stable/> > > >>> > > >>> Thanks. > > >>> > > >>> Christian Ehrhardt christian.ehrhardt@canonical.com>> > > >>> > > >>> --- > > >>> 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érez (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 > adapters > > >>> 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 > > >> > > > > > > > > > > > -- > Christian Ehrhardt > Staff Engineer, Ubuntu Server > Canonical Ltd > -- Regards, Kalesh A P