Re: [dpdk-dev] [dpdk-stable] 18.11.2 (LTS) patches review and test

[dwilder <dwilder@us.ibm.com>]

Hi Kevin-

I tested 8.11.2 RC1 on IBM Power9 hardware.  No issues or performance 
degradation were observed.

Tests run:
  * Single port stability test using l3fwd (16 cpus) and TRex, tested 64 
and 1500 byte packets at a 0.0% drop rate for 4 hours each.

System tested:
  - IBM Power9 Model 8335-101 CPU: 2.3 (pvr 004e 1203)

Tested NICs:
  - ConnectX-5 (fw 16.23.1020).

OS Tested
  - Ubuntu 18.04.2 LTS (kernel 4.15.0-47-generic)

Regards,
Dave


On 2019-05-21 07:01, Kevin Traynor wrote:
> Hi all,
> 
> Here is a list of patches targeted for LTS release 18.11.2.
> 
> The planned date for the final release is 11th June.
> 
> Please help with testing and validation of your use cases and report
> any issues/results. For the final release I will update the release
> notes with fixes and reported validations.
> 
> A release candidate tarball can be found at:
> 
> 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__dpdk.org_browse_dpdk-2Dstable_tag_-3Fid-3Dv18.11.2-2Drc1&d=DwIDaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=7ndxyKjH9UrBD68Us2WP1wI4BwEBQbeAyz8i_vwCCaI&m=_uELDTGd8N4r6nMedCtB7BZ7smk16u_gOekVjKO__nM&s=A2p-8kC3tA0EjBPnJwZ03RdD9-qTfAkg2Z135xdJ78k&e=
> 
> These patches are located at branch 18.11 of dpdk-stable repo:
> 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__dpdk.org_browse_dpdk-2Dstable_&d=DwIDaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=7ndxyKjH9UrBD68Us2WP1wI4BwEBQbeAyz8i_vwCCaI&m=_uELDTGd8N4r6nMedCtB7BZ7smk16u_gOekVjKO__nM&s=wxF_Eze1ZxjaJ6I_lQ9_wT9IafwwECHlKcDIEIgkGP8&e=
> 
> Thanks.
> 
> Kevin Traynor
> 
> ---
> Aaron Conole (4):
>       net/virtio: fix dangling pointer on failure
>       ci: enable ccache in Travis
>       ci: add a distinguisher to the extra Travis builds
>       ipc: unlock on failure
> 
> Abhinandan Gujjar (1):
>       eventdev: fix crypto adapter
> 
> Agalya Babu RadhaKrishnan (2):
>       doc: fix interactive commands in testpmd guide
>       app/testpmd: fix help info for interactive commands
> 
> Akhil Goyal (3):
>       crypto/dpaa2_sec: fix session clearing
>       crypto/dpaa2_sec: fix offset calculation for GCM
>       crypto/dpaa: fix session destroy
> 
> Alejandro Lucero (5):
>       net/nfp: fix RSS query
>       net/nfp: fix file descriptor check
>       net/nfp: fix potential integer overflow
>       net/nfp: check return value
>       net/nfp: fix memory leak
> 
> Anatoly Burakov (11):
>       malloc: fix documentation of realloc function
>       vfio: document multiprocess limitation for container API
>       malloc: fix IPC message initialization
>       mem: warn user when running without NUMA support
>       eal: fix cleanup in no-shconf mode
>       ipc: fix send error handling
>       ipc: harden message receive
>       ipc: handle more invalid parameter cases
>       doc: fix typo in IPC guide
>       ipc: add warnings about not using IPC with memory API
>       ipc: add warnings about correct API usage
> 
> Andrew Rybchenko (3):
>       net/sfc: remove control path logging from Rx queue count
>       net/sfc: fix logging from secondary process
>       net/sfc: fix speed capabilities reported in device info
> 
> Andrius Sirvys (4):
>       drivers/net: fix shifting 32-bit signed variable 31 times
>       acl: fix compiler flags with meson and AVX2 runtime
>       net/virtio: fix duplicate naming of include guard
>       bitrate: fix unchecked return value
> 
> Andy Pei (1):
>       raw/ifpga: modify log output
> 
> Anoob Joseph (2):
>       common/cpt: fix null auth only
>       cryptodev: fix driver name comparison
> 
> Arek Kusztal (2):
>       crypto/openssl: fix big numbers after computations
>       crypto/openssl: fix modexp
> 
> Beilei Xing (2):
>       net/i40e: fix logging on VF close
>       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 (24):
>       test/compress: fix missing header include
>       examples/ip_pipeline: disable build when no epoll
>       examples/vhost_crypto: fix dependency on vhost library
>       devtools: fix build test on FreeBSD
>       eal: support strlcat function
>       telemetry: fix mapping of statistics
>       eal: tighten permissions on shared memory files
>       net/bonding: fix buffer length when printing strings
>       devtools: skip meson build for missing compilers
>       devtools: support older compilers with meson test
>       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/i40e: fix dereference before check when getting EEPROM
>       app/testpmd: fix variable use before null check
>       crypto/snow3g: add to meson build
>       crypto/kasumi: fix dependency check
>       crypto/zuc: fix dependency check
>       net/ixgbe: fix warning with GCC 9
>       bus/fslmc: fix warning with GCC 9
>       raw/skeleton: fix warnings with GCC 9
>       raw/dpaa2_cmdif: fix warnings with GCC 9
>       build: fix ninja install on FreeBSD
> 
> Chaitanya Babu Talluri (3):
>       drivers/net: fix possible overflow using strlcat
>       test/crypto: fix possible overflow using strlcat
>       cfgfile: replace strcat with strlcat
> 
> Chas Williams (3):
>       net/bonding: fix Tx in 802.3ad mode
>       net/bonding: fix slave Tx burst for mode 4
>       net/bonding: fix link status
> 
> Chenbo Xia (2):
>       app/testpmd: fix return value check
>       net/virtio-user: fix return value check
> 
> Darek Stojaczyk (1):
>       eal: initialize alarms early
> 
> David Christensen (5):
>       maintainers: update for IBM POWER
>       examples/power: fix build with some disabled PMDs
>       test/barrier: fix typo in log
>       test/barrier: fix allocation check
>       test/barrier: fix for Power CPUs
> 
> David Harton (1):
>       net/ixgbe: restore VLAN filter for VF
> 
> David Hunt (5):
>       power: fix governor storage to trim newlines
>       examples/power: fix resource leak
>       examples/power: fix unreachable VF MAC init
>       examples/power: fix buffer overrun
>       examples/power: fix json null termination
> 
> David Marchand (19):
>       eal: fix check when retrieving current CPU affinity
>       eal: fix control threads pinnning
>       eal: restrict control threads to startup CPU affinity
>       eal: remove dead code in core list parsing
>       eal: fix core list validation with disabled cores
>       app/testpmd: remove unused field from port struct
>       doc: fix examples in bonding guide
>       net/bonding: fix port id types
>       net/bonding: fix slave id types
>       net/bonding: fix packet count type for LACP
>       net/bonding: fix queue index types
>       app/testpmd: add missing newline when showing statistics
>       app/testpmd: extend forwarding statistics to 64 bits
>       app/testpmd: remove useless casts on statistics
>       doc: fix ABI check script examples
>       test: clean remaining trace of devargs autotest
>       devtools: fix symbol name in check log
>       devtools: accept experimental symbol promotion
>       devtools: fix check of symbol added as stable API
> 
> Davide Caratti (1):
>       net/bnxt: support IOVA VA mode
> 
> Dekel Peled (13):
>       net/mlx5: fix Tx metadata for multi-segment packet
>       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
>       net/mlx5: fix errno typos in comments
>       doc: fix typos in mlx5 guide
>       doc: fix typos in testpmd user guide
>       app/testpmd: fix typo in comment
>       ethdev: fix typo in error messages
>       net/mlx5: fix comments mixing Rx and Tx
>       net/mlx5: fix release of Rx queue object
> 
> Dharmik Thakkar (2):
>       hash: fix position returned in free slots
>       hash: fix total entries count
> 
> Erik Gabriel Carrillo (1):
>       eventdev: update references to removed function
> 
> Fan Zhang (3):
>       examples/ipsec-secgw: fix AES-CTR block size
>       vhost/crypto: fix parens
>       crypto/virtio: fix IV offset
> 
> Ferruh Yigit (6):
>       kni: fix build with Linux 5.1
>       mbuf: update Tx VLAN and QinQ flags documentation
>       build: fix meson binutils workaround
>       build: fix crash by disabling AVX512 with binutils 2.31
>       net/ring: fix return value check
>       net/kni: fix return value check
> 
> Fiona Trahe (2):
>       doc: fix missing asymmetric crypto table
>       crypto/qat: fix null cipher algo for non 8-byte multiple
> 
> Gagandeep Singh (6):
>       crypto/caam_jr: fix shared descriptor endianness
>       crypto/caam_jr: fix total length in auth only s/g
>       crypto/caam_jr: fix memory leak and illegal access
>       net/enetc: fix SMMU unhandled context fault
>       net/enetc: fix big endian build and buffer allocation
>       net/enetc: fix crash at high speed traffic
> 
> Gage Eads (2):
>       app/pdump: remove only created vdevs
>       ring: fix namesize macro documentation block
> 
> Gavin Hu (4):
>       ring: enforce reading tail before slots
>       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 values of descriptor limits
> 
> Harry van Haaren (1):
>       event/sw: fix enqueue checks in self-test
> 
> Hemant Agrawal (3):
>       mempool/dpaa2: fix continuous print on empty pool
>       test/crypto: fix duplicate id used by CCP device
>       raw/dpaa2_qdma: fix spin lock release
> 
> Herakliusz Lipiec (5):
>       net/tap: fix multi process reply buffer
>       ipc: fix memory leak on request failure
>       net/tap: fix memory leak on IPC request
>       net/tap: fix potential IPC buffer overrun
>       doc: fix broken link in LPM guide
> 
> Hyong Youb Kim (12):
>       net/bonding: avoid warning for invalid port
>       net/enic: fix flow director SCTP matching
>       net/enic: fix SCTP match for flow API
>       net/enic: allow flow mark ID 0
>       net/enic: check for unsupported flow item types
>       net/enic: reset VXLAN port regardless of overlay offload
>       net/enic: fix VXLAN match
>       net/enic: fix endianness in VLAN match
>       net/enic: fix max MTU calculation
>       net/enic: move arguments into struct
>       net/enic: fix inner packet matching
>       net/enic: fix VLAN inner type matching for old hardware
> 
> Igor Romanov (4):
>       app/testpmd: fix stdout flush after printing stats
>       net/sfc: improve TSO header length check in EFX datapath
>       net/sfc: improve TSO header length check in EF10 datapath
>       net/sfc: fix MTU change to check Rx scatter consistency
> 
> Igor Russkikh (11):
>       net/atlantic: fix negative error codes
>       net/atlantic: remove unused variable
>       net/atlantic: remove extra checks for error codes
>       net/atlantic: fix link configuration
>       net/atlantic: fix missing VLAN filter offload
>       net/atlantic: fix xstats return
>       net/atlantic: enable broadcast traffic
>       net/atlantic: extra line at eof
>       net/atlantic: fix max eeprom size
>       net/atlantic: validity check for eeprom dev address
>       net/atlantic: eliminate excessive log levels on Rx/Tx
> 
> Ilya Maximets (4):
>       net/virtio: add barriers for extra descriptors on Rx split
>       vhost: fix device leak on connection add failure
>       vhost: fix passing destroyed device to destroy callback
>       vhost: fix silent queue enabling with legacy guests
> 
> Ivan Malov (1):
>       net/sfc: log port ID as 16-bit unsigned integer on panic
> 
> Jens Freimann (1):
>       net/virtio: set offload flag for jumbo frames
> 
> Jerin Jacob (3):
>       mk: fix AVX512 disabled warning on non x86
>       build: remove meson warning for Arm
>       table: fix arm64 hash function selection
> 
> Jiayu Hu (1):
>       vhost: fix interrupt suppression for the split ring
> 
> Jie Pan (1):
>       kni: fix type for MAC address
> 
> John McNamara (2):
>       doc: fix spelling reported by aspell in guides
>       doc: fix spelling reported by aspell in comments
> 
> Joyce Kong (2):
>       doc: update cross Arm toolchain in Linux guide
>       rwlock: reimplement with atomic builtins
> 
> Julien Meunier (1):
>       test/pmd_perf: fix the way to drain the port
> 
> Kevin Traynor (5):
>       net/qede: support IOVA VA mode
>       eal/linux: fix log levels for pagemap reading failure
>       doc: fix formatting in testpmd guide
>       bus/fslmc: fix warning with GCC 9
>       mk: disable packed member pointer warning for telemetry
> 
> Konstantin Ananyev (1):
>       examples/ipsec-secgw: fix SPD no-match case
> 
> Krzysztof Kanas (1):
>       test/bonding: fix MAC assignment for re-run
> 
> Li Qiang (1):
>       raw/ifpga: fix file descriptor leak in error path
> 
> Liang Zhang (1):
>       net/bonding: fix LACP negotiation
> 
> Luca Boccassi (1):
>       build: use default flags for default Arm machine
> 
> Lukasz Krakowiak (1):
>       doc: fix JSON interface for power sample
> 
> Marcin Hajkowski (1):
>       power: fix thread-safety environment modification
> 
> Marcin Smoczynski (1):
>       examples/ipsec-secgw: fix build error log
> 
> Mark Gillott (1):
>       net/vmxnet3: add VLAN filter capability
> 
> Marko Kovacevic (2):
>       compress/qat: fix setup inter buffers
>       examples/fips_validation: fix CMAC test
> 
> Mattias Rönnblom (3):
>       event/dsw: fix capability flags
>       power: fix cache line alignment
>       event/dsw: ignore scheduling type for single-link queues
> 
> Maxime Coquelin (1):
>       vhost: prevent disabled rings to be processed with zero-copy
> 
> Michael Santana (2):
>       devtools: fix result of svg include check
>       ci: introduce Travis builds for GitHub repositories
> 
> Mohammad Abdul Awal (1):
>       vhost: fix null pointer checking
> 
> Natanael Copa (7):
>       eal/linux: remove thread ID from debug message
>       bus/fslmc: remove unused include of error.h
>       bus/fslmc: fix build with musl libc
>       net/nfp: fix build with musl libc
>       net/netvsc: fix include of fcntl.h
>       app/test: fix flags with meson
>       app/test: fix build with musl libc
> 
> Nikhil Rao (1):
>       eventdev: fix Rx adapter event flush
> 
> Nithin Dabilpuram (2):
>       app/testpmd: fix Tx VLAN and QinQ dependency
>       app/testpmd: fix Tx QinQ set
> 
> Oleg Polyakov (1):
>       net/tap: fix getting max iovec
> 
> Ori Kam (1):
>       net/mlx5: fix RSS validation function
> 
> Pablo Cascón (1):
>       net/nfp: fix setting MAC address
> 
> Pablo de Lara (1):
>       doc: add missing algorithms for AESNI-MB PMD
> 
> Pallantla Poornima (10):
>       vhost: fix sprintf with snprintf
>       net/softnic: fix possible buffer overflow
>       event/opdl: replace sprintf with snprintf
>       net/nfp: fix possible buffer overflow
>       app/test: fix sprintf with strlcat
>       test/event: replace sprintf with snprintf
>       power: remove unused variable
>       test/distributor: replace sprintf with strlcpy
>       test/hash: replace sprintf with snprintf
>       app/bbdev: replace sprintf with snprintf or strlcpy
> 
> Pavel Belous (7):
>       net/atlantic: fix buffer overflow
>       net/atlantic: fix EEPROM get for small and uneven lengths
>       net/atlantic: error handling for mailbox access
>       net/atlantic: eeprom get/set should consider offset
>       net/atlantic: bad logic with offsets talking with firmware
>       net/atlantic: flow control settings synchronization on rx
>       net/atlantic: use capability bits to detect eeprom access
> 
> Qi Zhang (3):
>       net/pcap: fix memory leak
>       net/i40e: fix time sync for 25G
>       net/i40e: fix scattered Rx enabling
> 
> Qiming Yang (1):
>       net/iavf: fix info get
> 
> Rahul Lakkireddy (1):
>       net/cxgbe: update Chelsio T5/T6 NIC device ids
> 
> Rami Rosen (11):
>       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
>       doc: fix tag for inner RSS feature
>       app/testpmd: fix a typo in log message
>       ethdev: fix method name in doxygen comment
>       ethdev: fix a typo
>       doc: fix two typos in contributing guide
> 
> Rasesh Mody (3):
>       net/bnx2x: fix MTU for jumbo frame
>       net/bnx2x: fix race for periodic flags
>       net/bnx2x: fix optic module verification
> 
> Raslan Darawsheh (2):
>       net/tap: fix multi-process request
>       bus/vdev: fix hotplug twice
> 
> Reshma Pattan (4):
>       examples/power: fix string null termination
>       examples/power: fix string overflow
>       examples/power: fix overflowed value
>       mk: disable warning for packed member pointer
> 
> Shahaf Shuler (3):
>       net/mlx5: fix packet inline on Tx queue wraparound
>       mem: limit use of address hint
>       app/testpmd: fix mempool free on exit
> 
> Shahed Shaikh (6):
>       net/qede: fix Tx packet prepare for tunnel packets
>       net/qede: fix Rx packet drop
>       doc: add flow API to qede NIC features
>       net/bnx2x: fix memory leak
>       net/bnx2x: fix ramrod timeout
>       net/bnx2x: fix DMAE timeout
> 
> Shreyansh Jain (2):
>       bus/dpaa: fix Rx discard register mask
>       raw/dpaa2_qdma: fix to support multiprocess execution
> 
> Somnath Kotur (1):
>       net/bnxt: fix big endian build
> 
> Stephen Hemminger (26):
>       crypto/virtio: use local log type
>       net/bnxt: silence IOVA warnings
>       net/bnxt: suppress spurious error log
>       net/octeontx: fix vdev name
>       drivers: fix SPDX license id consistency
>       bus/vmbus: fix secondary process setup
>       net/netvsc: fix VF support with secondary process
>       bus/vmbus: fix check for mmap failure
>       bus/vmbus: stop mapping if empty resource found
>       bus/vmbus: map ring in secondary process
>       bus/fslmc: decrease log level for unsupported devices
>       net/vdev_netvsc: fix device cast
>       net/bnxt: fix Rx VLAN offload flags
>       net/netvsc: remove useless condition
>       net/virtio: remove useless condition
>       ring: fix an error message
>       doc: fix spelling in testpmd guide
>       net/virtio: fix buffer leak on VLAN insert
>       net/cxgbe: fix colliding function names
>       bus/vmbus: fix resource leak on error
>       eal: fix formatting of hotplug error message
>       net/ring: fix coding style
>       net/ring: avoid hard-coded length
>       net/ring: use calloc style where appropriate
>       net/ring: check length of ring name
>       net/netvsc: reset mbuf port on VF Rx
> 
> Thomas Monjalon (9):
>       ethdev: remove unused variable
>       mk: fix build of shared library with libbsd
>       devtools: add libelf dependency to build test
>       devtools: test build of zlib PMD
>       devtools: fix test of some build options
>       net/mlx: prefix private structure
>       doc: fix PCI whitelist typo in prog guide
>       bus/vdev: fix debug message on probing
>       doc: fix heading levels in bbdev test guide
> 
> Tiwei Bie (8):
>       net/virtio: fix in-order Tx path for split ring
>       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
>       net/virtio-user: fix multiqueue with vhost kernel
>       net/virtio: add barrier in interrupt enable
>       net/virtio-user: fix multi-process support
>       app/testpmd: fix unintentional integer overflow
> 
> Tomasz Cel (2):
>       compress/isal: fix compression stream initialization
>       compress/isal: fix getting information about CPU
> 
> Tomasz Jozwiak (1):
>       drivers/qat: fix queue pair NUMA node
> 
> Viacheslav Ovsiienko (3):
>       net/mlx5: fix flow priorities probing error path
>       net/mlx5: fix memory region cleanup
>       net/mlx4: fix memory region cleanup
> 
> Vishal Kulkarni (1):
>       net/cxgbe: fix missing checksum flags and packet type
> 
> Wei Zhao (5):
>       net/avf: fix admin queue interrupt for ICE
>       net/i40e: log when provided RSS key is not valid
>       app/testpmd: fix hex string parser support for flow API
>       net/i40e: fix queue number check
>       app/testpmd: fix offload flags after port config
> 
> Xiao Wang (1):
>       net/fm10k: fix VLAN strip offload flag
> 
> Yipeng Wang (1):
>       hash: fix doc about thread/process safety
> 
> Yongseok Koh (11):
>       net/mlx5: fix memory event on secondary process
>       net/mlx4: change device reference for secondary process
>       net/mlx: remove debug messages on datapath
>       net/mlx5: fix external memory registration
>       drivers/event: disable OcteonTx for buggy Arm compilers
>       net/mlx5: fix instruction hotspot on replenishing Rx buffer
>       examples/multi_process: fix buffer underrun
>       net/mlx5: check Tx queue size overflow
>       net/mlx5: fix max number of queues for NEON Tx
>       net/mlx5: fix Multi-Packet RQ mempool name
>       doc: fix typo in mlx5 guide