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

[Kevin Traynor <ktraynor@redhat.com>]

On 16/09/2020 10:25, Kevin Traynor wrote:
> On 16/09/2020 08:27, Chen, BoX C wrote:
>> Hi Kevin,
>>
>> Update DPDK LTS 18.11.10 test result for Intel part. virtio test found a new bug.
>>
>>
>>
>> * Intel(R) Testing
>>
>>
>>
>> # Basic Intel(R) NIC testing
>>
>> * PF(i40e):Passed
>>
>>     - Known issue: Exception message when starting testpmd with the '--mp-alloc=xmem' parameter. Patchset is provided by  Burakov, Anatoly <anatoly.burakov@intel.com<mailto:anatoly.burakov@intel.com>> but can not be applied on 18.11.10. Anotoly will update the new patchlink later.
>>
> 
> ok, thanks for reporting. Anatoly do you think this is a critical issue
> to block 18.11.10 release on? (it doesn't seem to me)
> 
> Is the fix on dpdk main branch - any rough ETA for backport to 18.11?
> 
>>  * PF(ixgbe):Passed
>>
>> * VF(i40e):Passed
>>
>> * VF(ixgbe):Passed
>>
>> * Build or compile: Passed
>>
>> * Intel NIC single core/NIC performance: Passed
>>
>> #Basic cryptodev and virtio testing
>>
>> * vhost/virtio basic loopback, PVP and performance test: Passed.
>>
>> New bug: Command for testpmd 'clear port xstats all' not worked. The bad commit id is: 3e64dd9f527d8191ec54286de664f17e12342fa8, already report this issue to Intel team and they are working on it.
>>
>> commit 3e64dd9f527d8191ec54286de664f17e12342fa8
>>    Author: Wei Hu (Xavier) <xavier.huwei@huawei.com>
>>   Date: Sat Jun 6 11:46:37 2020 +0800
>>
>> * cryptodev Function: Passed.
>>
>>  * cryptodev Performance: Passed.
>>
> 
> This issue looks to be like because
> rte_eth_xstats_reset() was dropped by mistake during rebase of
> nic_xstats_clear(). I will send a patch to fix it.
> 

Patch sent here:
http://inbox.dpdk.org/stable/20200916095844.130917-1-ktraynor@redhat.com

I will apply by EOD, in case you want to re-run this test tomorrow. As
it's isolated I won't make an RC2 for this issue.

thanks,
Kevin.

> Thanks Chen Bo.
> 
> Kevin.
> 
>>>
>> Thanks.
>>
>>
>>
>> Regards,
>>
>> Chen Bo
>>
>>
>>
>>> -----Original Message-----
>>
>>> From: dev <dev-bounces@dpdk.org> On Behalf Of Kevin Traynor
>>
>>> Sent: September 7, 2020 18:12
>>
>>> To: stable@dpdk.org
>>
>>> Cc: dev@dpdk.org; Abhishek Marathe <Abhishek.Marathe@microsoft.com>;
>>
>>> Akhil Goyal <akhil.goyal@nxp.com>; Ali Alnubani <alialnu@mellanox.com>;
>>
>>> Walker, Benjamin <benjamin.walker@intel.com>; David Christensen
>>
>>> <drc@linux.vnet.ibm.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@mellanox.com>; Thomas Monjalon
>>
>>> <thomas@monjalon.net>; Peng, Yuan <yuan.peng@intel.com>; Chen,
>>
>>> Zhaoyan <zhaoyan.chen@intel.com>
>>
>>> Subject: [dpdk-dev] 18.11.10 (LTS) patches review and test
>>
>>>
>>
>>> Hi all,
>>
>>>
>>
>>> Here is a list of patches targeted for LTS release 18.11.10.
>>
>>>
>>
>>> The planned date for the final release is 21 September.
>>
>>>
>>
>>> 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=v18.11.10-rc1
>>
>>>
>>
>>> These patches are located at branch 18.11 of dpdk-stable repo:
>>
>>>     https://dpdk.org/browse/dpdk-stable/
>>
>>>
>>
>>> Thanks.
>>
>>>
>>
>>> Kevin.
>>
>>>
>>
>>> ---
>>
>>> Adam Dybkowski (2):
>>
>>>       test/crypto: fix asymmetric session mempool creation
>>
>>>       common/qat: fix uninitialized variable
>>
>>>
>>
>>> Akhil Goyal (1):
>>
>>>       crypto/dpaax_sec: fix inline query for descriptors
>>
>>>
>>
>>> Alan Dewar (1):
>>
>>>       sched: fix port time rounding
>>
>>>
>>
>>> Alexander Kozyrev (1):
>>
>>>       net/mlx5: fix vectorized Rx burst termination
>>
>>>
>>
>>> Ankur Dwivedi (1):
>>
>>>       common/cpt: fix encryption offset
>>
>>>
>>
>>> Apeksha Gupta (2):
>>
>>>       app/eventdev: fix capability check in pipeline ATQ test
>>
>>>       event/dpaa2: add all-types queue capability flag
>>
>>>
>>
>>> Archana Muniganti (1):
>>
>>>       examples/fips_validation: fix count overwrite for TDES
>>
>>>
>>
>>> Ayuj Verma (1):
>>
>>>       examples/fips_validation: fix parsing of TDES vectors
>>
>>>
>>
>>> Bruce Richardson (4):
>>
>>>       mem: fix 32-bit init config with meson
>>
>>>       rawdev: allow getting info for unknown device
>>
>>>       rawdev: fill NUMA socket ID in info
>>
>>>       rawdev: export dump function in map file
>>
>>>
>>
>>> Chas Williams (1):
>>
>>>       net/netvsc: fix crash during Tx
>>
>>>
>>
>>> Chenbo Xia (1):
>>
>>>       net/virtio-user: check tap system call setting
>>
>>>
>>
>>> Chenmin Sun (1):
>>
>>>       net/i40e: fix flow director Rx writeback packet
>>
>>>
>>
>>> David Marchand (4):
>>
>>>       net/mvpp2: fix non-EAL thread support
>>
>>>       devtools: fix path in forbidden token check
>>
>>>       lib: remind experimental status in headers
>>
>>>       net/bonding: fix socket ID check
>>
>>>
>>
>>> Devendra Singh Rawat (1):
>>
>>>       net/qede: fix multicast drop in promiscuous mode
>>
>>>
>>
>>> Dongyang Pan (1):
>>
>>>       net/bonding: delete redundant code
>>
>>>
>>
>>> Ferruh Yigit (4):
>>
>>>       net/iavf: fix uninitialized variable
>>
>>>       net/ixgbe/base: remove dead code
>>
>>>       ethdev: fix log type for some error messages
>>
>>>       drivers/net: fix exposing internal headers
>>
>>>
>>
>>> Gaurav Singh (1):
>>
>>>       net/i40e: remove duplicate tunnel type check
>>
>>>
>>
>>> Georgiy Levashov (1):
>>
>>>       examples: add flush after stats printing
>>
>>>
>>
>>> Guinan Sun (5):
>>
>>>       net/ixgbe/base: fix host interface shadow RAM read
>>
>>>       net/ixgbe/base: fix x550em 10G NIC link status
>>
>>>       net/ixgbe/base: fix infinite recursion on PCIe link down
>>
>>>       net/ixgbe: fix MAC control frame forward
>>
>>>       net/ixgbe: fix flow control status
>>
>>>
>>
>>> Haiyue Wang (1):
>>
>>>       eal: fix uuid header dependencies
>>
>>>
>>
>>> Harman Kalra (1):
>>
>>>       eal: fix parentheses in alignment macros
>>
>>>
>>
>>> Harry van Haaren (1):
>>
>>>       examples/eventdev: fix 32-bit coremask
>>
>>>
>>
>>> Heinrich Kuhn (1):
>>
>>>       net/nfp: fix RSS hash configuration reporting
>>
>>>
>>
>>> Hemant Agrawal (1):
>>
>>>       rawdev: remove remaining experimental tags
>>
>>>
>>
>>> Hongzhi Guo (3):
>>
>>>       net: fix IPv4 checksum
>>
>>>       net: fix unneeded replacement of TCP checksum 0
>>
>>>       net: fix checksum on big endian CPUs
>>
>>>
>>
>>> Honnappa Nagarahalli (4):
>>
>>>       eal/armv8: fix timer frequency calibration with PMU
>>
>>>       app/testpmd: use clock time in throughput calculation
>>
>>>       app/testpmd: fix burst percentage calculation
>>
>>>       doc: fix some typos in Linux guide
>>
>>>
>>
>>> Ian Dolzhansky (1):
>>
>>>       net/failsafe: fix RSS RETA size info
>>
>>>
>>
>>> Jecky Pei (1):
>>
>>>       net/kni: set packet input port in Rx
>>
>>>
>>
>>> Jeff Guo (1):
>>
>>>       net/e1000: fix crash on Tx done clean up
>>
>>>
>>
>>> Kalesh AP (1):
>>
>>>       net/bnxt: remove unused enum declaration
>>
>>>
>>
>>> Kevin Traynor (1):
>>
>>>       kni: fix ethtool build error on kernel 5.7
>>
>>>
>>
>>> Konstantin Ananyev (1):
>>
>>>       bpf: fix add/sub min/max estimations
>>
>>>
>>
>>> Levend Sayar (1):
>>
>>>       service: fix C++ linkage
>>
>>>
>>
>>> Long Li (3):
>>
>>>       bus/vmbus: fix ring buffer mapping
>>
>>>       net/netvsc: fix underflow when Rx external mbuf
>>
>>>       net/netvsc: fix chimney index
>>
>>>
>>
>>> Lukasz Wojciechowski (1):
>>
>>>       service: fix core mapping reset
>>
>>>
>>
>>> Mao Jiang (2):
>>
>>>       net/i40e: fix binding interrupt without MSI-X vector
>>
>>>       net/i40e: fix flow director MSI-X resource allocation
>>
>>>
>>
>>> Matan Azrad (1):
>>
>>>       vhost: fix features definition location
>>
>>>
>>
>>> Maxime Coquelin (1):
>>
>>>       vhost: fix virtio ready flag check
>>
>>>
>>
>>> Michael Baum (4):
>>
>>>       net/mlx5: fix unreachable MPLS error path
>>
>>>       net/mlx5: fix iterator type in Rx queue management
>>
>>>       net/mlx5: fix crash in NVGRE item translation
>>
>>>       net/mlx4: optimize stack memory size in probe
>>
>>>
>>
>>> Muhammad Bilal (1):
>>
>>>       bus/vdev: fix a typo in doxygen comment
>>
>>>
>>
>>> Nipun Gupta (2):
>>
>>>       bus/fslmc: fix getting FD error
>>
>>>       net/dpaa: fix FD offset data type
>>
>>>
>>
>>> Pablo de Lara (1):
>>
>>>       drivers/crypto: add missing OOP feature flag
>>
>>>
>>
>>> Patrick Fu (1):
>>
>>>       vhost: fix double-free with zero-copy
>>
>>>
>>
>>> Pavel Ivashchenko (1):
>>
>>>       net/i40e: fix getting EEPROM information
>>
>>>
>>
>>> Rahul Lakkireddy (3):
>>
>>>       net/cxgbe: fix CLIP leak in filter error path
>>
>>>       net/cxgbe: fix double MPS alloc by flow validate and create
>>
>>>       net/cxgbe: fix L2T leak in filter error and free path
>>
>>>
>>
>>> Raslan Darawsheh (2):
>>
>>>       net: fix pedantic build
>>
>>>       net/mlx5: fix flow items size calculation
>>
>>>
>>
>>> Ruifeng Wang (6):
>>
>>>       net/i40e: enable NEON Rx/Tx in meson
>>
>>>       eal/arm: add vcopyq intrinsic for aarch32
>>
>>>       net/ixgbe: fix include of vector header file
>>
>>>       test/cycles: restore default delay callback
>>
>>>       crypto/armv8: remove debug option
>>
>>>       crypto/armv8: use dedicated log type
>>
>>>
>>
>>> Sarosh Arif (3):
>>
>>>       doc: fix typo in bbdev test guide
>>
>>>       examples/packet_ordering: use proper exit method
>>
>>>       doc: fix ethtool app path
>>
>>>
>>
>>> Stephen Hemminger (4):
>>
>>>       net/netvsc: do not spin forever waiting for reply
>>
>>>       cfgfile: fix stack buffer underflow
>>
>>>       kni: fix reference to master/slave process
>>
>>>       doc: fix reference to master process
>>
>>>
>>
>>> Tal Shnaiderman (1):
>>
>>>       pci: fix address domain format size
>>
>>>
>>
>>> Timothy Redaelli (1):
>>
>>>       test: fix rpath for drivers with meson
>>
>>>
>>
>>> Wei Hu (Xavier) (5):
>>
>>>       ethdev: fix data room size verification in Rx queue setup
>>
>>>       net/bonding: fix error code on device creation
>>
>>>       net/bonding: fix MAC address when switching active port
>>
>>>       net/bonding: fix MAC address when one port resets
>>
>>>       app/testpmd: fix stats error message
>>
>>>
>>
>>> Wei Zhao (1):
>>
>>>       net/ixgbe: report 10Mbps link speed for x553
>>
>>>
>>
>>> Xiao Wang (1):
>>
>>>       net/virtio-user: fix status management
>>
>>>
>>
>>> Yicai Lu (1):
>>
>>>       net/bonding: fix LACP negotiation
>>
>>>
>>
>>> Yunjian Wang (8):
>>
>>>       app/testpmd: fix memory leak on error path
>>
>>>       vfio: remove unused variable
>>
>>>       net/af_packet: fix check of file descriptors
>>
>>>       net/af_packet: fix memory leak on init failure
>>
>>>       net/af_packet: fix munmap on init failure
>>
>>>       hash: fix out-of-memory handling in hash creation
>>
>>>       event/dpaa: remove dead code
>>
>>>       net/qede: remove dead code
>>
>>>
>>
>>> Zhiguang He (1):
>>
>>>       net/bonding: fix dead loop on RSS RETA update
>>
>>>
>>
>>> Zhihong Peng (2):
>>
>>>       net/i40e: report VLAN filter capability
>>
>>>       net/e1000: report VLAN extend capability
>>
>>
>>
>