Re: 21.11.3 patches review and test

[Christian Ehrhardt <christian.ehrhardt@canonical.com>]

On Tue, Dec 6, 2022 at 12:30 PM Kevin Traynor <ktraynor@redhat.com> wrote:
>
> Hi all,
>
> Here is a list of patches targeted for stable release 21.11.3.
>
> The planned date for the final release is 19th December.
>
> 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=v21.11.3-rc1
>
> These patches are located at branch 21.11 of dpdk-stable repo:
>     https://dpdk.org/browse/dpdk-stable/

Hi Kevin,
testing 22.11.3 builds and a bunch of workloads and stressing things
under Ubuntu 22.04.
TL;DR: for what I ran all LGTM, no regressions found (but I only have
a small subset of HW and use cases, I hope others will reply as well
later this week).

V: 21.11.3~rc1-0ubuntu0.22.04.1~jammyppa1

1.0.0 (08:19:13): phys (BM) tests
  1.1.0 (08:19:13): initialize environment
    1.1.1 (08:21:02): testpmd  => Pass
    1.1.2 (08:23:07): check testpmd output  => Pass
2.0.0 (08:23:07): prep virtual test environment

1.0.0 (08:25:37): virt tests
  1.1.0 (08:25:37): initialize environment
3.0.0 (08:27:02): performance tests
  3.1.0 (08:27:02): prep benchmarks
  3.2.0 (08:27:24): performance tests
    3.2.1 (08:27:32): test guest-openvswitch for OVS-5CPU  => Pass
    3.2.2 (08:46:45): test guest-dpdk-vhost-user-client-multiq for
OVSDPDK-VUC  => Pass
4.0.0 (09:06:10): VUC endurance checks
  4.1.0 (09:06:10): prep VUC endurance tests
    4.1.1 (09:18:48): start stop guests (client)  => Pass
    4.1.2 (10:31:22): add/remove ports (client)  => Pass
  4.2.0 (10:40:52): Final cleanup

> Thanks.
>
> Kevin
>
> ---
> Abdullah Sevincer (1):
>       event/dlb2: handle enqueuing more than maximum depth
>
> Abhimanyu Saini (1):
>       common/sfc_efx/base: remove VQ index check during VQ start
>
> Aleksandr Miloshenko (1):
>       net/iavf: fix Tx done descriptors cleanup
>
> Alex Kiselev (1):
>       net/tap: fix overflow of network interface index
>
> Alexander Chernavin (1):
>       net/virtio: fix crash when configured twice
>
> Alexander Kozyrev (3):
>       net/mlx5: fix shared Rx queue config reuse
>       net/mlx5: fix first segment inline length
>       net/mlx5: fix indexed pool local cache crash
>
> Ali Alnubani (1):
>       examples/l2fwd-crypto: fix typo in error message
>
> Amit Prakash Shukla (6):
>       net/mvneta: fix build with GCC 12
>       test/ipsec: fix build with GCC 12
>       ipsec: fix build with GCC 12
>       crypto/qat: fix build with GCC 12
>       net/i40e: fix build with MinGW GCC 12
>       net/qede/base: fix 32-bit build with GCC 12
>
> Andrew Boyer (5):
>       net/ionic: fix endianness for Rx and Tx
>       net/ionic: fix endianness for RSS
>       net/ionic: fix adapter name for logging
>       net/ionic: fix Rx filter save
>       net/ionic: fix reported error stats
>
> Anoob Joseph (1):
>       test/crypto: fix PDCP vectors
>
> Apeksha Gupta (2):
>       net/enetfec: fix restart
>       net/enetfec: fix buffer leak
>
> Arek Kusztal (1):
>       common/qat: fix VF to PF answer
>
> Ashwin Sekhar T K (1):
>       mempool/cnxk: fix destroying empty pool
>
> Ben Magistro (1):
>       doc: fix dumpcap interface parameter option
>
> Benjamin Le Berre (1):
>       net/bnxt: fix error code during MTU change
>
> Bhagyada Modali (9):
>       net/axgbe: fix scattered Rx
>       net/axgbe: fix mbuf lengths in scattered Rx
>       net/axgbe: fix length of each segment in scattered Rx
>       net/axgbe: fix checksum and RSS in scattered Rx
>       net/axgbe: optimise scattered Rx
>       net/axgbe: remove freeing buffer in scattered Rx
>       net/axgbe: reset end of packet in scattered Rx
>       net/axgbe: clear buffer on scattered Rx chaining failure
>       net/axgbe: save segment data in scattered Rx
>
> Bing Zhao (2):
>       net/mlx5: fix build with recent compilers
>       bus/auxiliary: prevent device from being probed again
>
> Brian Dooley (1):
>       crypto/qat: fix null hash algorithm digest size
>
> Changpeng Liu (1):
>       vhost: add non-blocking API for posting interrupt
>
> Chaoyong He (1):
>       net/nfp: fix Rx descriptor DMA address
>
> Chengwen Feng (8):
>       net/hns3: fix crash in SVE Tx
>       net/hns3: fix next-to-use overflow in SVE Tx
>       net/hns3: fix next-to-use overflow in simple Tx
>       net/hns3: fix crash when secondary process access FW
>       net/hns3: revert Tx performance optimization
>       net/hns3: revert fix mailbox communication with HW
>       net/hns3: fix VF mailbox message handling
>       app/testpmd: remove jumbo offload
>
> Ciara Power (1):
>       test/crypto: fix wireless auth digest segment
>
> Conor Walsh (1):
>       doc: fix reference to dma application example
>
> Dariusz Sosnowski (1):
>       net/mlx5: fix hairpin split with set VLAN VID action
>
> David Marchand (23):
>       vhost: fix virtqueue use after free on NUMA reallocation
>       app/testpmd: restore ixgbe bypass commands
>       net/failsafe: fix interrupt handle leak
>       net/bnxt: fix build with GCC 13
>       trace: fix mode for new trace point
>       trace: fix mode change
>       trace: fix leak with regexp
>       trace: fix dynamically enabling trace points
>       trace: fix race in debug dump
>       ci: bump versions of actions in GHA
>       ci: update to new API for step outputs in GHA
>       service: fix build with clang 15
>       vhost: fix build with clang 15
>       bus/dpaa: fix build with clang 15
>       net/atlantic: fix build with clang 15
>       net/dpaa2: fix build with clang 15
>       app/testpmd: fix build with clang 15
>       app/testpmd: fix build with clang 15 in flow code
>       test/efd: fix build with clang 15
>       test/member: fix build with clang 15
>       test/event: fix build with clang 15
>       ci: enable ABI check in GHA
>       trace: fix metadata dump
>
> Dmitry Kozlyuk (4):
>       build: enable developer mode for all working trees
>       eal: fix side effect in some pointer arithmetic macros
>       mempool: make event callbacks process-private
>       common/mlx5: fix multi-process mempool registration
>
> Dong Zhou (1):
>       net/mlx5: fix thread workspace memory leak
>
> Dongdong Liu (2):
>       doc: fix application name in procinfo guide
>       doc: document device dump in procinfo guide
>
> Erik Gabriel Carrillo (1):
>       service: fix early move to inactive status
>
> Fidaullah Noonari (1):
>       malloc: fix storage size for some allocations
>
> Frank Du (1):
>       net/ice: fix interrupt handler unregister
>
> Gagandeep Singh (5):
>       net/dpaa: fix buffer freeing in slow path
>       net/dpaa: use internal mempool for SG table
>       net/dpaa: fix buffer freeing on SG Tx
>       net/dpaa2: use internal mempool for SG table
>       net/dpaa2: fix buffer freeing on SG Tx
>
> Ganapati Kundapura (1):
>       eventdev/crypto: fix multi-process
>
> Gregory Etelson (6):
>       net/mlx5: fix RSS expansion buffer size
>       app/testpmd: fix MAC header in checksum forward engine
>       common/mlx5: fix shared mempool subscription
>       net/mlx5: fix port initialization with small LRO
>       net/mlx5: fix maximum LRO message size
>       doc: add LRO size limitation in mlx5 guide
>
> Haiyue Wang (1):
>       ring: fix description
>
> Hamza Khan (1):
>       examples/vm_power_manager: use safe list iterator
>
> Hanumanth Pothula (1):
>       net/cnxk: fix DF bit in vector mode
>
> Hernan Vargas (14):
>       baseband/acc100: fix memory leak
>       baseband/acc100: check turbo dec/enc input
>       baseband/acc100: add null checks
>       baseband/acc100: fix input length for CRC24B
>       baseband/acc100: fix clearing PF IR outside handler
>       baseband/acc100: fix device minimum alignment
>       baseband/acc100: fix close cleanup
>       baseband/acc100: add LDPC encoder padding function
>       baseband/acc100: check AQ availability
>       baseband/acc100: fix ring availability calculation
>       baseband/acc100: enforce additional check on FCW
>       baseband/acc100: fix null HARQ input case
>       baseband/acc100: fix ring/queue allocation
>       baseband/acc100: fix double MSI intr in TB mode
>
> Huisong Li (18):
>       net/hns3: fix Rx with PTP
>       net/hns3: delete unused markup
>       net/hns3: fix clearing hardware MAC statistics
>       net/hns3: fix RSS filter restore
>       net/hns3: fix RSS flow rule restore
>       net/hns3: move flow direction rule recovery
>       net/hns3: fix packet type for GENEVE
>       net/hns3: fix IPv4 and IPv6 RSS
>       net/hns3: fix typos in IPv6 SCTP fields
>       net/hns3: fix IPv4 RSS
>       net/hns3: add L3 and L4 RSS types
>       net/bonding: fix slave device Rx/Tx offload configuration
>       net/bonding: fix dropping valid MAC packets
>       net/bonding: fix mbuf fast free handling
>       net/hns3: extract functions to create RSS and FDIR flow rule
>       net/hns3: fix RSS rule restore
>       net/hns3: fix lock protection of RSS flow rule
>       net/hns3: fix restore filter function input
>
> Huzaifa Rahman (1):
>       net/memif: fix crash with different number of Rx/Tx queues
>
> Ilya Maximets (1):
>       doc: fix support table for Ethernet/VLAN flow items
>
> Ivan Malov (3):
>       common/sfc_efx/base: fix maximum Tx data count
>       net/bonding: fix descriptor limit reporting
>       net/bonding: fix flow flush order on close
>
> James Hershaw (1):
>       net/nfp: improve HW info header log readability
>
> Jeremy Spewock (1):
>       test/ipsec: skip if no compatible device
>
> Jerin Jacob (2):
>       eal: fix doxygen comments for UUID
>       power: fix some doxygen comments
>
> Jiawei Wang (4):
>       net/mlx5: fix modify action with tunnel decapsulation
>       net/mlx5: fix tunnel header with IPIP offload
>       net/mlx5: fix source port checking in sample flow rule
>       net/mlx5: fix mirror flow validation with ASO action
>
> Jiawen Wu (6):
>       net/txgbe: fix IPv6 flow rule
>       net/txgbe: remove semaphore between SW/FW
>       net/txgbe: rename some extended statistics
>       net/ngbe: rename some extended statistics
>       net/ngbe: remove semaphore between SW/FW
>       net/ngbe: fix maximum frame size
>
> Jie Hai (1):
>       net/hns3: fix minimum Tx frame length
>
> Jie Wang (1):
>       net/i40e: fix jumbo frame Rx with X722
>
> Jun Qiu (3):
>       gro: trim tail padding bytes
>       net/bonding: fix Tx hash for TCP
>       hash: fix RCU configuration memory leak
>
> Kai Ji (1):
>       test/crypto: fix bitwise operator in a SNOW3G case
>
> Kalesh AP (2):
>       net/bnxt: remove unnecessary check
>       net/bnxt: fix representor info freeing
>
> Ke Zhang (2):
>       net/i40e: fix VF representor release
>       net/iavf: fix L3 checksum Tx offload flag
>
> Kevin Liu (2):
>       net/iavf: check illegal packet sizes
>       net/ice: check illegal packet sizes
>
> Kevin Traynor (1):
>       Revert "cryptodev: fix missing SHA3 algorithm strings"
>
> Kumara Parameshwaran (1):
>       gro: check payload length after trim
>
> Long Li (2):
>       net/mlx4: fix Verbs FD leak in secondary process
>       net/mlx5: fix Verbs FD leak in secondary process
>
> Long Wu (1):
>       net/nfp: fix memory leak in Rx
>
> Luca Boccassi (1):
>       drivers: fix typos found by Lintian
>
> Mao YingMing (1):
>       net/bnxt: fix null pointer dereference in LED config
>
> Mattias Rönnblom (3):
>       net: accept unaligned data in checksum routines
>       event/dsw: fix flow migration
>       doc: fix event timer adapter guide
>
> Maxime Coquelin (1):
>       vhost: fix build with GCC 12
>
> Megha Ajmera (2):
>       sched: fix subport profile configuration
>       examples/qos_sched: fix number of subport profiles
>
> Michael Baum (5):
>       net/mlx5: fix null check in devargs parsing
>       doc: fix underlines in testpmd guide
>       doc: fix colons in testpmd aged flow rules
>       net/mlx5: fix race condition in counter pool resizing
>       net/mlx5: fix port event cleaning order
>
> Mingjin Ye (4):
>       net/ice: support VXLAN-GPE tunnel offload
>       net/i40e: fix pctype configuration for X722
>       net/ice: fix scalar Rx path segment
>       net/ice: fix scalar Tx path segment
>
> Mário Kuka (1):
>       pcapng: fix write more packets than IOV_MAX limit
>
> Naga Harish K S V (4):
>       eventdev/eth_tx: add spinlock for adapter start/stop
>       eventdev/eth_tx: fix adapter stop
>       timer: fix stopping all timers
>       eventdev/eth_tx: fix queue delete
>
> Nithin Dabilpuram (3):
>       examples/ipsec-secgw: use Tx checksum offload conditionally
>       examples/l3fwd: fix MTU configuration with event mode
>       net/cnxk: fix later skip to include mbuf private data
>
> Olivier Matz (7):
>       cryptodev: fix unduly newlines in logs
>       mem: fix API doc about allocation on secondary processes
>       event/sw: fix flow ID init in self test
>       event/sw: fix log in self test
>       net/ixgbe: fix broadcast Rx on VF after promisc removal
>       net/ixgbe: fix unexpected VLAN Rx in promisc mode on VF
>       net/ixgbevf: fix promiscuous and allmulti
>
> Pablo de Lara (1):
>       examples/fips_validation: fix typo in error log
>
> Pavan Nikhilesh (3):
>       event/cnxk: fix missing xstats operations
>       event/cnxk: fix mbuf offset calculation
>       event/cnxk: fix missing mempool cookie marking
>
> Peng Zhang (3):
>       net/nfp: compose firmware file name with new hwinfo
>       buildtools: fix NUMA nodes count
>       net/nfp: fix internal buffer size and MTU check
>
> Qi Zhang (12):
>       net/ice/base: fix division during E822 PTP init
>       net/ice/base: fix 100M speed capability
>       net/ice/base: fix DSCP PFC TLV creation
>       net/ice/base: fix media type of PHY 10G SFI C2C
>       net/ice/base: fix function descriptions for parser
>       net/ice/base: fix endian format
>       net/ice/base: fix array overflow in add switch recipe
>       net/ice/base: fix bit finding range over ptype bitmap
>       net/ice/base: fix add MAC rule
>       net/ice/base: fix double VLAN in promiscuous mode
>       net/ice/base: ignore promiscuous already exist
>       net/ice/base: fix input set of GTPoGRE
>
> Qiming Yang (1):
>       app/testpmd: skip port reset in secondary process
>
> Radu Nicolau (5):
>       net/iavf: update IPsec ESN values when updating session
>       net/iavf: fix IPsec flow create error check
>       net/iavf: fix SPI check
>       net/iavf: fix queue stop for large VF
>       examples/ipsec-secgw: fix Tx checksum offload flag
>
> Raja Zidane (1):
>       net/mlx5: fix Tx check for hardware descriptor length
>
> Rohit Raj (1):
>       net/dpaa: fix jumbo packet Rx in case of VSP
>
> Satha Rao (1):
>       common/cnxk: fix schedule weight update
>
> Satheesh Paul (3):
>       common/cnxk: fix log level during MCAM allocation
>       common/cnxk: fix missing flow counter reset
>       common/cnxk: fix printing disabled MKEX registers
>
> Shiqi Liu (2):
>       node: check Rx element allocation
>       dma/idxd: check DSA device allocation
>
> Shun Hao (4):
>       net/mlx5: fix meter profile delete after disable
>       net/mlx5: fix action flag data type
>       net/mlx5: fix drop action validation
>       net/mlx5: fix assert when creating meter policy
>
> Stephen Coleman (1):
>       doc: fix typo depreciated instead of deprecated
>
> Stephen Hemminger (8):
>       event/sw: fix device name in dump
>       eal: fix data race in multi-process support
>       pdump: do not allow enable/disable in primary process
>       app/dumpcap: fix crash on cleanup
>       app/dumpcap: fix pathname for output file
>       app/testpmd: make quit flag volatile
>       ring: remove leftover comment about watermark
>       doc: avoid meson deprecation in setup
>
> Steve Yang (1):
>       net/iavf: fix pattern check for flow director parser
>
> Steven Zou (1):
>       common/iavf: avoid copy in async mode
>
> Sunyang Wu (1):
>       test/crypto: fix debug messages
>
> Taekyung Kim (1):
>       vdpa/ifc: handle data path update failure
>
> Tal Shnaiderman (1):
>       net/mlx5: fix thread termination check on Windows
>
> Thomas Monjalon (2):
>       drivers: remove unused build variable
>       doc: add Rx buffer split capability for mlx5
>
> Ting Xu (1):
>       net/ice/base: fix inner symmetric RSS hash in raw flow
>
> Tomasz Jonak (1):
>       net/ice: fix null function pointer call
>
> Vanshika Shukla (1):
>       net/dpaa2: fix DPDMUX error behaviour
>
> Viacheslav Ovsiienko (3):
>       net/mlx5: fix check for orphan wait descriptor
>       net/mlx5: fix single not inline packet storing
>       net/mlx5: fix inline length exceeding descriptor limit
>
> Vladimir Medvedkin (2):
>       test/hash: remove dead code in extendable bucket test
>       test/hash: fix bulk lookup check
>
> Volodymyr Fialko (3):
>       cryptodev: fix missing SHA3 algorithm strings
>       eventdev: fix name of Rx conf type in documentation
>       app/eventdev: fix limits in error message
>
> Wenwu Ma (1):
>       examples/vhost: fix use after free
>
> Wenzhuo Lu (1):
>       net/iavf: fix VLAN offload
>
> Yi Li (1):
>       doc: fix maximum packet size of virtio driver
>
> Yiding Zhou (4):
>       net/iavf: fix VLAN insertion
>       net/iavf: revert VLAN insertion fix
>       net/ice/base: fix duplicate flow rules
>       net/iavf: add thread for event callbacks
>
> Yunjian Wang (2):
>       net/bonding: fix array overflow in Rx burst
>       net/bonding: fix double slave link status query
>
> Zhichao Zeng (3):
>       net/ice: fix RSS hash update
>       net/iavf: fix processing VLAN TCI in SSE path
>       net/iavf: fix outer checksum flags
>
> Zhirun Yan (1):
>       graph: fix node objects allocation
>


-- 
Christian Ehrhardt
Senior Staff Engineer, Ubuntu Server
Canonical Ltd