Re: [dpdk-dev] 20.11.3 patches review and test

["Jiang, YuX" <yux.jiang@intel.com>]

Hi Luca,

Testing with dpdk v20.11.3-rc1 from Intel looks good, no critical issue is found.
Below issues have been fixed in 20.11.3-rc1:
1)ddp_gtp_qregion/fd_gtpu_ipv4_dstip: flow director does not work.
2)rxtx_offload/rxoffload_port: Pkt1 can't be distributed to the same queue. 

Test Scope:
# Basic Intel(R) NIC testing
* Build: cover the build test combination with latest GCC/Clang/ICC version and the popular OS revision such as Ubuntu20.04, CentOS8.3, Fedora34, RHEL8.3 and so on. Listed but not all.
- All passed. 
*PF(i40e, ixgbe): test scenarios including rte_flow/TSO/Jumboframe/checksum offload/Tunnel, etc. Listed but not all.
- All test is done, no new issue is found.
Known issues as below:
  1)https://bugs.dpdk.org/show_bug.cgi?id=790 unit_tests_power/power_cpufreq: unit test failed: Clone from https://bugs.dpdk.org/show_bug.cgi?id=687 , this issue is found in 21.05/LTS20.11.2, fixed in 21.08. But still failed in LTS20.11.3-rc1.
*VF(i40e,ixgbe): test scenarios including vf-rte_flow/TSO/Jumboframe/checksum offload/Tunnel, Listed but not all.
- All passed. No new issues are found.              
*PF/VF(ice): test scenarios including switch features/Flow Director/Advanced RSS/ACL/DCF/Flexible Descriptor and so on, Listed but not all.
- All test is done, no new issue is found.
Two known DPDK issues: 
1)cvl_advanced_iavf_rss: change the SCTP port value, the hash value remains unchanged. This issue is found in 20.11-rc3, fixed in 21.02, but it’s belong to 21.02 new feature, won’t backporting to LTS20.11.
2)Can't create 512 acl rules after creating a full mask switch rule. This issue is also occurred in dpdk 20.11. Has fixed patch, but not merged.
  Fixed patch link: https://patchwork.dpdk.org/project/dpdk/patch/20210728022429.237010-1-simei.su@intel.com/ 
* Intel NIC single core/NIC performance: test scenarios including PF/VF single core performance test(AVX2+AVX512) test and so on. Listed but not all.
- All passed. No big performance data drop. 

# Basic cryptodev and virtio testing
* Virtio: both function and performance test are covered. Such as PVP/Virtio_loopback/virtio-user loopback/virtio-net VM2VM perf testing, etc.. Listed but not all.
- All test is done, no new issue is found.
One known issues as below:
1)The UDP fragmentation offload feature of Virtio-net device can’t be turned on in the VM, long time kernel issue, bugzilla has been submited: https://bugzilla.kernel.org/show_bug.cgi?id=207075 , not fixed yet.                     
* Cryptodev: 
  **Function test: test scenarios including Cryptodev API testing/CompressDev ISA-L/QAT/ZLIB PMD Testing/FIPS, etc. Listed but not all.
  - All passed.
  **Performance test: test scenarios including Thoughput Performance /Cryptodev Latency, etc. Listed but not all.
  - No big performance data drop.

Best regards,
Yu Jiang

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of luca.boccassi@gmail.com
> Sent: Monday, August 9, 2021 4:44 PM
> 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>;
> Walker, Benjamin <benjamin.walker@intel.com>; David Christensen
> <drc@linux.vnet.ibm.com>; Govindharajan, Hariprasad
> <hariprasad.govindharajan@intel.com>; Hemant Agrawal
> <hemant.agrawal@nxp.com>; Stokes, Ian <ian.stokes@intel.com>; Jerin
> Jacob <jerinj@marvell.com>; Mcnamara, John <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>; Yu, PingX <pingx.yu@intel.com>; Xu, Qian Q
> <qian.q.xu@intel.com>; Raslan Darawsheh <rasland@nvidia.com>; Thomas
> Monjalon <thomas@monjalon.net>; Peng, Yuan <yuan.peng@intel.com>;
> Chen, Zhaoyan <zhaoyan.chen@intel.com>
> Subject: [dpdk-dev] 20.11.3 patches review and test
> 
> Hi all,
> 
> Here is a list of patches targeted for stable release 20.11.3.
> 
> The planned date for the final release is August 23rd.
> 
> 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.3-rc1
> 
> These patches are located at branch 20.11 of dpdk-stable repo:
>     https://dpdk.org/browse/dpdk-stable/
> 
> Thanks.
> 
> Luca Boccassi
> 
> ---
> Abhinandan Gujjar (1):
>       test/crypto: fix mempool size for session-less
> 
> Ajit Khaparde (3):
>       doc: fix default burst size in testpmd
>       net/bnxt: fix Rx interrupt setting
>       net/bnxt: fix ring allocation and free
> 
> Akhil Goyal (1):
>       crypto/octeontx: fix freeing after device release
> 
> Alexander Kozyrev (1):
>       net/mlx5: fix threshold for mbuf replenishment in MPRQ
> 
> Anoob Joseph (1):
>       crypto/octeontx2: fix IPsec session member overlap
> 
> Arek Kusztal (1):
>       crypto/qat: disable asymmetric crypto on GEN3
> 
> Asaf Penso (1):
>       net/mlx5: fix flow engine type in function name
> 
> Beilei Xing (2):
>       net/iavf: fix scalar Rx
>       net/i40e: fix flow director input set conflict
> 
> Bing Zhao (1):
>       net/mlx5: fix queue leaking in hairpin auto bind check
> 
> Chenbo Xia (1):
>       net/virtio: fix default duplex mode
> 
> Cheng Jiang (1):
>       net/virtio: fix refill order in packed ring datapath
> 
> Chengchang Tang (2):
>       net/hns3: fix VLAN strip log
>       net/hns3: fix residual MAC address entry
> 
> Chenglian Sun (1):
>       examples/l2fwd: fix [no-]mac-updating options
> 
> Chengwen Feng (3):
>       net/hns3: fix Arm SVE build with GCC 8.3
>       net/hns3: fix filter parsing comment
>       net/hns3: fix flow rule list in multi-process
> 
> Christian Ehrhardt (1):
>       vfio: add stdbool include
> 
> Ciara Power (1):
>       cryptodev: fix freeing after device release
> 
> Dana Vardi (5):
>       crypto/mvsam: fix AES-GCM session parameters
>       crypto/mvsam: fix session data reset
>       crypto/mvsam: fix options parsing
>       net/mvpp2: fix port speed overflow
>       net/mvpp2: fix configured state dependency
> 
> Dapeng Yu (8):
>       net/ice: fix default RSS key generation
>       net/i40e: fix use after free in FDIR release
>       net/i40e: fix multi-process shared data
>       net/ixgbe: fix flow entry access after freeing
>       net/softnic: fix connection memory leak
>       net/softnic: fix memory leak in arguments parsing
>       net/softnic: fix null dereference in arguments parsing
>       net/softnic: fix memory leak as profile is freed
> 
> David Christensen (1):
>       bus/pci: fix IOVA as VA support for PowerNV
> 
> David Hunt (1):
>       distributor: fix 128-bit write alignment
> 
> David Marchand (6):
>       malloc: fix size annotation for NUMA-aware realloc
>       bus/pci: fix leak for unbound devices
>       drivers/net: fix memzone allocations for DMA memory
>       ipc: stop mp control thread on cleanup
>       net/ice: fix memzone leak when firmware is missing
>       net/octeontx/base: fix debug build with clang
> 
> Dmitry Kozlyuk (6):
>       doc: add limitation for ConnectX-4 with L2 in mlx5 guide
>       doc: fix build on Windows with Meson 0.58
>       net/mlx5: fix Rx/Tx queue checks
>       net/mlx5: fix indirect action modify rollback
>       eal/windows: cleanup virt2phys handle
>       bus: clarify log for non-NUMA-aware devices
> 
> Feifei Wang (2):
>       crypto/qat: fix Arm build with special memcpy
>       net/mlx5: fix r/w lock usage in DMA unmap
> 
> Ferruh Yigit (2):
>       kni: fix crash on userspace VA for segmented packets
>       app/testpmd: fix help string for port reset
> 
> Gaoxiang Liu (1):
>       net/virtio: fix interrupt handle leak
> 
> Ghalem Boudour (1):
>       net/ena: enable multi-segment in Tx offload flags
> 
> Gregory Etelson (5):
>       net/mlx5: fix RSS pattern expansion
>       net/mlx5: fix pattern expansion in RSS flow rules
>       net/mlx5: fix representor interrupt handler
>       app/testpmd: fix Tx checksum calculation for tunnel
>       app/testpmd: fix IPv4 checksum
> 
> Guoyang Zhou (3):
>       net/hinic: increase protection of the VLAN
>       net/hinic/base: fix LRO
>       net/hinic: fix MTU consistency with firmware
> 
> Haiyue Wang (1):
>       net/iavf: fix RSS key access out of bound
> 
> Hemant Agrawal (1):
>       bus/dpaa: fix freeing in FMAN interface destructor
> 
> Henry Nadeau (1):
>       doc: fix spelling
> 
> Hongbo Zheng (4):
>       graph: fix memory leak in stats
>       graph: fix null dereference in stats
>       net/hns3: increase VF reset retry maximum
>       net/hns3: fix timing of clearing interrupt source
> 
> Huisong Li (7):
>       net/hns3: fix delay for waiting to stop Rx/Tx
>       net/hns3: fix fake queue rollback
>       net/hns3: fix maximum queues on configuration failure
>       app/testpmd: change port link speed without stopping all
>       net/hns3: fix Tx prepare after stop
>       sched: fix profile allocation failure handling
>       sched: rework configuration failure handling
> 
> Ivan Ilchenko (8):
>       bitmap: fix buffer overrun in bitmap init
>       net/virtio: fix Rx scatter offload
>       net/sfc: fix MAC stats lock in xstats query by ID
>       net/sfc: fix reading adapter state without locking
>       net/sfc: fix xstats query by ID according to ethdev
>       net/sfc: fix xstats query by unsorted list of IDs
>       net/sfc: fix MAC stats update for stopped device
>       net/virtio: report maximum MTU in device info
> 
> Ivan Malov (2):
>       net/sfc: check ID overflow in action port ID
>       net/sfc: fix outer match in MAE backend
> 
> Jan Viktorin (1):
>       ethdev: fix doc of flow action
> 
> Jasvinder Singh (1):
>       doc: remove old deprecation notice for sched
> 
> Jiayu Hu (1):
>       vhost: fix lock on device readiness notification
> 
> Jie Zhou (2):
>       app/testpmd: fix type of FEC mode parsing output
>       eal/windows: check callback parameter of alarm functions
> 
> Joyce Kong (2):
>       net/i40e: fix descriptor scan on Arm
>       doc: update atomic operation deprecation
> 
> Juraj Linkeš (1):
>       net/virtio: fix aarch32 build
> 
> Kai Ji (2):
>       app/crypto-perf: fix out-of-place mempool allocation
>       crypto/aesni_gcm: fix performance on some AVX512 CPUs
> 
> Kalesh AP (16):
>       net/bnxt: fix error messages in VNIC prepare
>       net/bnxt: set flow error when free filter not available
>       net/bnxt: remove unnecessary code
>       net/bnxt: fix error handling in VNIC prepare
>       net/bnxt: set flow error after tunnel redirection free
>       net/bnxt: use common function to free VNIC resource
>       net/bnxt: fix check for PTP support in FW
>       net/bnxt: improve probing log message
>       net/bnxt: remove unnecessary comment
>       net/bnxt: cleanup code
>       net/bnxt: fix typo in log message
>       net/bnxt: fix auto-negociation on Whitney+
>       net/bnxt: invoke device removal event on recovery failure
>       net/bnxt: clear cached statistics
>       net/bnxt: remove workaround for default VNIC
>       net/bnxt: fix null dereference in interrupt handler
> 
> Kevin Laatz (1):
>       raw/ioat: fix config script queue size calculation
> 
> Kevin Traynor (1):
>       doc: fix typo in SPDX tag
> 
> Lance Richardson (6):
>       net/bnxt: fix Rx burst size constraint
>       app/test: fix IPv6 header initialization
>       net/bnxt: fix ring and context memory allocation
>       net/bnxt: fix Tx descriptor status implementation
>       net/bnxt: fix scalar Tx completion handling
>       net/bnxt: fix missing barriers in completion handling
> 
> Liang Ma (1):
>       net/mlx5: export PMD-specific API file
> 
> Lior Margalit (5):
>       net/mlx5: fix IPIP multi-tunnel validation
>       net/mlx5: fix RSS expansion for GTP
>       net/mlx5: reject inner ethernet matching in GTP
>       net/mlx5: fix RSS flow rule with L4 mismatch
>       net/mlx5: fix default queue number in RSS flow rule
> 
> Martin Havlik (2):
>       net/bonding: fix error message on flow verify
>       net/bonding: check flow setting
> 
> Maxime Coquelin (4):
>       vhost: fix missing memory table NUMA realloc
>       vhost: fix missing guest pages table NUMA realloc
>       vhost: fix NUMA reallocation with multi-queue
>       vhost: fix crash on reconnect
> 
> Michael Baum (5):
>       common/mlx5: fix memory region leak
>       regex/mlx5: fix size of setup constants
>       vdpa/mlx5: fix overflow in queue attribute
>       net/mlx5: fix overflow in mempool argument
>       regex/mlx5: fix redundancy in device removal
> 
> Michael Shamis (1):
>       crypto/mvsam: fix capabilities
> 
> Michal Krawczyk (1):
>       net/ena: trigger reset on Tx prepare failure
> 
> Naga Harish K S V (1):
>       eventdev: fix event port setup in Tx adapter
> 
> Nathan Skrzypczak (1):
>       net/memif: fix abstract socket address length
> 
> Nick Connolly (1):
>       build: support drivers symlink on Windows
> 
> Nipun Gupta (1):
>       net/dpaa: fix headroom in VSP case
> 
> Nithin Dabilpuram (1):
>       net/octeontx2: use runtime LSO format indices
> 
> Olivier Matz (3):
>       test/mbuf: fix virtual address conversion
>       net/tap: fix Rx checksum flags on IP options packets
>       net/tap: fix Rx checksum flags on TCP packets
> 
> Owen Hilyard (4):
>       flow_classify: fix leaking rules on delete
>       rib: fix max depth IPv6 lookup
>       tests/cmdline: fix memory leaks
>       tests/eal: fix memory leak
> 
> Pavan Nikhilesh (1):
>       mempool/octeontx2: fix shift calculation
> 
> Qi Zhang (3):
>       net/iavf: fix handling of unsupported promiscuous
>       net/ice: fix data path selection in secondary process
>       net/ice: fix data path in secondary process
> 
> Rebecca Troy (3):
>       test/crypto: fix autotest function parameters
>       test/crypto: fix typo in AES case
>       test/crypto: fix typo in ESN case
> 
> Richael Zhuang (1):
>       test/power: fix CPU frequency check for intel_pstate
> 
> Ruifeng Wang (3):
>       net/sfc: fix aarch32 build
>       net/bnxt: fix aarch32 build
>       net/mlx5: remove redundant operations in NEON Rx
> 
> Sarosh Arif (1):
>       net/mlx5: fix typo in vectorized Rx comments
> 
> Satha Rao (1):
>       net/octeontx2: fix TM node statistics query
> 
> Satheesh Paul (2):
>       net/octeontx2: fix flow creation limit on CN98xx
>       net/octeontx2: fix default MCAM allocation size
> 
> Somnath Kotur (2):
>       net/bnxt: workaround spurious zero stats in Thor
>       net/bnxt: detect bad opaque in Rx completion
> 
> Steve Yang (1):
>       net/i40e: fix raw packet flow director
> 
> Suanming Mou (1):
>       net/mlx5: workaround drop action with old kernel
> 
> Tejasree Kondoj (1):
>       crypto/octeontx2: fix lookaside IPsec IV pointer
> 
> Thierry Herbelot (6):
>       net/bnxt: check access to possible null pointer
>       vhost/crypto: check request pointer before dereference
>       net/pfe: remove unnecessary null check
>       net/sfc: fix outer L4 checksum Rx
>       test/crypto: fix mbuf reset after null check
>       table: fix bucket empty check
> 
> Thomas Monjalon (1):
>       devtools: fix file listing in maintainers check
> 
> Tudor Cornea (1):
>       net/ice: fix overflow in maximum packet length config
> 
> Viacheslav Ovsiienko (11):
>       net/mlx5: fix switchdev mode recognition
>       common/mlx5: fix Netlink port name padding in probing
>       net/mlx5: fix multi-segment inline for the first segments
>       net/mlx5: fix TSO multi-segment inline length
>       common/mlx5: fix Netlink receive message buffer size
>       common/mlx5: use new port query API if available
>       common/mlx5: fix compatibility with OFED port query API
>       app/testpmd: fix offloads for newly attached port
>       net/mlx5: fix RoCE LAG bond device probing
>       net/mlx5: fix timestamp initialization on empty clock queue
>       net/mlx5: add Tx scheduling check on queue creation
> 
> Weifeng Li (1):
>       net/bnxt: fix nested lock during bonding
> 
> Xiao Wang (1):
>       vhost: check header for legacy dequeue offload
> 
> Xiaoyu Min (5):
>       net/mlx5: fix match MPLS over GRE with key
>       net/mlx5: remove unsupported flow item MPLS over IP
>       net/mlx5: fix MPLS RSS expansion
>       net/mlx5: fix missing RSS expandable items
>       net/mlx5: fix missing RSS expansion of IPv6 frag
> 
> Xiaoyun Li (1):
>       net/iavf: fix Tx threshold check
> 
> Xueming Li (1):
>       vdpa/mlx5: fix TSO offload without checksum
> 
> Yunjian Wang (1):
>       kni: fix mbuf allocation for kernel side use
> 
> Yuying Zhang (1):
>       app/testpmd: fix MAC address after port reset