Re: [dpdk-stable] 20.11.1 patches review and test

[Pei Zhang <pezhang@redhat.com>]

Hello Luca,

The testing with dpdk 20.11.1-rc1 from Red Hat looks good. We tested below 17 scenarios and and all got PASS on RHEL8:

(1)Guest with device assignment(PF) throughput testing(1G hugepage size): PASS
(2)Guest with device assignment(PF) throughput testing(2M hugepage size) : PASS
(3)Guest with device assignment(VF) throughput testing: PASS
(4)PVP (host dpdk testpmd as vswitch) 1Q: throughput testing: PASS
(5)PVP vhost-user 2Q throughput testing: PASS
(6)PVP vhost-user 1Q - cross numa node throughput testing: PASS
(7)Guest with vhost-user 2 queues throughput testing: PASS
(8)vhost-user reconnect with dpdk-client, qemu-server: qemu reconnect: PASS
(9)vhost-user reconnect with dpdk-client, qemu-server: ovs reconnect: PASS
(10)PVP 1Q live migration testing: PASS
(11)PVP 1Q cross numa node live migration testing: PASS
(12)Guest with ovs+dpdk+vhost-user 1Q live migration testing: PASS
(13)Guest with ovs+dpdk+vhost-user 1Q live migration testing (2M): PASS
(14)Guest with ovs+dpdk+vhost-user 2Q live migration testing: PASS
(15)Guest with ovs+dpdk+vhost-user 4Q live migration testing: PASS
(16)Host PF + DPDK testing: PASS
(17)Host VF + DPDK testing: PASS

Versions:

kernel 4.18
qemu 5.2

dpdk: git://dpdk.org/dpdk-stable  
# git log -1
commit fa27a3c11fba9ff30b2c4c4fcc862cea2b654803 (HEAD, tag: v20.11.1-rc1, origin/20.11)
Author: Luca Boccassi <luca.boccassi@microsoft.com>
Date:   Mon Feb 22 14:59:17 2021 +0000

    version: 20.11.1-rc1
    
    Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>


# git branch
remotes/origin/20.11

NICs: X540-AT2 NIC(ixgbe, 10G)

Best regards,

Pei



----- Original Message -----
> From: "luca boccassi" <luca.boccassi@gmail.com>
> To: stable@dpdk.org
> Cc: dev@dpdk.org, "Abhishek Marathe" <Abhishek.Marathe@microsoft.com>, "Akhil Goyal" <akhil.goyal@nxp.com>, "Ali
> Alnubani" <alialnu@nvidia.com>, "benjamin walker" <benjamin.walker@intel.com>, "David Christensen"
> <drc@linux.vnet.ibm.com>, "hariprasad govindharajan" <hariprasad.govindharajan@intel.com>, "Hemant Agrawal"
> <hemant.agrawal@nxp.com>, "Ian Stokes" <ian.stokes@intel.com>, "Jerin Jacob" <jerinj@marvell.com>, "John McNamara"
> <john.mcnamara@intel.com>, "Ju-Hyoung Lee" <juhlee@microsoft.com>, "Kevin Traynor" <ktraynor@redhat.com>, "Luca
> Boccassi" <bluca@debian.org>, "Pei Zhang" <pezhang@redhat.com>, "pingx yu" <pingx.yu@intel.com>, "qian q xu"
> <qian.q.xu@intel.com>, "Raslan Darawsheh" <rasland@nvidia.com>, "Thomas Monjalon" <thomas@monjalon.net>, "yuan peng"
> <yuan.peng@intel.com>, "zhaoyan chen" <zhaoyan.chen@intel.com>
> Sent: Monday, February 22, 2021 11:09:18 PM
> Subject: 20.11.1 patches review and test
> 
> Hi all,
> 
> Here is a list of patches targeted for stable release 20.11.1.
> 
> The planned date for the final release is the 8th of March.
> 
> 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=v20.11.1-rc1
> 
> These patches are located at branch 20.11 of dpdk-stable repo:
>     https://dpdk.org/browse/dpdk-stable/
> 
> Thanks.
> 
> Luca Boccassi
> 
> ---
> Ajit Khaparde (3):
>       net/bnxt: fix cleanup on mutex init failure
>       net/bnxt: fix format specifier for unsigned int
>       net/bnxt: fix freeing mbuf
> 
> Alexander Kozyrev (4):
>       net/mlx5: fix mbuf freeing in vectorized MPRQ
>       net/mlx5: fix flow tag decompression
>       net/mlx5: check FW miniCQE format capabilities
>       net/mlx5: fix miniCQE configuration for Verbs
> 
> Alvin Zhang (9):
>       net/ixgbe: detect failed VF MTU set
>       net/i40e: fix Rx bytes statistics
>       net/iavf: fix queue pairs configuration
>       doc: fix RSS flow description in i40e guide
>       net/i40e: fix returned code for RSS hardware failure
>       net/ice: fix RSS lookup table initialization
>       test: fix buffer overflow in Tx burst
>       net/ixgbe: fix configuration of max frame size
>       app/testpmd: fix key for RSS flow rule
> 
> Amit Bernstein (1):
>       net/ena: fix Tx doorbell statistics
> 
> Anatoly Burakov (1):
>       fbarray: fix overlap check
> 
> Andrew Boyer (5):
>       net/ionic: do minor logging fixups
>       net/ionic: fix link speed and autonegotiation
>       net/ionic: allow separate L3 and L4 checksum offload
>       net/ionic: fix up function attribute tags
>       net/ionic: fix address handling in Tx
> 
> Ankur Dwivedi (1):
>       test/event_crypto: set cipher operation in transform
> 
> Ashish Sadanandan (1):
>       mbuf: add C++ include guard for dynamic fields header
> 
> Balazs Nemeth (1):
>       net/qede: fix promiscuous enable
> 
> Beilei Xing (2):
>       net/i40e: fix global register recovery
>       net/i40e: fix flex payload rule conflict
> 
> Bernard Iremonger (1):
>       doc: fix QinQ flow rules in testpmd guide
> 
> Bruce Richardson (29):
>       ethdev: avoid blocking telemetry for link status
>       build: provide suitable error for "both" libraries option
>       eal: fix reciprocal header include
>       telemetry: fix missing header include
>       ethdev: fix missing header include
>       net: fix missing header include
>       mbuf: fix missing header include
>       bitrate: fix missing header include
>       rib: fix missing header includes
>       vhost: fix missing header includes
>       ipsec: fix missing header include
>       fib: fix missing header includes
>       table: fix missing header include
>       pipeline: fix missing header includes
>       metrics: fix variable declaration in header
>       node: fix missing header include
>       app: fix build with extra include paths
>       build: force pkg-config for dependency detection
>       power: create guest channel public header file
>       power: make channel message functions public
>       power: rename public structs
>       power: rename constants
>       power: export guest channel header file
>       power: clean up includes
>       eal: fix MCS lock header include
>       eal: fix internal ABI tag with clang
>       power: fix missing header includes
>       rib: fix missing header include
>       eal: fix automatic loading of drivers as shared libs
> 
> Chengchang Tang (7):
>       net/hns3: fix register length when dumping registers
>       net/hns3: fix data overwriting during register dump
>       net/hns3: fix dump register out of range
>       net/hns3: fix interrupt resources in Rx interrupt mode
>       net/hns3: fix firmware exceptions by concurrent commands
>       net/hns3: fix VF reset on mailbox failure
>       net/hns3: fix stats flip overflow
> 
> Chengwen Feng (3):
>       net/hns3: fix VF query link status in dev init
>       net/hns3: remove MPLS from supported flow items
>       net/hns3: fix flow director rule residue on malloc failure
> 
> Ciara Power (3):
>       app/crypto-perf: fix spelling in output
>       app/crypto-perf: fix latency CSV output
>       app/crypto-perf: fix CSV output format
> 
> Cristian Dumitrescu (1):
>       examples/pipeline: fix CLI parsing crash
> 
> Dapeng Yu (4):
>       net/ixgbe: fix flex bytes flow director rule
>       net/ice: check Rx queue number on RSS init
>       net/ixgbe: disable NFS filtering
>       app/testpmd: avoid exit without terminal restore
> 
> David Marchand (3):
>       net/hinic: restore vectorised code
>       examples/pipeline: fix VXLAN script permission
>       mbuf: remove unneeded atomic generic header include
> 
> Dekel Peled (8):
>       net/mlx5: fix shared age action validation
>       net/mlx5: fix hairpin flow split decision
>       net/mlx5: fix flow split combined with counter
>       net/mlx5: fix flow split combined with age action
>       net/mlx5: fix shared RSS translation and cleanup
>       app/testpmd: support shared age action query
>       net/mlx5: fix shared RSS capability check
>       net/mlx5: validate hash Rx queue pointer
> 
> Dmitry Kozlyuk (4):
>       eal/windows: fix build with MinGW-w64 8
>       bus/pci: fix build with MinGW-w64 8
>       bus/pci: fix hardware ID limit on Windows
>       build: fix linker flags on Windows
> 
> Eugeny Parshutin (1):
>       doc: add vtune profiling config to prog guide
> 
> Fan Zhang (1):
>       crypto/qat: fix digest in buffer
> 
> Fei Chen (1):
>       vhost: fix vid allocation race
> 
> Feifei Wang (7):
>       test/ring: reduce duration of performance tests
>       app/eventdev: adjust event count order for pipeline test
>       app/eventdev: remove redundant enqueue in burst Tx
>       examples/eventdev: check CPU core enabling
>       examples/eventdev: add info output for main core
>       examples/eventdev: move ethdev stop to the end
>       app/eventdev: fix SMP barrier in performance test
> 
> Ferruh Yigit (13):
>       app/procinfo: fix _filters stats reporting
>       app/procinfo: fix check on xstats-ids
>       app/procinfo: remove useless memset
>       app/procinfo: remove useless assignment
>       net/pcap: remove local variable shadowing outer one
>       net/bonding: remove local variable shadowing outer one
>       net/af_xdp: remove useless assignment
>       net/bnxt: remove redundant return
>       app/crypto-perf: remove always true condition
>       net/avp: remove always true condition
>       net/pcap: fix byte stats for drop Tx
>       net/pcap: fix infinite Rx with large files
>       app/testpmd: fix help of metering commands
> 
> Gaetan Rivet (2):
>       net/bonding: fix port id validity check on parsing
>       net/bonding: fix PCI address comparison on non-PCI ports
> 
> Gagandeep Singh (2):
>       test/ipsec: fix result code for not supported
>       crypto/dpaa2_sec: fix memory allocation check
> 
> George Prekas (1):
>       app/testpmd: fix IP checksum calculation
> 
> Gregory Etelson (5):
>       net/mlx5: fix Direct Verbs flow descriptor allocation
>       app/testpmd: release flows left before port stop
>       net/mlx5: fix tunnel rules validation on VF representor
>       net/mlx5: fix mark action in active tunnel offload
>       net/mlx5: fix drop action in tunnel offload mode
> 
> Guy Kaneti (1):
>       regex/octeontx2: fix PCI table overflow
> 
> Haiyue Wang (2):
>       net/ice: drain out DCF AdminQ command queue
>       net/ixgbe: fix UDP zero checksum on x86
> 
> Harman Kalra (1):
>       examples/l3fwd: remove limitation on Tx queue count
> 
> Harry van Haaren (1):
>       eventdev: fix a return value comment
> 
> Heinrich Kuhn (1):
>       net/nfp: read chip model from PluDevice register
> 
> Hemant Agrawal (1):
>       app/procinfo: fix security context info
> 
> Hongbo Zheng (1):
>       net/hns3: use new opcode for clearing hardware resource
> 
> Huisong Li (7):
>       app/testpmd: fix queue stats mapping configuration
>       net/hns3: fix xstats with id and names
>       net/hns3: fix error code in xstats
>       net/hns3: fix Rx/Tx errors stats
>       net/hns3: fix link status change from firmware
>       net/hns3: validate requested maximum Rx frame length
>       net/hns3: fix query order of link status and link info
> 
> Hyong Youb Kim (2):
>       net/enic: fix filter type used for flow API
>       net/enic: fix filter log message
> 
> Ido Segev (1):
>       net/ena: flush Rx buffers memory pool cache
> 
> Igor Chauskin (2):
>       net/ena: fix Tx SQ free space assessment
>       net/ena: prevent double doorbell
> 
> Igor Ryzhov (1):
>       net/i40e: fix stats counters
> 
> Ivan Malov (11):
>       common/sfc_efx/base: remove warnings about inline specifiers
>       common/sfc_efx/base: fix signed/unsigned mismatch warnings
>       common/sfc_efx/base: support alternative MAE match fields
>       common/sfc_efx/base: update MCDI headers for MAE privilege
>       common/sfc_efx/base: check for MAE privilege
>       common/sfc_efx/base: fix MPORT related byte order handling
>       common/sfc_efx/base: fix MAE match spec validation helper
>       common/sfc_efx/base: fix MAE match spec class comparison API
>       common/sfc_efx/base: enhance field ID check in field set API
>       common/sfc_efx/base: apply mask to value on match field set
>       net/sfc: fix TSO and checksum offloads for EF10
> 
> Jiawei Wang (4):
>       net/mlx5: fix unnecessary checking for RSS action
>       app/testpmd: fix packets dump overlapping
>       net/mlx5: fix count actions query in sample flow
>       net/mlx5: fix counter and age flow action validation
> 
> Jiawei Zhu (1):
>       net/virtio-user: fix run closing stdin and close callfd
> 
> Jingjing Wu (1):
>       net/iavf: fix vector mapping with queue
> 
> John McNamara (1):
>       license: add licenses for exception cases
> 
> Joyce Kong (1):
>       eal/arm: fix debug build with gcc for 128-bit atomics
> 
> Junfeng Guo (1):
>       net/iavf: fix GTPU UL and DL support for flow director
> 
> Kalesh AP (4):
>       net/bnxt: release HWRM lock in error
>       net/bnxt: propagate FW command failure to application
>       net/bnxt: fix VNIC RSS configure function
>       net/bnxt: fix FW version log
> 
> Karra Satwik (2):
>       net/cxgbe: accept VLAN flow items without ethertype
>       app/testpmd: fix start index for showing FEC array
> 
> Lance Richardson (10):
>       net/bnxt: disable end of packet padding for Rx
>       net/bnxt: limit Rx representor packets per poll
>       net/bnxt: fix doorbell write ordering
>       net/bnxt: fix outer UDP checksum Rx offload capability
>       net/bnxt: make offload flags mapping per-ring
>       net/bnxt: set correct checksum status in mbuf
>       net/bnxt: fix packet type index calculation
>       net/bnxt: fix null termination of Rx mbuf chain
>       net/bnxt: fix fallback mbuf allocation logic
>       net/bnxt: fix Rx completion ring size calculation
> 
> Leyi Rong (1):
>       net/ice: enlarge Rx queue rearm threshold to 64
> 
> Lijun Ou (6):
>       net/hns3: fix interception with flow director
>       net/hns3: fix memory leak on secondary process exit
>       net/hns3: adjust some comments
>       net/hns3: adjust format specifier for enum
>       doc: fix product link in hns3 guide
>       net/hns3: fix RSS indirection table size
> 
> Liron Himi (5):
>       net/octeontx2: fix PF flow action for Tx
>       net/mvpp2: remove debug log on fast-path
>       net/mvpp2: remove VLAN flush
>       net/mvpp2: remove CRC length from MRU validation
>       net/mvpp2: fix frame size checking
> 
> Long Li (1):
>       net/netvsc: ignore unsupported packet on sync command
> 
> Lukasz Wojciechowski (1):
>       test/distributor: fix return buffer queue overload
> 
> Marvin Liu (1):
>       vhost: fix packed ring dequeue offloading
> 
> Matan Azrad (1):
>       vdpa/mlx5: fix configuration mutex cleanup
> 
> Maxime Coquelin (3):
>       net/virtio: add missing backend features negotiation
>       net/virtio: fix memory init with vDPA backend
>       net/virtio: fix getting old status on reconnect
> 
> Michael Baum (7):
>       net/mlx5: fix leak on Rx queue creation failure
>       net/mlx5: fix leak on Tx queue creation failure
>       common/mlx5: fix completion queue entry size configuration
>       net/mlx5: remove CQE padding device argument
>       net/mlx5: fix leak on ASO SQ creation failure
>       net/mlx4: fix device detach
>       net/mlx4: fix handling of probing failure
> 
> Michal Krawczyk (1):
>       net/ena: validate Rx req ID upon acquiring descriptor
> 
> Min Hu (Connor) (3):
>       net/hns3: fix FEC state query
>       net/hns3: fix crash with multi-process
>       doc: add FEC to NIC features
> 
> Murphy Yang (6):
>       net/ice: fix outer UDP Tx checksum offload
>       net/i40e: fix L4 checksum flag
>       net/ice: fix outer checksum flags
>       net/iavf: fix conflicting RSS combination rules
>       net/ice: disable IPv4 checksum offload in vector Tx
>       net/i40e: add null input checks
> 
> Nick Connolly (2):
>       eal/windows: fix debug build with MinGW
>       eal/windows: fix vfprintf warning with clang
> 
> Olivier Matz (5):
>       build: fix plugin load on static build
>       net/virtio-user: fix protocol features advertising
>       service: propagate init error in EAL
>       test/mcslock: remove unneeded per lcore copy
>       mempool: fix panic on dump or audit
> 
> Ophir Munk (4):
>       net/mlx5: fix freeing packet pacing
>       net/mlx5: fix flow action destroy wrapper
>       net/mlx5: fix flow operation wrapper per OS
>       net/mlx5: unify operations for all OS
> 
> Ori Kam (3):
>       regex/mlx5: fix memory rule alignment
>       regex/mlx5: fix support for group id
>       regex/mlx5: fix number of supported queues
> 
> Qi Zhang (4):
>       doc: fix some statements for ice vector PMD
>       net/ice/base: fix tunnel destroy
>       net/ice/base: fix null pointer dereference
>       net/ice/base: fix memory handling
> 
> Ruifeng Wang (4):
>       lpm: fix vector IPv4 lookup
>       net/hns3: fix build with SVE
>       net/octeontx: fix build with SVE
>       common/octeontx2: fix build with SVE
> 
> Samik Gupta (2):
>       net/bnxt: fix Rx rings in RSS redirection table
>       net/bnxt: fix VNIC config on Rx queue stop
> 
> Shiri Kuzin (2):
>       net/mlx5: fix VXLAN decap on non-VXLAN flow
>       net/mlx5: refuse empty VLAN in flow pattern
> 
> Somnath Kotur (4):
>       net/bnxt: fix PF resource query
>       net/bnxt: fix lock init and destroy
>       net/bnxt: fix error handling in device start
>       net/bnxt: refactor init/uninit
> 
> Souvik Dey (2):
>       net/i40e: fix VLAN stripping in VF
>       common/mlx5: fix storing synced MAC to internal table
> 
> Sriharsha Basavapatna (1):
>       net/bnxt: fix max rings computation
> 
> Stephen Hemminger (2):
>       test/rwlock: fix spelling and missing whitespace
>       test: fix terminal settings on exit
> 
> Steve Yang (23):
>       ethdev: fix max Rx packet length check
>       app/testpmd: fix max Rx packet length for VLAN packets
>       net/dpaa: fix jumbo frame flag condition for MTU set
>       net/dpaa2: fix jumbo frame flag condition for MTU set
>       net/e1000: fix jumbo frame flag condition for MTU set
>       net/hns3: fix jumbo frame flag condition for MTU set
>       net/i40e: fix jumbo frame flag condition
>       net/iavf: fix jumbo frame flag condition
>       net/ice: fix jumbo frame flag condition
>       net/ipn3ke: fix jumbo frame flag condition for MTU set
>       net/octeontx: fix jumbo frame flag condition for MTU set
>       net/octeontx2: fix jumbo frame flag condition for MTU
>       net/qede: fix jumbo frame flag condition for MTU set
>       net/sfc: fix jumbo frame flag condition for MTU set
>       net/thunderx: fix jumbo frame flag condition for MTU set
>       net/ixgbe: fix jumbo frame flag condition
>       net/cxgbe: fix jumbo frame flag condition
>       net/axgbe: fix jumbo frame flag condition for MTU set
>       net/enetc: fix jumbo frame flag condition for MTU set
>       net/hinic: fix jumbo frame flag condition for MTU set
>       net/nfp: fix jumbo frame flag condition for MTU set
>       net/liquidio: fix jumbo frame flag condition for MTU set
>       app/testpmd: fix setting maximum packet length
> 
> Suanming Mou (5):
>       net/mlx5: fix shared RSS and mark actions combination
>       net/mlx5: fix multi-process port ID
>       net/mlx5: fix crash on secondary process port close
>       net/mlx5: fix port attach in secondary process
>       net/mlx4: fix port attach in secondary process
> 
> Sunil Kumar Kori (2):
>       net/octeontx2: fix corruption in segments list
>       net/octeontx: fix max Rx packet length
> 
> Tal Shnaiderman (5):
>       bus/pci: ignore missing NUMA node on Windows
>       net/mlx5: fix constant array size
>       net/mlx5: fix device name size on Windows
>       net/mlx5: fix comparison sign in flow engine
>       common/mlx5: fix pointer cast on Windows
> 
> Thomas Monjalon (3):
>       doc: fix figure numbering in graph guide
>       lib: fix doxygen for parameters of function pointers
>       ethdev: fix close failure handling
> 
> Timothy McDaniel (1):
>       event/dlb: fix accessing uninitialized variables
> 
> Ting Xu (1):
>       net/iavf: fix memory leak in large VF
> 
> Tyler Retzlaff (2):
>       bus/pci: fix build with Windows SDK >= 10.0.20253
>       eal/windows: fix C++ compatibility
> 
> Viacheslav Galaktionov (1):
>       net/sfc: fix generic byte statistics to exclude FCS bytes
> 
> Viacheslav Ovsiienko (8):
>       net/mlx5: fix Verbs memory allocation callback
>       net/mlx5: fix buffer split offload advertising
>       doc: update flow mark action in mlx5 guide
>       net/mlx5: fix wire vport hint
>       app/testpmd: fix queue reconfig request on Rx split update
>       doc: fix supported feature table in mlx5 guide
>       doc: fix mark action zero value in mlx5 guide
>       net/mlx5: fix Tx queue size created with DevX
> 
> Vladimir Medvedkin (2):
>       rib: fix insertion in some cases
>       crypto/qat: fix access to uninitialized variable
> 
> Weifeng Li (1):
>       net/i40e: fix X722 for 802.1ad frames ability
> 
> Wenjun Wu (1):
>       net/e1000: fix flow control mode setting
> 
> Wisam Jaddo (1):
>       app/flow-perf: simplify objects initialization
> 
> Xuan Ding (1):
>       net/iavf: fix symmetric flow rule creation
> 
> Yicai Lu (1):
>       ip_frag: remove padding length of fragment
> 
> Yongxin Liu (1):
>       usertools: fix binding built-in kernel driver
> 
> Yunjian Wang (3):
>       eal/linux: fix handling of error events from epoll
>       net/bnxt: fix memory leak when mapping fails
>       net/mvneta: check allocation in Rx queue flush
> 
> Yuri Chipchev (1):
>       net/mvpp2: fix stack corruption
> 
>