* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-13 12:47 [dpdk-dev] 17.11.10 (LTS) patches review and test Luca Boccassi
@ 2020-01-17 2:52 ` Pei Zhang
2020-01-17 10:44 ` Luca Boccassi
2020-01-22 5:27 ` Yu, PingX
` (2 subsequent siblings)
3 siblings, 1 reply; 18+ messages in thread
From: Pei Zhang @ 2020-01-17 2:52 UTC (permalink / raw)
To: Luca Boccassi
Cc: dpdk stable, dev, Akhil Goyal, Ali Alnubani, benjamin walker,
David Christensen, Hemant Agrawal, Ian Stokes, Jerin Jacob,
John McNamara, Kevin Traynor, pingx yu, qian q xu,
Raslan Darawsheh, Thomas Monjalon, yuan peng, Zhaoyan Chen
Hi Luca,
Testing with dpdk 17.11.10-rc1 from Red Hat looks good.
We cover below 13 scenarios and and all get PASS:
(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)PVP 1Q live migration testing: PASS
(10)PVP 1Q cross numa node live migration testing: PASS
(11)Guest with ovs+dpdk+vhost-user 1Q live migration testing: PASS
(12)Guest with ovs+dpdk+vhost-user 1Q live migration testing (2M): PASS
(13)Guest with ovs+dpdk+vhost-user 2Q live migration testing: PASS
Versions:
kernel 4.18
qemu 4.2
dpdk: http://dpdk.org/git/dpdk-stable remotes/origin/17.11
# git log -1
commit 3f8426c2ee83d0ee6f761bac72c85e1b96c20806
Author: Luca Boccassi <luca.boccassi@microsoft.com>
Date: Mon Jan 13 12:32:53 2020 +0000
version: 17.11.10-rc1
Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>
Best regards,
Pei
----- Original Message -----
From: "Luca Boccassi" <bluca@debian.org>
To: "dpdk stable" <stable@dpdk.org>
Cc: dev@dpdk.org, "Akhil Goyal" <akhil.goyal@nxp.com>, "Ali Alnubani" <alialnu@mellanox.com>, "benjamin walker" <benjamin.walker@intel.com>, "David Christensen" <drc@linux.vnet.ibm.com>, "Hemant Agrawal" <hemant.agrawal@nxp.com>, "Ian Stokes" <ian.stokes@intel.com>, "Jerin Jacob" <jerinj@marvell.com>, "John McNamara" <john.mcnamara@intel.com>, "Kevin Traynor" <ktraynor@redhat.com>, "Pei Zhang" <pezhang@redhat.com>, "pingx yu" <pingx.yu@intel.com>, "qian q xu" <qian.q.xu@intel.com>, "Raslan Darawsheh" <rasland@mellanox.com>, "Thomas Monjalon" <thomas@monjalon.net>, "yuan peng" <yuan.peng@intel.com>, "Zhaoyan Chen" <zhaoyan.chen@intel.com>
Sent: Monday, January 13, 2020 8:47:18 PM
Subject: 17.11.10 (LTS) patches review and test
Hi all,
Here is a list of patches targeted for LTS release 17.11.10.
This will be the last 17.11 release.
The planned date for the final release is the 30th of January.
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=v17.11.10-rc1
These patches are located at branch 17.11 of dpdk-stable repo:
https://dpdk.org/browse/dpdk-stable/
Thanks.
Luca Boccassi
---
Aaron Conole (1):
test/interrupt: account for race with callback
Abhishek Sachan (1):
net/af_packet: fix stale sockets
Adrian Moreno (4):
vhost: translate incoming log address to GPA
vhost: prevent zero copy mode if IOMMU is on
vhost: fix vring memory partially mapped
vhost: convert buffer addresses to GPA for logging
Ajit Khaparde (3):
net/bnxt: fix async link handling and update
net/bnxt: get default HWRM command timeout from FW
net/bnxt: fix crash in xstats get
Akhil Goyal (1):
crypto/dpaa2_sec: fix length retrieved from hardware
Andrew Rybchenko (3):
ethdev: remove redundant device info cleanup before get
net/virtio: fix descriptor addressed in Tx
net/virtio: fix Tx checksum offloads
Andrzej Ostruszka (3):
doc: fix description of versioning macros
net/dpaa2: fix possible use of uninitialized vars
doc: fix tap guide
Archana Muniganti (1):
app/crypto-perf: fix input of AEAD decrypt
Arek Kusztal (1):
crypto/qat: fix null auth issues when using vfio_pci
Bruce Richardson (1):
examples/vm_power: fix type of cmdline token in cli
Christian Ehrhardt (2):
kni: fix build with kernel 5.3
net/mlx4: fix build on ppc64
Ciara Power (3):
app/testpmd: fix help for loop topology option
ethdev: fix include of ethernet header file
app/proc_info: fix string copying to use strlcpy
David Hunt (1):
examples/vm_power: fix build without i40e
David Marchand (7):
net/bonding: fix LACP fast queue Rx handler
net/bonding: fix unicast packets filtering
ethdev: fix endian annotation for SPI item
net/qede: limit Rx ring index read for debug
net/bonding: use non deprecated PCI API
doc/guides: clean repeated words
eal: remove dead code on NUMA node detection
Dilshod Urazov (2):
net/virtio: reject deferred Rx start
net/virtio: reject deferred Tx start
Eelco Chaudron (1):
net/i40e: downgrade error log
Fan Zhang (1):
doc: fix AESNI-GCM limitations in crypto guide
Ferruh Yigit (3):
net/igb: fix global variable multiple definitions
test: fix global variable multiple definitions
mk: remove library search path from binary
Fiona Trahe (1):
crypto/qat: fix digest length in XCBC capability
Flavia Musatescu (1):
net/af_packet: improve Tx statistics accuracy
Flavio Leitner (1):
vhost: fix IPv4 checksum
Gage Eads (1):
event/sw: fix xstats reset value
Gavin Hu (5):
ring: enforce reading tail before slots
net/i40e: remove memory barrier from NEON Rx
net/i40e: remove compiler barrier from NEON Rx
net/bnxt: remove duplicate barrier
net/bnxt: enforce IO barrier for doorbell command
Harry van Haaren (1):
test/service: fix wait for service core
Hemant Agrawal (1):
event/dpaa2: fix default queue configuration
Honnappa Nagarahalli (1):
test/lpm: fix measured cycles for delete
Hui Zhao (2):
net/bonding: fix slave id types
net/bonding: fix OOB access in other aggregator modes
Igor Romanov (1):
net/bonding: fix link speed update in broadcast mode
Ivan Malov (1):
net/sfc: fix adapter lock usage on rule creation
Joyce Kong (2):
net/i40e: fix address of first segment
net/ixgbe: fix address of first segment
Julien Meunier (2):
cryptodev: fix initialization on multi-process
cryptodev: fix checks related to device id
Junyu Jiang (2):
net/ixgbe: fix queue interrupt for X552/557
net/bonding: fix port ID check
Kalesh AP (10):
net/bnxt: fix setting default MAC address
net/bnxt: fix multicast filter programming
net/bnxt: return error if setting link up fails
net/bnxt: remove redundant header file inclusion
net/bnxt: fix coding style
net/bnxt: remove unnecessary variable assignment
net/bnxt: fix dereference before null check
net/bnxt: cleanup comments
net/bnxt: move macro definitions to header file
net/bnxt: fix memory leak
Kevin Traynor (3):
net/bnxt: remove commented out code
lib: fix log typos
lib: fix doxygen typos
Krzysztof Kanas (3):
test/bonding: fix LSC related cases
test: optimise fd closing in forks
net/bonding: fix selection logic
Lance Richardson (1):
net/bnxt: fix mbuf free when clearing Tx queue
Lu Qiuwen (1):
net/fm10k: fix stats crash in multi-process
Lukasz Bartosik (1):
examples/ipsec-secgw: fix default configuration
Lunyuan Cui (2):
net/ixgbe: fix link status
net/e1000: fix link status
Marcin Hajkowski (1):
power: fix socket indicator value
Marcin Smoczynski (2):
net/tap: fix blocked Rx packets
examples/ipsec-secgw: fix GCM IV length
Marvin Liu (3):
net/virtio: fix mbuf data and packet length mismatch
vhost: fix virtqueue not accessible
net/virtio-user: fix setting filters
Mattias Rönnblom (1):
power: handle frequency increase with turbo disabled
Maxime Coquelin (1):
vhost: un-inline dirty pages logging functions
Michal Krawczyk (1):
vfio: fix truncated BAR offset for 32-bit
Nipun Gupta (2):
net/dpaa2: set port in mbuf
net/dpaa2: add retry and timeout in packet enqueue API
Noa Ezra (1):
net/vhost: fix redundant queue state event
Patrick Fu (1):
net/virtio: init MTU in case no control channel
Pavan Nikhilesh (2):
app/testpmd: use better randomness for Tx split
event/octeontx: fix partial Rx packet handling
Peng Huang (1):
app/testpmd: fix Tx checksum when TSO enabled
Phil Yang (1):
net/mlx5: fix Rx CQ doorbell synchronization on aarch64
Radu Bulie (1):
mempool/dpaa2: report error on endless loop in mbuf release
Radu Nicolau (1):
security: fix doxygen fields
Rahul Gupta (1):
net/bnxt: fix Rx queue count
Rahul Lakkireddy (1):
net/cxgbe: fix prefetch for non-coalesced Tx packets
Raslan Darawsheh (1):
net/mlx: fix debug build with icc
Robin Jarry (1):
usertools: fix pmdinfo with python 3 and pyelftools>=0.24
Ruifeng Wang (3):
net/ixgbe: remove memory barrier from NEON Rx
lib/distributor: fix deadlock on aarch64
net/ixgbe: support packet type with NEON
Seth Howell (1):
bus/pci: remove useless link dependency on ethdev
Shahed Shaikh (2):
net/qede: fix setting MTU
net/qede: fix setting VLAN strip mode
Shougang Wang (1):
app/testpmd: fix crash on port reset
Shweta Choudaha (1):
net/igb: fix PHY status if PHY reset is not blocked
Somnath Kotur (1):
net/bnxt: expose some missing counters in port stats
Stephen Hemminger (6):
bus/pci: fix Intel IOMMU sysfs access check
net/bnxt: fix crash in secondary process
service: use log for error messages
test/mbuf: fix forged mbuf in clone test
app/testpmd: block xstats for hidden ports
examples/multi_process: fix client crash with sparse ports
Thibaut Collet (1):
net/virtio: get all pending Rx packets in vectorized paths
Thierry Herbelot (3):
drivers/crypto: remove some invalid comments
doc: fix a common typo in NIC guides
crypto/openssl: use local copy for session contexts
Thomas Monjalon (3):
doc: robustify PDF build
net/mlx: fix build with make and recent gcc
ethdev: limit maximum number of queues
Tiwei Bie (5):
doc: fix format in virtio guide
vhost: fix slave request fd leak
vhost: forbid reallocation when running
vhost: fix vring address handling during live migration
vhost: protect vring access done by application
Vakul Garg (1):
examples/ipsec-secgw: fix SHA256-HMAC digest length
Venkat Duvvuru (1):
net/bnxt: fix log message level
Wangyu (Eric) (1):
bus/pci: align next mapping address on page boundary
Wei Zhao (2):
net/ixgbe: enable new PF host mbox version
net/ixgbe: fix VF RSS offloads configuration
Xiao Wang (2):
doc: fix typo in l2fwd-crypto guide
net/fm10k: fix mbuf free in vector Rx
Xiao Zhang (3):
net/ixgbe: fix X553 speed capability
examples/l3fwd-power: fix Rx interrupt disabling
net/ixgbe: fix link status
Xiaolong Ye (1):
ethdev: fix typos for ENOTSUP
Xueming Li (2):
malloc: fix realloc copy size
malloc: fix realloc padded element size
Yong Wang (1):
net/ixgbe: remove redundant assignment
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-17 2:52 ` Pei Zhang
@ 2020-01-17 10:44 ` Luca Boccassi
0 siblings, 0 replies; 18+ messages in thread
From: Luca Boccassi @ 2020-01-17 10:44 UTC (permalink / raw)
To: Pei Zhang
Cc: dpdk stable, dev, Akhil Goyal, Ali Alnubani, benjamin walker,
David Christensen, Hemant Agrawal, Ian Stokes, Jerin Jacob,
John McNamara, Kevin Traynor, pingx yu, qian q xu,
Raslan Darawsheh, Thomas Monjalon, yuan peng, Zhaoyan Chen
On Thu, 2020-01-16 at 21:52 -0500, Pei Zhang wrote:
> Hi Luca,
>
> Testing with dpdk 17.11.10-rc1 from Red Hat looks good.
>
> We cover below 13 scenarios and and all get PASS:
>
> (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)PVP 1Q live migration testing: PASS
> (10)PVP 1Q cross numa node live migration testing: PASS
> (11)Guest with ovs+dpdk+vhost-user 1Q live migration testing: PASS
> (12)Guest with ovs+dpdk+vhost-user 1Q live migration testing (2M):
> PASS
> (13)Guest with ovs+dpdk+vhost-user 2Q live migration testing: PASS
>
>
> Versions:
> kernel 4.18
> qemu 4.2
> dpdk:
> http://dpdk.org/git/dpdk-stable
> remotes/origin/17.11
> # git log -1
> commit 3f8426c2ee83d0ee6f761bac72c85e1b96c20806
> Author: Luca Boccassi <
> luca.boccassi@microsoft.com
> >
> Date: Mon Jan 13 12:32:53 2020 +0000
> version: 17.11.10-rc1
> Signed-off-by: Luca Boccassi <
> luca.boccassi@microsoft.com
> >
>
>
> Best regards,
>
> Pei
Great, thank you!
> ----- Original Message -----
> From: "Luca Boccassi" <
> bluca@debian.org
> >
> To: "dpdk stable" <
> stable@dpdk.org
> >
> Cc:
> dev@dpdk.org
> , "Akhil Goyal" <
> akhil.goyal@nxp.com
> >, "Ali Alnubani" <
> alialnu@mellanox.com
> >, "benjamin walker" <
> benjamin.walker@intel.com
> >, "David Christensen" <
> drc@linux.vnet.ibm.com
> >, "Hemant Agrawal" <
> hemant.agrawal@nxp.com
> >, "Ian Stokes" <
> ian.stokes@intel.com
> >, "Jerin Jacob" <
> jerinj@marvell.com
> >, "John McNamara" <
> john.mcnamara@intel.com
> >, "Kevin Traynor" <
> ktraynor@redhat.com
> >, "Pei Zhang" <
> pezhang@redhat.com
> >, "pingx yu" <
> pingx.yu@intel.com
> >, "qian q xu" <
> qian.q.xu@intel.com
> >, "Raslan Darawsheh" <
> rasland@mellanox.com
> >, "Thomas Monjalon" <
> thomas@monjalon.net
> >, "yuan peng" <
> yuan.peng@intel.com
> >, "Zhaoyan Chen" <
> zhaoyan.chen@intel.com
> >
> Sent: Monday, January 13, 2020 8:47:18 PM
> Subject: 17.11.10 (LTS) patches review and test
>
> Hi all,
>
> Here is a list of patches targeted for LTS release 17.11.10.
> This will be the last 17.11 release.
>
> The planned date for the final release is the 30th of January.
>
> 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=v17.11.10-rc1
>
>
> These patches are located at branch 17.11 of dpdk-stable repo:
>
> https://dpdk.org/browse/dpdk-stable/
>
>
> Thanks.
>
> Luca Boccassi
>
> ---
> Aaron Conole (1):
> test/interrupt: account for race with callback
>
> Abhishek Sachan (1):
> net/af_packet: fix stale sockets
>
> Adrian Moreno (4):
> vhost: translate incoming log address to GPA
> vhost: prevent zero copy mode if IOMMU is on
> vhost: fix vring memory partially mapped
> vhost: convert buffer addresses to GPA for logging
>
> Ajit Khaparde (3):
> net/bnxt: fix async link handling and update
> net/bnxt: get default HWRM command timeout from FW
> net/bnxt: fix crash in xstats get
>
> Akhil Goyal (1):
> crypto/dpaa2_sec: fix length retrieved from hardware
>
> Andrew Rybchenko (3):
> ethdev: remove redundant device info cleanup before get
> net/virtio: fix descriptor addressed in Tx
> net/virtio: fix Tx checksum offloads
>
> Andrzej Ostruszka (3):
> doc: fix description of versioning macros
> net/dpaa2: fix possible use of uninitialized vars
> doc: fix tap guide
>
> Archana Muniganti (1):
> app/crypto-perf: fix input of AEAD decrypt
>
> Arek Kusztal (1):
> crypto/qat: fix null auth issues when using vfio_pci
>
> Bruce Richardson (1):
> examples/vm_power: fix type of cmdline token in cli
>
> Christian Ehrhardt (2):
> kni: fix build with kernel 5.3
> net/mlx4: fix build on ppc64
>
> Ciara Power (3):
> app/testpmd: fix help for loop topology option
> ethdev: fix include of ethernet header file
> app/proc_info: fix string copying to use strlcpy
>
> David Hunt (1):
> examples/vm_power: fix build without i40e
>
> David Marchand (7):
> net/bonding: fix LACP fast queue Rx handler
> net/bonding: fix unicast packets filtering
> ethdev: fix endian annotation for SPI item
> net/qede: limit Rx ring index read for debug
> net/bonding: use non deprecated PCI API
> doc/guides: clean repeated words
> eal: remove dead code on NUMA node detection
>
> Dilshod Urazov (2):
> net/virtio: reject deferred Rx start
> net/virtio: reject deferred Tx start
>
> Eelco Chaudron (1):
> net/i40e: downgrade error log
>
> Fan Zhang (1):
> doc: fix AESNI-GCM limitations in crypto guide
>
> Ferruh Yigit (3):
> net/igb: fix global variable multiple definitions
> test: fix global variable multiple definitions
> mk: remove library search path from binary
>
> Fiona Trahe (1):
> crypto/qat: fix digest length in XCBC capability
>
> Flavia Musatescu (1):
> net/af_packet: improve Tx statistics accuracy
>
> Flavio Leitner (1):
> vhost: fix IPv4 checksum
>
> Gage Eads (1):
> event/sw: fix xstats reset value
>
> Gavin Hu (5):
> ring: enforce reading tail before slots
> net/i40e: remove memory barrier from NEON Rx
> net/i40e: remove compiler barrier from NEON Rx
> net/bnxt: remove duplicate barrier
> net/bnxt: enforce IO barrier for doorbell command
>
> Harry van Haaren (1):
> test/service: fix wait for service core
>
> Hemant Agrawal (1):
> event/dpaa2: fix default queue configuration
>
> Honnappa Nagarahalli (1):
> test/lpm: fix measured cycles for delete
>
> Hui Zhao (2):
> net/bonding: fix slave id types
> net/bonding: fix OOB access in other aggregator modes
>
> Igor Romanov (1):
> net/bonding: fix link speed update in broadcast mode
>
> Ivan Malov (1):
> net/sfc: fix adapter lock usage on rule creation
>
> Joyce Kong (2):
> net/i40e: fix address of first segment
> net/ixgbe: fix address of first segment
>
> Julien Meunier (2):
> cryptodev: fix initialization on multi-process
> cryptodev: fix checks related to device id
>
> Junyu Jiang (2):
> net/ixgbe: fix queue interrupt for X552/557
> net/bonding: fix port ID check
>
> Kalesh AP (10):
> net/bnxt: fix setting default MAC address
> net/bnxt: fix multicast filter programming
> net/bnxt: return error if setting link up fails
> net/bnxt: remove redundant header file inclusion
> net/bnxt: fix coding style
> net/bnxt: remove unnecessary variable assignment
> net/bnxt: fix dereference before null check
> net/bnxt: cleanup comments
> net/bnxt: move macro definitions to header file
> net/bnxt: fix memory leak
>
> Kevin Traynor (3):
> net/bnxt: remove commented out code
> lib: fix log typos
> lib: fix doxygen typos
>
> Krzysztof Kanas (3):
> test/bonding: fix LSC related cases
> test: optimise fd closing in forks
> net/bonding: fix selection logic
>
> Lance Richardson (1):
> net/bnxt: fix mbuf free when clearing Tx queue
>
> Lu Qiuwen (1):
> net/fm10k: fix stats crash in multi-process
>
> Lukasz Bartosik (1):
> examples/ipsec-secgw: fix default configuration
>
> Lunyuan Cui (2):
> net/ixgbe: fix link status
> net/e1000: fix link status
>
> Marcin Hajkowski (1):
> power: fix socket indicator value
>
> Marcin Smoczynski (2):
> net/tap: fix blocked Rx packets
> examples/ipsec-secgw: fix GCM IV length
>
> Marvin Liu (3):
> net/virtio: fix mbuf data and packet length mismatch
> vhost: fix virtqueue not accessible
> net/virtio-user: fix setting filters
>
> Mattias Rönnblom (1):
> power: handle frequency increase with turbo disabled
>
> Maxime Coquelin (1):
> vhost: un-inline dirty pages logging functions
>
> Michal Krawczyk (1):
> vfio: fix truncated BAR offset for 32-bit
>
> Nipun Gupta (2):
> net/dpaa2: set port in mbuf
> net/dpaa2: add retry and timeout in packet enqueue API
>
> Noa Ezra (1):
> net/vhost: fix redundant queue state event
>
> Patrick Fu (1):
> net/virtio: init MTU in case no control channel
>
> Pavan Nikhilesh (2):
> app/testpmd: use better randomness for Tx split
> event/octeontx: fix partial Rx packet handling
>
> Peng Huang (1):
> app/testpmd: fix Tx checksum when TSO enabled
>
> Phil Yang (1):
> net/mlx5: fix Rx CQ doorbell synchronization on aarch64
>
> Radu Bulie (1):
> mempool/dpaa2: report error on endless loop in mbuf release
>
> Radu Nicolau (1):
> security: fix doxygen fields
>
> Rahul Gupta (1):
> net/bnxt: fix Rx queue count
>
> Rahul Lakkireddy (1):
> net/cxgbe: fix prefetch for non-coalesced Tx packets
>
> Raslan Darawsheh (1):
> net/mlx: fix debug build with icc
>
> Robin Jarry (1):
> usertools: fix pmdinfo with python 3 and pyelftools>=0.24
>
> Ruifeng Wang (3):
> net/ixgbe: remove memory barrier from NEON Rx
> lib/distributor: fix deadlock on aarch64
> net/ixgbe: support packet type with NEON
>
> Seth Howell (1):
> bus/pci: remove useless link dependency on ethdev
>
> Shahed Shaikh (2):
> net/qede: fix setting MTU
> net/qede: fix setting VLAN strip mode
>
> Shougang Wang (1):
> app/testpmd: fix crash on port reset
>
> Shweta Choudaha (1):
> net/igb: fix PHY status if PHY reset is not blocked
>
> Somnath Kotur (1):
> net/bnxt: expose some missing counters in port stats
>
> Stephen Hemminger (6):
> bus/pci: fix Intel IOMMU sysfs access check
> net/bnxt: fix crash in secondary process
> service: use log for error messages
> test/mbuf: fix forged mbuf in clone test
> app/testpmd: block xstats for hidden ports
> examples/multi_process: fix client crash with sparse ports
>
> Thibaut Collet (1):
> net/virtio: get all pending Rx packets in vectorized paths
>
> Thierry Herbelot (3):
> drivers/crypto: remove some invalid comments
> doc: fix a common typo in NIC guides
> crypto/openssl: use local copy for session contexts
>
> Thomas Monjalon (3):
> doc: robustify PDF build
> net/mlx: fix build with make and recent gcc
> ethdev: limit maximum number of queues
>
> Tiwei Bie (5):
> doc: fix format in virtio guide
> vhost: fix slave request fd leak
> vhost: forbid reallocation when running
> vhost: fix vring address handling during live migration
> vhost: protect vring access done by application
>
> Vakul Garg (1):
> examples/ipsec-secgw: fix SHA256-HMAC digest length
>
> Venkat Duvvuru (1):
> net/bnxt: fix log message level
>
> Wangyu (Eric) (1):
> bus/pci: align next mapping address on page boundary
>
> Wei Zhao (2):
> net/ixgbe: enable new PF host mbox version
> net/ixgbe: fix VF RSS offloads configuration
>
> Xiao Wang (2):
> doc: fix typo in l2fwd-crypto guide
> net/fm10k: fix mbuf free in vector Rx
>
> Xiao Zhang (3):
> net/ixgbe: fix X553 speed capability
> examples/l3fwd-power: fix Rx interrupt disabling
> net/ixgbe: fix link status
>
> Xiaolong Ye (1):
> ethdev: fix typos for ENOTSUP
>
> Xueming Li (2):
> malloc: fix realloc copy size
> malloc: fix realloc padded element size
>
> Yong Wang (1):
> net/ixgbe: remove redundant assignment
>
>
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-13 12:47 [dpdk-dev] 17.11.10 (LTS) patches review and test Luca Boccassi
2020-01-17 2:52 ` Pei Zhang
@ 2020-01-22 5:27 ` Yu, PingX
2020-01-22 11:37 ` Luca Boccassi
2020-01-22 16:06 ` [dpdk-dev] " Ali Alnubani
2020-01-30 10:08 ` Luca Boccassi
3 siblings, 1 reply; 18+ messages in thread
From: Yu, PingX @ 2020-01-22 5:27 UTC (permalink / raw)
To: Luca Boccassi, dpdk stable
Cc: dev, Akhil Goyal, Ali Alnubani, Walker, Benjamin,
David Christensen, Hemant Agrawal, Stokes, Ian, Jerin Jacob,
Mcnamara, John, Kevin Traynor, Pei Zhang, Xu, Qian Q,
Raslan Darawsheh, Thomas Monjalon, Peng, Yuan, Chen, Zhaoyan
Luca,
Update the regression test result of Intel part. See the details as below.
* Intel(R) Testing
# Basic Intel(R) NIC testing
* PF(i40e): one compile issue is found and we fix it with attached patch to go ahead the test. All passed.
1. Bugs:make -j 110 -C test Compile failed
* PF(ixgbe): the same as PF(i40e).
* VF: Pass
* Build or compile: Pass
* Intel NIC single core/NIC performance: Pass
#Basic cryptodev and virtio testing
* vhost/virtio basic loopback, PVP and performance test: Pass.
* cryptodev: 1 bug is found.
1. [dpdk-stable-17.11.10]Crypto:cryptodev_qat_autotest test failed. PS: issue passed on 17.11.9.
Regards,
Yu Ping
> -----Original Message-----
> From: Luca Boccassi [mailto:bluca@debian.org]
> Sent: Monday, January 13, 2020 8:47 PM
> To: dpdk stable <stable@dpdk.org>
> Cc: dev@dpdk.org; 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>; Kevin
> Traynor <ktraynor@redhat.com>; 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: 17.11.10 (LTS) patches review and test
>
> Hi all,
>
> Here is a list of patches targeted for LTS release 17.11.10.
> This will be the last 17.11 release.
>
> The planned date for the final release is the 30th of January.
>
> 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=v17.11.10-rc1
>
> These patches are located at branch 17.11 of dpdk-stable repo:
> https://dpdk.org/browse/dpdk-stable/
>
> Thanks.
>
> Luca Boccassi
>
> ---
> Aaron Conole (1):
> test/interrupt: account for race with callback
>
> Abhishek Sachan (1):
> net/af_packet: fix stale sockets
>
> Adrian Moreno (4):
> vhost: translate incoming log address to GPA
> vhost: prevent zero copy mode if IOMMU is on
> vhost: fix vring memory partially mapped
> vhost: convert buffer addresses to GPA for logging
>
> Ajit Khaparde (3):
> net/bnxt: fix async link handling and update
> net/bnxt: get default HWRM command timeout from FW
> net/bnxt: fix crash in xstats get
>
> Akhil Goyal (1):
> crypto/dpaa2_sec: fix length retrieved from hardware
>
> Andrew Rybchenko (3):
> ethdev: remove redundant device info cleanup before get
> net/virtio: fix descriptor addressed in Tx
> net/virtio: fix Tx checksum offloads
>
> Andrzej Ostruszka (3):
> doc: fix description of versioning macros
> net/dpaa2: fix possible use of uninitialized vars
> doc: fix tap guide
>
> Archana Muniganti (1):
> app/crypto-perf: fix input of AEAD decrypt
>
> Arek Kusztal (1):
> crypto/qat: fix null auth issues when using vfio_pci
>
> Bruce Richardson (1):
> examples/vm_power: fix type of cmdline token in cli
>
> Christian Ehrhardt (2):
> kni: fix build with kernel 5.3
> net/mlx4: fix build on ppc64
>
> Ciara Power (3):
> app/testpmd: fix help for loop topology option
> ethdev: fix include of ethernet header file
> app/proc_info: fix string copying to use strlcpy
>
> David Hunt (1):
> examples/vm_power: fix build without i40e
>
> David Marchand (7):
> net/bonding: fix LACP fast queue Rx handler
> net/bonding: fix unicast packets filtering
> ethdev: fix endian annotation for SPI item
> net/qede: limit Rx ring index read for debug
> net/bonding: use non deprecated PCI API
> doc/guides: clean repeated words
> eal: remove dead code on NUMA node detection
>
> Dilshod Urazov (2):
> net/virtio: reject deferred Rx start
> net/virtio: reject deferred Tx start
>
> Eelco Chaudron (1):
> net/i40e: downgrade error log
>
> Fan Zhang (1):
> doc: fix AESNI-GCM limitations in crypto guide
>
> Ferruh Yigit (3):
> net/igb: fix global variable multiple definitions
> test: fix global variable multiple definitions
> mk: remove library search path from binary
>
> Fiona Trahe (1):
> crypto/qat: fix digest length in XCBC capability
>
> Flavia Musatescu (1):
> net/af_packet: improve Tx statistics accuracy
>
> Flavio Leitner (1):
> vhost: fix IPv4 checksum
>
> Gage Eads (1):
> event/sw: fix xstats reset value
>
> Gavin Hu (5):
> ring: enforce reading tail before slots
> net/i40e: remove memory barrier from NEON Rx
> net/i40e: remove compiler barrier from NEON Rx
> net/bnxt: remove duplicate barrier
> net/bnxt: enforce IO barrier for doorbell command
>
> Harry van Haaren (1):
> test/service: fix wait for service core
>
> Hemant Agrawal (1):
> event/dpaa2: fix default queue configuration
>
> Honnappa Nagarahalli (1):
> test/lpm: fix measured cycles for delete
>
> Hui Zhao (2):
> net/bonding: fix slave id types
> net/bonding: fix OOB access in other aggregator modes
>
> Igor Romanov (1):
> net/bonding: fix link speed update in broadcast mode
>
> Ivan Malov (1):
> net/sfc: fix adapter lock usage on rule creation
>
> Joyce Kong (2):
> net/i40e: fix address of first segment
> net/ixgbe: fix address of first segment
>
> Julien Meunier (2):
> cryptodev: fix initialization on multi-process
> cryptodev: fix checks related to device id
>
> Junyu Jiang (2):
> net/ixgbe: fix queue interrupt for X552/557
> net/bonding: fix port ID check
>
> Kalesh AP (10):
> net/bnxt: fix setting default MAC address
> net/bnxt: fix multicast filter programming
> net/bnxt: return error if setting link up fails
> net/bnxt: remove redundant header file inclusion
> net/bnxt: fix coding style
> net/bnxt: remove unnecessary variable assignment
> net/bnxt: fix dereference before null check
> net/bnxt: cleanup comments
> net/bnxt: move macro definitions to header file
> net/bnxt: fix memory leak
>
> Kevin Traynor (3):
> net/bnxt: remove commented out code
> lib: fix log typos
> lib: fix doxygen typos
>
> Krzysztof Kanas (3):
> test/bonding: fix LSC related cases
> test: optimise fd closing in forks
> net/bonding: fix selection logic
>
> Lance Richardson (1):
> net/bnxt: fix mbuf free when clearing Tx queue
>
> Lu Qiuwen (1):
> net/fm10k: fix stats crash in multi-process
>
> Lukasz Bartosik (1):
> examples/ipsec-secgw: fix default configuration
>
> Lunyuan Cui (2):
> net/ixgbe: fix link status
> net/e1000: fix link status
>
> Marcin Hajkowski (1):
> power: fix socket indicator value
>
> Marcin Smoczynski (2):
> net/tap: fix blocked Rx packets
> examples/ipsec-secgw: fix GCM IV length
>
> Marvin Liu (3):
> net/virtio: fix mbuf data and packet length mismatch
> vhost: fix virtqueue not accessible
> net/virtio-user: fix setting filters
>
> Mattias Rönnblom (1):
> power: handle frequency increase with turbo disabled
>
> Maxime Coquelin (1):
> vhost: un-inline dirty pages logging functions
>
> Michal Krawczyk (1):
> vfio: fix truncated BAR offset for 32-bit
>
> Nipun Gupta (2):
> net/dpaa2: set port in mbuf
> net/dpaa2: add retry and timeout in packet enqueue API
>
> Noa Ezra (1):
> net/vhost: fix redundant queue state event
>
> Patrick Fu (1):
> net/virtio: init MTU in case no control channel
>
> Pavan Nikhilesh (2):
> app/testpmd: use better randomness for Tx split
> event/octeontx: fix partial Rx packet handling
>
> Peng Huang (1):
> app/testpmd: fix Tx checksum when TSO enabled
>
> Phil Yang (1):
> net/mlx5: fix Rx CQ doorbell synchronization on aarch64
>
> Radu Bulie (1):
> mempool/dpaa2: report error on endless loop in mbuf release
>
> Radu Nicolau (1):
> security: fix doxygen fields
>
> Rahul Gupta (1):
> net/bnxt: fix Rx queue count
>
> Rahul Lakkireddy (1):
> net/cxgbe: fix prefetch for non-coalesced Tx packets
>
> Raslan Darawsheh (1):
> net/mlx: fix debug build with icc
>
> Robin Jarry (1):
> usertools: fix pmdinfo with python 3 and pyelftools>=0.24
>
> Ruifeng Wang (3):
> net/ixgbe: remove memory barrier from NEON Rx
> lib/distributor: fix deadlock on aarch64
> net/ixgbe: support packet type with NEON
>
> Seth Howell (1):
> bus/pci: remove useless link dependency on ethdev
>
> Shahed Shaikh (2):
> net/qede: fix setting MTU
> net/qede: fix setting VLAN strip mode
>
> Shougang Wang (1):
> app/testpmd: fix crash on port reset
>
> Shweta Choudaha (1):
> net/igb: fix PHY status if PHY reset is not blocked
>
> Somnath Kotur (1):
> net/bnxt: expose some missing counters in port stats
>
> Stephen Hemminger (6):
> bus/pci: fix Intel IOMMU sysfs access check
> net/bnxt: fix crash in secondary process
> service: use log for error messages
> test/mbuf: fix forged mbuf in clone test
> app/testpmd: block xstats for hidden ports
> examples/multi_process: fix client crash with sparse ports
>
> Thibaut Collet (1):
> net/virtio: get all pending Rx packets in vectorized paths
>
> Thierry Herbelot (3):
> drivers/crypto: remove some invalid comments
> doc: fix a common typo in NIC guides
> crypto/openssl: use local copy for session contexts
>
> Thomas Monjalon (3):
> doc: robustify PDF build
> net/mlx: fix build with make and recent gcc
> ethdev: limit maximum number of queues
>
> Tiwei Bie (5):
> doc: fix format in virtio guide
> vhost: fix slave request fd leak
> vhost: forbid reallocation when running
> vhost: fix vring address handling during live migration
> vhost: protect vring access done by application
>
> Vakul Garg (1):
> examples/ipsec-secgw: fix SHA256-HMAC digest length
>
> Venkat Duvvuru (1):
> net/bnxt: fix log message level
>
> Wangyu (Eric) (1):
> bus/pci: align next mapping address on page boundary
>
> Wei Zhao (2):
> net/ixgbe: enable new PF host mbox version
> net/ixgbe: fix VF RSS offloads configuration
>
> Xiao Wang (2):
> doc: fix typo in l2fwd-crypto guide
> net/fm10k: fix mbuf free in vector Rx
>
> Xiao Zhang (3):
> net/ixgbe: fix X553 speed capability
> examples/l3fwd-power: fix Rx interrupt disabling
> net/ixgbe: fix link status
>
> Xiaolong Ye (1):
> ethdev: fix typos for ENOTSUP
>
> Xueming Li (2):
> malloc: fix realloc copy size
> malloc: fix realloc padded element size
>
> Yong Wang (1):
> net/ixgbe: remove redundant assignment
>
> --
> Kind regards,
> Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-22 5:27 ` Yu, PingX
@ 2020-01-22 11:37 ` Luca Boccassi
2020-01-22 14:17 ` [dpdk-dev] [dpdk-stable] " Kevin Traynor
0 siblings, 1 reply; 18+ messages in thread
From: Luca Boccassi @ 2020-01-22 11:37 UTC (permalink / raw)
To: Yu, PingX, dpdk stable
Cc: dev, Walker, Benjamin, Xu, Qian Q, Xu, Qian Q, Raslan Darawsheh,
Thomas Monjalon, Peng, Yuan, Chen, Zhaoyan, Damian Nowak,
Arek Kusztal, Fiona Trahe, john.griffin, deepak.k.jain
On Wed, 2020-01-22 at 05:27 +0000, Yu, PingX wrote:
> Luca,
> Update the regression test result of Intel part. See the details as
> below.
>
> * Intel(R) Testing
> # Basic Intel(R) NIC testing
> * PF(i40e): one compile issue is found and we fix it with attached
> patch to go ahead the test. All passed.
> 1. Bugs:make -j 110 -C test Compile failed
> * PF(ixgbe): the same as PF(i40e).
> * VF: Pass
> * Build or compile: Pass
> * Intel NIC single core/NIC performance: Pass
>
> #Basic cryptodev and virtio testing
> * vhost/virtio basic loopback, PVP and performance test: Pass.
> * cryptodev: 1 bug is found.
> 1. [dpdk-stable-17.11.10]Crypto:cryptodev_qat_autotest test failed.
> PS: issue passed on 17.11.9.
>
> Regards,
> Yu Ping
Hello Yu,
Thanks for the report. For the first issue, you mention the patch is
attached, but there were no attachments in your mail?
For the crypto/qat issue, could you please tell us what's the exact
failure?
CC'ing the QAT developers and maintainers: I have done the following
backports, could you please double check them:
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=70116ae88ca7276591f9b16650386c828a92b0c8
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=1c015c688197f89664ef7e6360ab0ef0131de5af
>
> > -----Original Message-----
> > From: Luca Boccassi [mailto:
> > bluca@debian.org
> > ]
> > Sent: Monday, January 13, 2020 8:47 PM
> > To: dpdk stable <
> > stable@dpdk.org
> > >
> > Cc:
> > dev@dpdk.org
> > ; 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
> > >; Kevin
> > Traynor <
> > ktraynor@redhat.com
> > >; 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: 17.11.10 (LTS) patches review and test
> >
> > Hi all,
> >
> > Here is a list of patches targeted for LTS release 17.11.10.
> > This will be the last 17.11 release.
> >
> > The planned date for the final release is the 30th of January.
> >
> > 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=v17.11.10-rc1
> >
> >
> > These patches are located at branch 17.11 of dpdk-stable repo:
> >
> > https://dpdk.org/browse/dpdk-stable/
> >
> >
> > Thanks.
> >
> > Luca Boccassi
> >
> > ---
> > Aaron Conole (1):
> > test/interrupt: account for race with callback
> >
> > Abhishek Sachan (1):
> > net/af_packet: fix stale sockets
> >
> > Adrian Moreno (4):
> > vhost: translate incoming log address to GPA
> > vhost: prevent zero copy mode if IOMMU is on
> > vhost: fix vring memory partially mapped
> > vhost: convert buffer addresses to GPA for logging
> >
> > Ajit Khaparde (3):
> > net/bnxt: fix async link handling and update
> > net/bnxt: get default HWRM command timeout from FW
> > net/bnxt: fix crash in xstats get
> >
> > Akhil Goyal (1):
> > crypto/dpaa2_sec: fix length retrieved from hardware
> >
> > Andrew Rybchenko (3):
> > ethdev: remove redundant device info cleanup before get
> > net/virtio: fix descriptor addressed in Tx
> > net/virtio: fix Tx checksum offloads
> >
> > Andrzej Ostruszka (3):
> > doc: fix description of versioning macros
> > net/dpaa2: fix possible use of uninitialized vars
> > doc: fix tap guide
> >
> > Archana Muniganti (1):
> > app/crypto-perf: fix input of AEAD decrypt
> >
> > Arek Kusztal (1):
> > crypto/qat: fix null auth issues when using vfio_pci
> >
> > Bruce Richardson (1):
> > examples/vm_power: fix type of cmdline token in cli
> >
> > Christian Ehrhardt (2):
> > kni: fix build with kernel 5.3
> > net/mlx4: fix build on ppc64
> >
> > Ciara Power (3):
> > app/testpmd: fix help for loop topology option
> > ethdev: fix include of ethernet header file
> > app/proc_info: fix string copying to use strlcpy
> >
> > David Hunt (1):
> > examples/vm_power: fix build without i40e
> >
> > David Marchand (7):
> > net/bonding: fix LACP fast queue Rx handler
> > net/bonding: fix unicast packets filtering
> > ethdev: fix endian annotation for SPI item
> > net/qede: limit Rx ring index read for debug
> > net/bonding: use non deprecated PCI API
> > doc/guides: clean repeated words
> > eal: remove dead code on NUMA node detection
> >
> > Dilshod Urazov (2):
> > net/virtio: reject deferred Rx start
> > net/virtio: reject deferred Tx start
> >
> > Eelco Chaudron (1):
> > net/i40e: downgrade error log
> >
> > Fan Zhang (1):
> > doc: fix AESNI-GCM limitations in crypto guide
> >
> > Ferruh Yigit (3):
> > net/igb: fix global variable multiple definitions
> > test: fix global variable multiple definitions
> > mk: remove library search path from binary
> >
> > Fiona Trahe (1):
> > crypto/qat: fix digest length in XCBC capability
> >
> > Flavia Musatescu (1):
> > net/af_packet: improve Tx statistics accuracy
> >
> > Flavio Leitner (1):
> > vhost: fix IPv4 checksum
> >
> > Gage Eads (1):
> > event/sw: fix xstats reset value
> >
> > Gavin Hu (5):
> > ring: enforce reading tail before slots
> > net/i40e: remove memory barrier from NEON Rx
> > net/i40e: remove compiler barrier from NEON Rx
> > net/bnxt: remove duplicate barrier
> > net/bnxt: enforce IO barrier for doorbell command
> >
> > Harry van Haaren (1):
> > test/service: fix wait for service core
> >
> > Hemant Agrawal (1):
> > event/dpaa2: fix default queue configuration
> >
> > Honnappa Nagarahalli (1):
> > test/lpm: fix measured cycles for delete
> >
> > Hui Zhao (2):
> > net/bonding: fix slave id types
> > net/bonding: fix OOB access in other aggregator modes
> >
> > Igor Romanov (1):
> > net/bonding: fix link speed update in broadcast mode
> >
> > Ivan Malov (1):
> > net/sfc: fix adapter lock usage on rule creation
> >
> > Joyce Kong (2):
> > net/i40e: fix address of first segment
> > net/ixgbe: fix address of first segment
> >
> > Julien Meunier (2):
> > cryptodev: fix initialization on multi-process
> > cryptodev: fix checks related to device id
> >
> > Junyu Jiang (2):
> > net/ixgbe: fix queue interrupt for X552/557
> > net/bonding: fix port ID check
> >
> > Kalesh AP (10):
> > net/bnxt: fix setting default MAC address
> > net/bnxt: fix multicast filter programming
> > net/bnxt: return error if setting link up fails
> > net/bnxt: remove redundant header file inclusion
> > net/bnxt: fix coding style
> > net/bnxt: remove unnecessary variable assignment
> > net/bnxt: fix dereference before null check
> > net/bnxt: cleanup comments
> > net/bnxt: move macro definitions to header file
> > net/bnxt: fix memory leak
> >
> > Kevin Traynor (3):
> > net/bnxt: remove commented out code
> > lib: fix log typos
> > lib: fix doxygen typos
> >
> > Krzysztof Kanas (3):
> > test/bonding: fix LSC related cases
> > test: optimise fd closing in forks
> > net/bonding: fix selection logic
> >
> > Lance Richardson (1):
> > net/bnxt: fix mbuf free when clearing Tx queue
> >
> > Lu Qiuwen (1):
> > net/fm10k: fix stats crash in multi-process
> >
> > Lukasz Bartosik (1):
> > examples/ipsec-secgw: fix default configuration
> >
> > Lunyuan Cui (2):
> > net/ixgbe: fix link status
> > net/e1000: fix link status
> >
> > Marcin Hajkowski (1):
> > power: fix socket indicator value
> >
> > Marcin Smoczynski (2):
> > net/tap: fix blocked Rx packets
> > examples/ipsec-secgw: fix GCM IV length
> >
> > Marvin Liu (3):
> > net/virtio: fix mbuf data and packet length mismatch
> > vhost: fix virtqueue not accessible
> > net/virtio-user: fix setting filters
> >
> > Mattias Rönnblom (1):
> > power: handle frequency increase with turbo disabled
> >
> > Maxime Coquelin (1):
> > vhost: un-inline dirty pages logging functions
> >
> > Michal Krawczyk (1):
> > vfio: fix truncated BAR offset for 32-bit
> >
> > Nipun Gupta (2):
> > net/dpaa2: set port in mbuf
> > net/dpaa2: add retry and timeout in packet enqueue API
> >
> > Noa Ezra (1):
> > net/vhost: fix redundant queue state event
> >
> > Patrick Fu (1):
> > net/virtio: init MTU in case no control channel
> >
> > Pavan Nikhilesh (2):
> > app/testpmd: use better randomness for Tx split
> > event/octeontx: fix partial Rx packet handling
> >
> > Peng Huang (1):
> > app/testpmd: fix Tx checksum when TSO enabled
> >
> > Phil Yang (1):
> > net/mlx5: fix Rx CQ doorbell synchronization on aarch64
> >
> > Radu Bulie (1):
> > mempool/dpaa2: report error on endless loop in mbuf release
> >
> > Radu Nicolau (1):
> > security: fix doxygen fields
> >
> > Rahul Gupta (1):
> > net/bnxt: fix Rx queue count
> >
> > Rahul Lakkireddy (1):
> > net/cxgbe: fix prefetch for non-coalesced Tx packets
> >
> > Raslan Darawsheh (1):
> > net/mlx: fix debug build with icc
> >
> > Robin Jarry (1):
> > usertools: fix pmdinfo with python 3 and pyelftools>=0.24
> >
> > Ruifeng Wang (3):
> > net/ixgbe: remove memory barrier from NEON Rx
> > lib/distributor: fix deadlock on aarch64
> > net/ixgbe: support packet type with NEON
> >
> > Seth Howell (1):
> > bus/pci: remove useless link dependency on ethdev
> >
> > Shahed Shaikh (2):
> > net/qede: fix setting MTU
> > net/qede: fix setting VLAN strip mode
> >
> > Shougang Wang (1):
> > app/testpmd: fix crash on port reset
> >
> > Shweta Choudaha (1):
> > net/igb: fix PHY status if PHY reset is not blocked
> >
> > Somnath Kotur (1):
> > net/bnxt: expose some missing counters in port stats
> >
> > Stephen Hemminger (6):
> > bus/pci: fix Intel IOMMU sysfs access check
> > net/bnxt: fix crash in secondary process
> > service: use log for error messages
> > test/mbuf: fix forged mbuf in clone test
> > app/testpmd: block xstats for hidden ports
> > examples/multi_process: fix client crash with sparse ports
> >
> > Thibaut Collet (1):
> > net/virtio: get all pending Rx packets in vectorized paths
> >
> > Thierry Herbelot (3):
> > drivers/crypto: remove some invalid comments
> > doc: fix a common typo in NIC guides
> > crypto/openssl: use local copy for session contexts
> >
> > Thomas Monjalon (3):
> > doc: robustify PDF build
> > net/mlx: fix build with make and recent gcc
> > ethdev: limit maximum number of queues
> >
> > Tiwei Bie (5):
> > doc: fix format in virtio guide
> > vhost: fix slave request fd leak
> > vhost: forbid reallocation when running
> > vhost: fix vring address handling during live migration
> > vhost: protect vring access done by application
> >
> > Vakul Garg (1):
> > examples/ipsec-secgw: fix SHA256-HMAC digest length
> >
> > Venkat Duvvuru (1):
> > net/bnxt: fix log message level
> >
> > Wangyu (Eric) (1):
> > bus/pci: align next mapping address on page boundary
> >
> > Wei Zhao (2):
> > net/ixgbe: enable new PF host mbox version
> > net/ixgbe: fix VF RSS offloads configuration
> >
> > Xiao Wang (2):
> > doc: fix typo in l2fwd-crypto guide
> > net/fm10k: fix mbuf free in vector Rx
> >
> > Xiao Zhang (3):
> > net/ixgbe: fix X553 speed capability
> > examples/l3fwd-power: fix Rx interrupt disabling
> > net/ixgbe: fix link status
> >
> > Xiaolong Ye (1):
> > ethdev: fix typos for ENOTSUP
> >
> > Xueming Li (2):
> > malloc: fix realloc copy size
> > malloc: fix realloc padded element size
> >
> > Yong Wang (1):
> > net/ixgbe: remove redundant assignment
> >
> > --
> > Kind regards,
> > Luca Boccassi
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] [dpdk-stable] 17.11.10 (LTS) patches review and test
2020-01-22 11:37 ` Luca Boccassi
@ 2020-01-22 14:17 ` Kevin Traynor
2020-01-22 16:07 ` Luca Boccassi
0 siblings, 1 reply; 18+ messages in thread
From: Kevin Traynor @ 2020-01-22 14:17 UTC (permalink / raw)
To: Luca Boccassi, Yu, PingX, dpdk stable
Cc: dev, Walker, Benjamin, Xu, Qian Q, Raslan Darawsheh,
Thomas Monjalon, Peng, Yuan, Chen, Zhaoyan, Damian Nowak,
Arek Kusztal, Fiona Trahe, john.griffin, deepak.k.jain
On 22/01/2020 12:37, Luca Boccassi wrote:
> On Wed, 2020-01-22 at 05:27 +0000, Yu, PingX wrote:
>> Luca,
>> Update the regression test result of Intel part. See the details as
>> below.
>>
>> * Intel(R) Testing
>> # Basic Intel(R) NIC testing
>> * PF(i40e): one compile issue is found and we fix it with attached
>> patch to go ahead the test. All passed.
>> 1. Bugs:make -j 110 -C test Compile failed
>> * PF(ixgbe): the same as PF(i40e).
>> * VF: Pass
>> * Build or compile: Pass
>> * Intel NIC single core/NIC performance: Pass
>>
>> #Basic cryptodev and virtio testing
>> * vhost/virtio basic loopback, PVP and performance test: Pass.
>> * cryptodev: 1 bug is found.
>> 1. [dpdk-stable-17.11.10]Crypto:cryptodev_qat_autotest test failed.
>> PS: issue passed on 17.11.9.
>>
>> Regards,
>> Yu Ping
>
> Hello Yu,
>
> Thanks for the report. For the first issue, you mention the patch is
> attached, but there were no attachments in your mail?
>
I got the attachment, pasting below as it's straightforward. Didn't
check but I wonder is that the right Fixes commit, or another commit
removed path[] being used.
From bfb6609bc7c8d9d19f74eff6798c9a6931fe0d75 Mon Sep 17 00:00:00 2001
From: Jiang Mao <maox.jiang@intel.com>
Date: Fri, 17 Jan 2020 12:53:11 -0500
Subject: [PATCH] test/test: fix test app defining unused variable
Delete the unused variable to solve compiling error.
Signed-off-by: Jiang Mao <maox.jiang@intel.com>
Fixes: af75078f ("first public release")
---
Note: This patch just fixes v17.11.10-rc1, the other version have no
compiling error.
test/test/process.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/test/test/process.h b/test/test/process.h
index 505d2f9..1c369e8 100644
--- a/test/test/process.h
+++ b/test/test/process.h
@@ -56,7 +56,6 @@
int num;
char *argv_cpy[numargs + 1];
int i, status;
- char path[32];
pid_t pid = fork();
if (pid < 0)
--
1.8.3.1
> For the crypto/qat issue, could you please tell us what's the exact
> failure?
>
> CC'ing the QAT developers and maintainers: I have done the following
> backports, could you please double check them:
>
> https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=70116ae88ca7276591f9b16650386c828a92b0c8
> https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=1c015c688197f89664ef7e6360ab0ef0131de5af
>
>>
>>> -----Original Message-----
>>> From: Luca Boccassi [mailto:
>>> bluca@debian.org
>>> ]
>>> Sent: Monday, January 13, 2020 8:47 PM
>>> To: dpdk stable <
>>> stable@dpdk.org
>>>>
>>> Cc:
>>> dev@dpdk.org
>>> ; 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
>>>> ; Kevin
>>> Traynor <
>>> ktraynor@redhat.com
>>>> ; 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: 17.11.10 (LTS) patches review and test
>>>
>>> Hi all,
>>>
>>> Here is a list of patches targeted for LTS release 17.11.10.
>>> This will be the last 17.11 release.
>>>
>>> The planned date for the final release is the 30th of January.
>>>
>>> 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=v17.11.10-rc1
>>>
>>>
>>> These patches are located at branch 17.11 of dpdk-stable repo:
>>>
>>> https://dpdk.org/browse/dpdk-stable/
>>>
>>>
>>> Thanks.
>>>
>>> Luca Boccassi
>>>
>>> ---
>>> Aaron Conole (1):
>>> test/interrupt: account for race with callback
>>>
>>> Abhishek Sachan (1):
>>> net/af_packet: fix stale sockets
>>>
>>> Adrian Moreno (4):
>>> vhost: translate incoming log address to GPA
>>> vhost: prevent zero copy mode if IOMMU is on
>>> vhost: fix vring memory partially mapped
>>> vhost: convert buffer addresses to GPA for logging
>>>
>>> Ajit Khaparde (3):
>>> net/bnxt: fix async link handling and update
>>> net/bnxt: get default HWRM command timeout from FW
>>> net/bnxt: fix crash in xstats get
>>>
>>> Akhil Goyal (1):
>>> crypto/dpaa2_sec: fix length retrieved from hardware
>>>
>>> Andrew Rybchenko (3):
>>> ethdev: remove redundant device info cleanup before get
>>> net/virtio: fix descriptor addressed in Tx
>>> net/virtio: fix Tx checksum offloads
>>>
>>> Andrzej Ostruszka (3):
>>> doc: fix description of versioning macros
>>> net/dpaa2: fix possible use of uninitialized vars
>>> doc: fix tap guide
>>>
>>> Archana Muniganti (1):
>>> app/crypto-perf: fix input of AEAD decrypt
>>>
>>> Arek Kusztal (1):
>>> crypto/qat: fix null auth issues when using vfio_pci
>>>
>>> Bruce Richardson (1):
>>> examples/vm_power: fix type of cmdline token in cli
>>>
>>> Christian Ehrhardt (2):
>>> kni: fix build with kernel 5.3
>>> net/mlx4: fix build on ppc64
>>>
>>> Ciara Power (3):
>>> app/testpmd: fix help for loop topology option
>>> ethdev: fix include of ethernet header file
>>> app/proc_info: fix string copying to use strlcpy
>>>
>>> David Hunt (1):
>>> examples/vm_power: fix build without i40e
>>>
>>> David Marchand (7):
>>> net/bonding: fix LACP fast queue Rx handler
>>> net/bonding: fix unicast packets filtering
>>> ethdev: fix endian annotation for SPI item
>>> net/qede: limit Rx ring index read for debug
>>> net/bonding: use non deprecated PCI API
>>> doc/guides: clean repeated words
>>> eal: remove dead code on NUMA node detection
>>>
>>> Dilshod Urazov (2):
>>> net/virtio: reject deferred Rx start
>>> net/virtio: reject deferred Tx start
>>>
>>> Eelco Chaudron (1):
>>> net/i40e: downgrade error log
>>>
>>> Fan Zhang (1):
>>> doc: fix AESNI-GCM limitations in crypto guide
>>>
>>> Ferruh Yigit (3):
>>> net/igb: fix global variable multiple definitions
>>> test: fix global variable multiple definitions
>>> mk: remove library search path from binary
>>>
>>> Fiona Trahe (1):
>>> crypto/qat: fix digest length in XCBC capability
>>>
>>> Flavia Musatescu (1):
>>> net/af_packet: improve Tx statistics accuracy
>>>
>>> Flavio Leitner (1):
>>> vhost: fix IPv4 checksum
>>>
>>> Gage Eads (1):
>>> event/sw: fix xstats reset value
>>>
>>> Gavin Hu (5):
>>> ring: enforce reading tail before slots
>>> net/i40e: remove memory barrier from NEON Rx
>>> net/i40e: remove compiler barrier from NEON Rx
>>> net/bnxt: remove duplicate barrier
>>> net/bnxt: enforce IO barrier for doorbell command
>>>
>>> Harry van Haaren (1):
>>> test/service: fix wait for service core
>>>
>>> Hemant Agrawal (1):
>>> event/dpaa2: fix default queue configuration
>>>
>>> Honnappa Nagarahalli (1):
>>> test/lpm: fix measured cycles for delete
>>>
>>> Hui Zhao (2):
>>> net/bonding: fix slave id types
>>> net/bonding: fix OOB access in other aggregator modes
>>>
>>> Igor Romanov (1):
>>> net/bonding: fix link speed update in broadcast mode
>>>
>>> Ivan Malov (1):
>>> net/sfc: fix adapter lock usage on rule creation
>>>
>>> Joyce Kong (2):
>>> net/i40e: fix address of first segment
>>> net/ixgbe: fix address of first segment
>>>
>>> Julien Meunier (2):
>>> cryptodev: fix initialization on multi-process
>>> cryptodev: fix checks related to device id
>>>
>>> Junyu Jiang (2):
>>> net/ixgbe: fix queue interrupt for X552/557
>>> net/bonding: fix port ID check
>>>
>>> Kalesh AP (10):
>>> net/bnxt: fix setting default MAC address
>>> net/bnxt: fix multicast filter programming
>>> net/bnxt: return error if setting link up fails
>>> net/bnxt: remove redundant header file inclusion
>>> net/bnxt: fix coding style
>>> net/bnxt: remove unnecessary variable assignment
>>> net/bnxt: fix dereference before null check
>>> net/bnxt: cleanup comments
>>> net/bnxt: move macro definitions to header file
>>> net/bnxt: fix memory leak
>>>
>>> Kevin Traynor (3):
>>> net/bnxt: remove commented out code
>>> lib: fix log typos
>>> lib: fix doxygen typos
>>>
>>> Krzysztof Kanas (3):
>>> test/bonding: fix LSC related cases
>>> test: optimise fd closing in forks
>>> net/bonding: fix selection logic
>>>
>>> Lance Richardson (1):
>>> net/bnxt: fix mbuf free when clearing Tx queue
>>>
>>> Lu Qiuwen (1):
>>> net/fm10k: fix stats crash in multi-process
>>>
>>> Lukasz Bartosik (1):
>>> examples/ipsec-secgw: fix default configuration
>>>
>>> Lunyuan Cui (2):
>>> net/ixgbe: fix link status
>>> net/e1000: fix link status
>>>
>>> Marcin Hajkowski (1):
>>> power: fix socket indicator value
>>>
>>> Marcin Smoczynski (2):
>>> net/tap: fix blocked Rx packets
>>> examples/ipsec-secgw: fix GCM IV length
>>>
>>> Marvin Liu (3):
>>> net/virtio: fix mbuf data and packet length mismatch
>>> vhost: fix virtqueue not accessible
>>> net/virtio-user: fix setting filters
>>>
>>> Mattias Rönnblom (1):
>>> power: handle frequency increase with turbo disabled
>>>
>>> Maxime Coquelin (1):
>>> vhost: un-inline dirty pages logging functions
>>>
>>> Michal Krawczyk (1):
>>> vfio: fix truncated BAR offset for 32-bit
>>>
>>> Nipun Gupta (2):
>>> net/dpaa2: set port in mbuf
>>> net/dpaa2: add retry and timeout in packet enqueue API
>>>
>>> Noa Ezra (1):
>>> net/vhost: fix redundant queue state event
>>>
>>> Patrick Fu (1):
>>> net/virtio: init MTU in case no control channel
>>>
>>> Pavan Nikhilesh (2):
>>> app/testpmd: use better randomness for Tx split
>>> event/octeontx: fix partial Rx packet handling
>>>
>>> Peng Huang (1):
>>> app/testpmd: fix Tx checksum when TSO enabled
>>>
>>> Phil Yang (1):
>>> net/mlx5: fix Rx CQ doorbell synchronization on aarch64
>>>
>>> Radu Bulie (1):
>>> mempool/dpaa2: report error on endless loop in mbuf release
>>>
>>> Radu Nicolau (1):
>>> security: fix doxygen fields
>>>
>>> Rahul Gupta (1):
>>> net/bnxt: fix Rx queue count
>>>
>>> Rahul Lakkireddy (1):
>>> net/cxgbe: fix prefetch for non-coalesced Tx packets
>>>
>>> Raslan Darawsheh (1):
>>> net/mlx: fix debug build with icc
>>>
>>> Robin Jarry (1):
>>> usertools: fix pmdinfo with python 3 and pyelftools>=0.24
>>>
>>> Ruifeng Wang (3):
>>> net/ixgbe: remove memory barrier from NEON Rx
>>> lib/distributor: fix deadlock on aarch64
>>> net/ixgbe: support packet type with NEON
>>>
>>> Seth Howell (1):
>>> bus/pci: remove useless link dependency on ethdev
>>>
>>> Shahed Shaikh (2):
>>> net/qede: fix setting MTU
>>> net/qede: fix setting VLAN strip mode
>>>
>>> Shougang Wang (1):
>>> app/testpmd: fix crash on port reset
>>>
>>> Shweta Choudaha (1):
>>> net/igb: fix PHY status if PHY reset is not blocked
>>>
>>> Somnath Kotur (1):
>>> net/bnxt: expose some missing counters in port stats
>>>
>>> Stephen Hemminger (6):
>>> bus/pci: fix Intel IOMMU sysfs access check
>>> net/bnxt: fix crash in secondary process
>>> service: use log for error messages
>>> test/mbuf: fix forged mbuf in clone test
>>> app/testpmd: block xstats for hidden ports
>>> examples/multi_process: fix client crash with sparse ports
>>>
>>> Thibaut Collet (1):
>>> net/virtio: get all pending Rx packets in vectorized paths
>>>
>>> Thierry Herbelot (3):
>>> drivers/crypto: remove some invalid comments
>>> doc: fix a common typo in NIC guides
>>> crypto/openssl: use local copy for session contexts
>>>
>>> Thomas Monjalon (3):
>>> doc: robustify PDF build
>>> net/mlx: fix build with make and recent gcc
>>> ethdev: limit maximum number of queues
>>>
>>> Tiwei Bie (5):
>>> doc: fix format in virtio guide
>>> vhost: fix slave request fd leak
>>> vhost: forbid reallocation when running
>>> vhost: fix vring address handling during live migration
>>> vhost: protect vring access done by application
>>>
>>> Vakul Garg (1):
>>> examples/ipsec-secgw: fix SHA256-HMAC digest length
>>>
>>> Venkat Duvvuru (1):
>>> net/bnxt: fix log message level
>>>
>>> Wangyu (Eric) (1):
>>> bus/pci: align next mapping address on page boundary
>>>
>>> Wei Zhao (2):
>>> net/ixgbe: enable new PF host mbox version
>>> net/ixgbe: fix VF RSS offloads configuration
>>>
>>> Xiao Wang (2):
>>> doc: fix typo in l2fwd-crypto guide
>>> net/fm10k: fix mbuf free in vector Rx
>>>
>>> Xiao Zhang (3):
>>> net/ixgbe: fix X553 speed capability
>>> examples/l3fwd-power: fix Rx interrupt disabling
>>> net/ixgbe: fix link status
>>>
>>> Xiaolong Ye (1):
>>> ethdev: fix typos for ENOTSUP
>>>
>>> Xueming Li (2):
>>> malloc: fix realloc copy size
>>> malloc: fix realloc padded element size
>>>
>>> Yong Wang (1):
>>> net/ixgbe: remove redundant assignment
>>>
>>> --
>>> Kind regards,
>>> Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] [dpdk-stable] 17.11.10 (LTS) patches review and test
2020-01-22 14:17 ` [dpdk-dev] [dpdk-stable] " Kevin Traynor
@ 2020-01-22 16:07 ` Luca Boccassi
0 siblings, 0 replies; 18+ messages in thread
From: Luca Boccassi @ 2020-01-22 16:07 UTC (permalink / raw)
To: Kevin Traynor, Yu, PingX, dpdk stable
Cc: dev, Walker, Benjamin, Xu, Qian Q, Raslan Darawsheh,
Thomas Monjalon, Peng, Yuan, Chen, Zhaoyan, Damian Nowak,
Arek Kusztal, Fiona Trahe, john.griffin, deepak.k.jain
On Wed, 2020-01-22 at 15:17 +0100, Kevin Traynor wrote:
> On 22/01/2020 12:37, Luca Boccassi wrote:
> > On Wed, 2020-01-22 at 05:27 +0000, Yu, PingX wrote:
> > > Luca,
> > > Update the regression test result of Intel part. See the details
> > > as
> > > below.
> > >
> > > * Intel(R) Testing
> > > # Basic Intel(R) NIC testing
> > > * PF(i40e): one compile issue is found and we fix it with
> > > attached
> > > patch to go ahead the test. All passed.
> > > 1. Bugs:make -j 110 -C test Compile failed
> > > * PF(ixgbe): the same as PF(i40e).
> > > * VF: Pass
> > > * Build or compile: Pass
> > > * Intel NIC single core/NIC performance: Pass
> > >
> > > #Basic cryptodev and virtio testing
> > > * vhost/virtio basic loopback, PVP and performance test: Pass.
> > > * cryptodev: 1 bug is found.
> > > 1. [dpdk-stable-17.11.10]Crypto:cryptodev_qat_autotest test
> > > failed.
> > > PS: issue passed on 17.11.9.
> > >
> > > Regards,
> > > Yu Ping
> >
> > Hello Yu,
> >
> > Thanks for the report. For the first issue, you mention the patch
> > is
> > attached, but there were no attachments in your mail?
> >
>
> I got the attachment, pasting below as it's straightforward. Didn't
> check but I wonder is that the right Fixes commit, or another commit
> removed path[] being used.
>
> From bfb6609bc7c8d9d19f74eff6798c9a6931fe0d75 Mon Sep 17 00:00:00
> 2001
> From: Jiang Mao <
> maox.jiang@intel.com
> >
> Date: Fri, 17 Jan 2020 12:53:11 -0500
> Subject: [PATCH] test/test: fix test app defining unused variable
>
> Delete the unused variable to solve compiling error.
>
> Signed-off-by: Jiang Mao <
> maox.jiang@intel.com
> >
>
> Fixes: af75078f ("first public release")
> ---
> Note: This patch just fixes v17.11.10-rc1, the other version have no
> compiling error.
> test/test/process.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/test/test/process.h b/test/test/process.h
> index 505d2f9..1c369e8 100644
> --- a/test/test/process.h
> +++ b/test/test/process.h
> @@ -56,7 +56,6 @@
> int num;
> char *argv_cpy[numargs + 1];
> int i, status;
> - char path[32];
>
> pid_t pid = fork();
> if (pid < 0)
>
Thanks - the culprit is this botched backport which was adjusted but
not enough and the unused variable was left behind:
Fixes: 0ecbf7984bb0 ("test: optimise fd closing in forks")
Applied with adjusted fixes line
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-13 12:47 [dpdk-dev] 17.11.10 (LTS) patches review and test Luca Boccassi
2020-01-17 2:52 ` Pei Zhang
2020-01-22 5:27 ` Yu, PingX
@ 2020-01-22 16:06 ` Ali Alnubani
2020-01-22 16:34 ` Luca Boccassi
2020-01-30 10:08 ` Luca Boccassi
3 siblings, 1 reply; 18+ messages in thread
From: Ali Alnubani @ 2020-01-22 16:06 UTC (permalink / raw)
To: Luca Boccassi, dpdk stable
Cc: dev, Akhil Goyal, benjamin.walker, David Christensen,
Hemant Agrawal, Ian Stokes, Jerin Jacob, John McNamara,
Kevin Traynor, Pei Zhang, pingx.yu, qian.q.xu, Raslan Darawsheh,
Thomas Monjalon, yuan.peng, Chen, Zhaoyan
Hi,
> -----Original Message-----
> From: Luca Boccassi <bluca@debian.org>
> Sent: Monday, January 13, 2020 2:47 PM
> To: dpdk stable <stable@dpdk.org>
> Cc: dev@dpdk.org; Akhil Goyal <akhil.goyal@nxp.com>; Ali Alnubani
> <alialnu@mellanox.com>; benjamin.walker@intel.com; David Christensen
> <drc@linux.vnet.ibm.com>; Hemant Agrawal <hemant.agrawal@nxp.com>;
> Ian Stokes <ian.stokes@intel.com>; Jerin Jacob <jerinj@marvell.com>; John
> McNamara <john.mcnamara@intel.com>; Kevin Traynor
> <ktraynor@redhat.com>; Pei Zhang <pezhang@redhat.com>;
> pingx.yu@intel.com; qian.q.xu@intel.com; Raslan Darawsheh
> <rasland@mellanox.com>; Thomas Monjalon <thomas@monjalon.net>;
> yuan.peng@intel.com; Chen, Zhaoyan <zhaoyan.chen@intel.com>
> Subject: 17.11.10 (LTS) patches review and test
>
> Hi all,
>
> Here is a list of patches targeted for LTS release 17.11.10.
> This will be the last 17.11 release.
>
> The planned date for the final release is the 30th of January.
>
> 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.
>
The following cases were tested on Mellanox devices:
- Verify sending and receiving multiple types of traffic.
- testpmd xstats counter tests.
- testpmd timestamp tests.
- Changing/checking link status through testpmd.
- RTE flow and flow_director tests.
- Some RSS tests.
- VLAN stripping and insertion tests.
- Checksum and TSO tests.
- ptype tests.
- Multi-process tests.
Testing matrix:
- NIC: ConnectX-4 Lx / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-3.2.9.0 / Firmware: 14.26.4012
- NIC: ConnectX-5 / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-3.2.9.0 / Firmware: 16.26.4012
Current issues:
- The message “PMD: Failed to send netlink message: Bad file descriptor (9)” is shown at testpmd startup. Still didn’t debug its effect on functionality.
- ibv_icmd_create_cq () from /lib64/libibverbs.so.1 segfaults after setting the number of queues and restarting the ports.
- Enabling MLX4_PMD causes a build failure with OFED-4.7-3.2.9.0 (“error: unknown type name ‘off_t’”).
- Count action isn’t supported in rte_flow.
- Error creating rte_flow rules with vlan items.
Regards,
Ali
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-22 16:06 ` [dpdk-dev] " Ali Alnubani
@ 2020-01-22 16:34 ` Luca Boccassi
2020-01-23 11:05 ` Luca Boccassi
0 siblings, 1 reply; 18+ messages in thread
From: Luca Boccassi @ 2020-01-22 16:34 UTC (permalink / raw)
To: Ali Alnubani, dpdk stable; +Cc: dev, Raslan Darawsheh, Thomas Monjalon
On Wed, 2020-01-22 at 16:06 +0000, Ali Alnubani wrote:
> Hi,
>
> > -----Original Message-----
> > From: Luca Boccassi <
> > bluca@debian.org
> > >
> > Sent: Monday, January 13, 2020 2:47 PM
> > To: dpdk stable <
> > stable@dpdk.org
> > >
> > Cc:
> > dev@dpdk.org
> > ; Akhil Goyal <
> > akhil.goyal@nxp.com
> > >; Ali Alnubani
> > <
> > alialnu@mellanox.com
> > >;
> > benjamin.walker@intel.com
> > ; David Christensen
> > <
> > drc@linux.vnet.ibm.com
> > >; Hemant Agrawal <
> > hemant.agrawal@nxp.com
> > >;
> > Ian Stokes <
> > ian.stokes@intel.com
> > >; Jerin Jacob <
> > jerinj@marvell.com
> > >; John
> > McNamara <
> > john.mcnamara@intel.com
> > >; Kevin Traynor
> > <
> > ktraynor@redhat.com
> > >; Pei Zhang <
> > pezhang@redhat.com
> > >;
> > pingx.yu@intel.com
> > ;
> > qian.q.xu@intel.com
> > ; Raslan Darawsheh
> > <
> > rasland@mellanox.com
> > >; Thomas Monjalon <
> > thomas@monjalon.net
> > >;
> > yuan.peng@intel.com
> > ; Chen, Zhaoyan <
> > zhaoyan.chen@intel.com
> > >
> > Subject: 17.11.10 (LTS) patches review and test
> >
> > Hi all,
> >
> > Here is a list of patches targeted for LTS release 17.11.10.
> > This will be the last 17.11 release.
> >
> > The planned date for the final release is the 30th of January.
> >
> > 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.
> >
>
> The following cases were tested on Mellanox devices:
> - Verify sending and receiving multiple types of traffic.
> - testpmd xstats counter tests.
> - testpmd timestamp tests.
> - Changing/checking link status through testpmd.
> - RTE flow and flow_director tests.
> - Some RSS tests.
> - VLAN stripping and insertion tests.
> - Checksum and TSO tests.
> - ptype tests.
> - Multi-process tests.
>
> Testing matrix:
> - NIC: ConnectX-4 Lx / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-
> 3.2.9.0 / Firmware: 14.26.4012
> - NIC: ConnectX-5 / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-3.2.9.0
> / Firmware: 16.26.4012
>
> Current issues:
> - The message “PMD: Failed to send netlink message: Bad file
> descriptor (9)” is shown at testpmd startup. Still didn’t debug its
> effect on functionality.
> - ibv_icmd_create_cq () from /lib64/libibverbs.so.1 segfaults after
> setting the number of queues and restarting the ports.
> - Enabling MLX4_PMD causes a build failure with OFED-4.7-3.2.9.0
> (“error: unknown type name ‘off_t’”).
> - Count action isn’t supported in rte_flow.
> - Error creating rte_flow rules with vlan items.
>
>
> Regards,
> Ali
Thanks - are these new regressions since 17.11.9?
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-22 16:34 ` Luca Boccassi
@ 2020-01-23 11:05 ` Luca Boccassi
2020-01-26 14:58 ` Ali Alnubani
0 siblings, 1 reply; 18+ messages in thread
From: Luca Boccassi @ 2020-01-23 11:05 UTC (permalink / raw)
To: Ali Alnubani, dpdk stable
Cc: dev, Raslan Darawsheh, Thomas Monjalon, Viacheslav Ovsiienko,
Matan Azrad, Phil Yang, David Marchand
On Wed, 2020-01-22 at 16:34 +0000, Luca Boccassi wrote:
> On Wed, 2020-01-22 at 16:06 +0000, Ali Alnubani wrote:
> > Hi,
> >
> > > -----Original Message-----
> > > From: Luca Boccassi <
> > > bluca@debian.org
> > >
> > >
> > > Sent: Monday, January 13, 2020 2:47 PM
> > > To: dpdk stable <
> > > stable@dpdk.org
> > >
> > >
> > > Cc:
> > > dev@dpdk.org
> > >
> > > ; Akhil Goyal <
> > > akhil.goyal@nxp.com
> > >
> > > > ; Ali Alnubani
> > >
> > > <
> > > alialnu@mellanox.com
> > >
> > > > ;
> > >
> > > benjamin.walker@intel.com
> > >
> > > ; David Christensen
> > > <
> > > drc@linux.vnet.ibm.com
> > >
> > > > ; Hemant Agrawal <
> > >
> > > hemant.agrawal@nxp.com
> > >
> > > > ;
> > >
> > > Ian Stokes <
> > > ian.stokes@intel.com
> > >
> > > > ; Jerin Jacob <
> > >
> > > jerinj@marvell.com
> > >
> > > > ; John
> > >
> > > McNamara <
> > > john.mcnamara@intel.com
> > >
> > > > ; Kevin Traynor
> > >
> > > <
> > > ktraynor@redhat.com
> > >
> > > > ; Pei Zhang <
> > >
> > > pezhang@redhat.com
> > >
> > > > ;
> > >
> > > pingx.yu@intel.com
> > >
> > > ;
> > > qian.q.xu@intel.com
> > >
> > > ; Raslan Darawsheh
> > > <
> > > rasland@mellanox.com
> > >
> > > > ; Thomas Monjalon <
> > >
> > > thomas@monjalon.net
> > >
> > > > ;
> > >
> > > yuan.peng@intel.com
> > >
> > > ; Chen, Zhaoyan <
> > > zhaoyan.chen@intel.com
> > >
> > >
> > > Subject: 17.11.10 (LTS) patches review and test
> > >
> > > Hi all,
> > >
> > > Here is a list of patches targeted for LTS release 17.11.10.
> > > This will be the last 17.11 release.
> > >
> > > The planned date for the final release is the 30th of January.
> > >
> > > 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.
> > >
> >
> > The following cases were tested on Mellanox devices:
> > - Verify sending and receiving multiple types of traffic.
> > - testpmd xstats counter tests.
> > - testpmd timestamp tests.
> > - Changing/checking link status through testpmd.
> > - RTE flow and flow_director tests.
> > - Some RSS tests.
> > - VLAN stripping and insertion tests.
> > - Checksum and TSO tests.
> > - ptype tests.
> > - Multi-process tests.
> >
> > Testing matrix:
> > - NIC: ConnectX-4 Lx / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-
> > 3.2.9.0 / Firmware: 14.26.4012
> > - NIC: ConnectX-5 / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-
> > 3.2.9.0
> > / Firmware: 16.26.4012
> >
> > Current issues:
> > - The message “PMD: Failed to send netlink message: Bad file
> > descriptor (9)” is shown at testpmd startup. Still didn’t debug its
> > effect on functionality.
> > - ibv_icmd_create_cq () from /lib64/libibverbs.so.1 segfaults after
> > setting the number of queues and restarting the ports.
> > - Enabling MLX4_PMD causes a build failure with OFED-4.7-3.2.9.0
> > (“error: unknown type name ‘off_t’”).
> > - Count action isn’t supported in rte_flow.
> > - Error creating rte_flow rules with vlan items.
> >
> >
> > Regards,
> > Ali
>
> Thanks - are these new regressions since 17.11.9?
It was mentioned in the release meeting call that they are new
regressions since 17.11.10-rc1 - these are the commits that were
backported that changed drivers/mlx*:
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=60d0f2e079
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=005b17cbab
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=4cd784b02d
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=569c98371e
They seem quite innocous to me, but I might be wrong. Authors and
reviewers CC'ed.
- there's no "off_t" reference in MLX4
- the only flow change was this commit and it seems unrelated:
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=940d6c60d5748
So I'd imagine it could be some unrelated change, in the core
libraries, that causes these issues. Would you be able to bisect the
tree to see exactly which commits cause issues?
Thanks!
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-23 11:05 ` Luca Boccassi
@ 2020-01-26 14:58 ` Ali Alnubani
2020-01-27 10:41 ` Luca Boccassi
0 siblings, 1 reply; 18+ messages in thread
From: Ali Alnubani @ 2020-01-26 14:58 UTC (permalink / raw)
To: Luca Boccassi, dpdk stable
Cc: dev, Raslan Darawsheh, Thomas Monjalon, Slava Ovsiienko,
Matan Azrad, Phil Yang, David Marchand, Xueming(Steven) Li
Hi Luca,
Sorry for the late response 😊
> -----Original Message-----
> From: Luca Boccassi <bluca@debian.org>
> Sent: Thursday, January 23, 2020 1:06 PM
> To: Ali Alnubani <alialnu@mellanox.com>; dpdk stable <stable@dpdk.org>
> Cc: dev@dpdk.org; Raslan Darawsheh <rasland@mellanox.com>; Thomas
> Monjalon <thomas@monjalon.net>; Slava Ovsiienko
> <viacheslavo@mellanox.com>; Matan Azrad <matan@mellanox.com>; Phil
> Yang <phil.yang@arm.com>; David Marchand
> <david.marchand@redhat.com>
> Subject: Re: 17.11.10 (LTS) patches review and test
>
> On Wed, 2020-01-22 at 16:34 +0000, Luca Boccassi wrote:
> > On Wed, 2020-01-22 at 16:06 +0000, Ali Alnubani wrote:
> > > Hi,
> > >
> > > > -----Original Message-----
> > > > From: Luca Boccassi <
> > > > bluca@debian.org
> > > >
> > > >
> > > > Sent: Monday, January 13, 2020 2:47 PM
> > > > To: dpdk stable <
> > > > stable@dpdk.org
> > > >
> > > >
> > > > Cc:
> > > > dev@dpdk.org
> > > >
> > > > ; Akhil Goyal <
> > > > akhil.goyal@nxp.com
> > > >
> > > > > ; Ali Alnubani
> > > >
> > > > <
> > > > alialnu@mellanox.com
> > > >
> > > > > ;
> > > >
> > > > benjamin.walker@intel.com
> > > >
> > > > ; David Christensen
> > > > <
> > > > drc@linux.vnet.ibm.com
> > > >
> > > > > ; Hemant Agrawal <
> > > >
> > > > hemant.agrawal@nxp.com
> > > >
> > > > > ;
> > > >
> > > > Ian Stokes <
> > > > ian.stokes@intel.com
> > > >
> > > > > ; Jerin Jacob <
> > > >
> > > > jerinj@marvell.com
> > > >
> > > > > ; John
> > > >
> > > > McNamara <
> > > > john.mcnamara@intel.com
> > > >
> > > > > ; Kevin Traynor
> > > >
> > > > <
> > > > ktraynor@redhat.com
> > > >
> > > > > ; Pei Zhang <
> > > >
> > > > pezhang@redhat.com
> > > >
> > > > > ;
> > > >
> > > > pingx.yu@intel.com
> > > >
> > > > ;
> > > > qian.q.xu@intel.com
> > > >
> > > > ; Raslan Darawsheh
> > > > <
> > > > rasland@mellanox.com
> > > >
> > > > > ; Thomas Monjalon <
> > > >
> > > > thomas@monjalon.net
> > > >
> > > > > ;
> > > >
> > > > yuan.peng@intel.com
> > > >
> > > > ; Chen, Zhaoyan <
> > > > zhaoyan.chen@intel.com
> > > >
> > > >
> > > > Subject: 17.11.10 (LTS) patches review and test
> > > >
> > > > Hi all,
> > > >
> > > > Here is a list of patches targeted for LTS release 17.11.10.
> > > > This will be the last 17.11 release.
> > > >
> > > > The planned date for the final release is the 30th of January.
> > > >
> > > > 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.
> > > >
> > >
> > > The following cases were tested on Mellanox devices:
> > > - Verify sending and receiving multiple types of traffic.
> > > - testpmd xstats counter tests.
> > > - testpmd timestamp tests.
> > > - Changing/checking link status through testpmd.
> > > - RTE flow and flow_director tests.
> > > - Some RSS tests.
> > > - VLAN stripping and insertion tests.
> > > - Checksum and TSO tests.
> > > - ptype tests.
> > > - Multi-process tests.
> > >
> > > Testing matrix:
> > > - NIC: ConnectX-4 Lx / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-
> > > 3.2.9.0 / Firmware: 14.26.4012
> > > - NIC: ConnectX-5 / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-
> > > 3.2.9.0
> > > / Firmware: 16.26.4012
> > >
> > > Current issues:
> > > - The message “PMD: Failed to send netlink message: Bad file
> > > descriptor (9)” is shown at testpmd startup. Still didn’t debug its
> > > effect on functionality.
> > > - ibv_icmd_create_cq () from /lib64/libibverbs.so.1 segfaults after
> > > setting the number of queues and restarting the ports.
> > > - Enabling MLX4_PMD causes a build failure with OFED-4.7-3.2.9.0
> > > (“error: unknown type name ‘off_t’”).
> > > - Count action isn’t supported in rte_flow.
> > > - Error creating rte_flow rules with vlan items.
> > >
> > >
> > > Regards,
> > > Ali
> >
> > Thanks - are these new regressions since 17.11.9?
Only the segfault is caused by new changes. It is caused by commit:
https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=a10a7083d164fb48cc3978067820c3d256899f3e (Added Xueming Li to CC list.)
I should've reported the rest of the issues earlier, my bad.
>
> It was mentioned in the release meeting call that they are new regressions
> since 17.11.10-rc1 - these are the commits that were backported that
> changed drivers/mlx*:
>
<removed>
>
> They seem quite innocous to me, but I might be wrong. Authors and
> reviewers CC'ed.
>
> - there's no "off_t" reference in MLX4
This started reproducing since driver (MLNX_OFED_LINUX) version 4.6-1.0.1.1. Full error:
"""
In file included from /path/to/drivers/net/mlx4/mlx4_rxq.c:49:0:
/usr/include/infiniband/mlx4dv.h:176:2: error: unknown type name ‘off_t’
off_t uar_mmap_offset;
^
make[6]: *** [mlx4_rxq.o] Error 1
make[6]: *** Waiting for unfinished jobs....
"""
> - the only flow change was this commit and it seems unrelated:
>
<removed>
>
> So I'd imagine it could be some unrelated change, in the core libraries, that
> causes these issues. Would you be able to bisect the tree to see exactly
> which commits cause issues?
Regarding issues:
- Count action isn’t supported in rte_flow.
- Error creating rte_flow rules with vlan items.
These are related to support in the OFED driver, and we still need to do more investigation.
>
> Thanks!
>
> --
> Kind regards,
> Luca Boccassi
Regards,
Ali
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-26 14:58 ` Ali Alnubani
@ 2020-01-27 10:41 ` Luca Boccassi
0 siblings, 0 replies; 18+ messages in thread
From: Luca Boccassi @ 2020-01-27 10:41 UTC (permalink / raw)
To: Ali Alnubani, dpdk stable
Cc: dev, Raslan Darawsheh, Thomas Monjalon, Slava Ovsiienko,
Matan Azrad, Phil Yang, David Marchand, Xueming(Steven) Li
On Sun, 2020-01-26 at 14:58 +0000, Ali Alnubani wrote:
> Hi Luca,
> Sorry for the late response 😊
>
> > -----Original Message-----
> > From: Luca Boccassi <
> > bluca@debian.org
> > >
> > Sent: Thursday, January 23, 2020 1:06 PM
> > To: Ali Alnubani <
> > alialnu@mellanox.com
> > >; dpdk stable <
> > stable@dpdk.org
> > >
> > Cc:
> > dev@dpdk.org
> > ; Raslan Darawsheh <
> > rasland@mellanox.com
> > >; Thomas
> > Monjalon <
> > thomas@monjalon.net
> > >; Slava Ovsiienko
> > <
> > viacheslavo@mellanox.com
> > >; Matan Azrad <
> > matan@mellanox.com
> > >; Phil
> > Yang <
> > phil.yang@arm.com
> > >; David Marchand
> > <
> > david.marchand@redhat.com
> > >
> > Subject: Re: 17.11.10 (LTS) patches review and test
> >
> > On Wed, 2020-01-22 at 16:34 +0000, Luca Boccassi wrote:
> > > On Wed, 2020-01-22 at 16:06 +0000, Ali Alnubani wrote:
> > > > Hi,
> > > >
> > > > > -----Original Message-----
> > > > > From: Luca Boccassi <
> > > > > bluca@debian.org
> > > > >
> > > > >
> > > > >
> > > > > Sent: Monday, January 13, 2020 2:47 PM
> > > > > To: dpdk stable <
> > > > > stable@dpdk.org
> > > > >
> > > > >
> > > > >
> > > > > Cc:
> > > > > dev@dpdk.org
> > > > >
> > > > >
> > > > > ; Akhil Goyal <
> > > > > akhil.goyal@nxp.com
> > > > >
> > > > >
> > > > > > ; Ali Alnubani
> > > > >
> > > > > <
> > > > > alialnu@mellanox.com
> > > > >
> > > > >
> > > > > > ;
> > > > >
> > > > > benjamin.walker@intel.com
> > > > >
> > > > >
> > > > > ; David Christensen
> > > > > <
> > > > > drc@linux.vnet.ibm.com
> > > > >
> > > > >
> > > > > > ; Hemant Agrawal <
> > > > >
> > > > > hemant.agrawal@nxp.com
> > > > >
> > > > >
> > > > > > ;
> > > > >
> > > > > Ian Stokes <
> > > > > ian.stokes@intel.com
> > > > >
> > > > >
> > > > > > ; Jerin Jacob <
> > > > >
> > > > > jerinj@marvell.com
> > > > >
> > > > >
> > > > > > ; John
> > > > >
> > > > > McNamara <
> > > > > john.mcnamara@intel.com
> > > > >
> > > > >
> > > > > > ; Kevin Traynor
> > > > >
> > > > > <
> > > > > ktraynor@redhat.com
> > > > >
> > > > >
> > > > > > ; Pei Zhang <
> > > > >
> > > > > pezhang@redhat.com
> > > > >
> > > > >
> > > > > > ;
> > > > >
> > > > > pingx.yu@intel.com
> > > > >
> > > > >
> > > > > ;
> > > > > qian.q.xu@intel.com
> > > > >
> > > > >
> > > > > ; Raslan Darawsheh
> > > > > <
> > > > > rasland@mellanox.com
> > > > >
> > > > >
> > > > > > ; Thomas Monjalon <
> > > > >
> > > > > thomas@monjalon.net
> > > > >
> > > > >
> > > > > > ;
> > > > >
> > > > > yuan.peng@intel.com
> > > > >
> > > > >
> > > > > ; Chen, Zhaoyan <
> > > > > zhaoyan.chen@intel.com
> > > > >
> > > > >
> > > > >
> > > > > Subject: 17.11.10 (LTS) patches review and test
> > > > >
> > > > > Hi all,
> > > > >
> > > > > Here is a list of patches targeted for LTS release 17.11.10.
> > > > > This will be the last 17.11 release.
> > > > >
> > > > > The planned date for the final release is the 30th of
> > > > > January.
> > > > >
> > > > > 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.
> > > > >
> > > >
> > > > The following cases were tested on Mellanox devices:
> > > > - Verify sending and receiving multiple types of traffic.
> > > > - testpmd xstats counter tests.
> > > > - testpmd timestamp tests.
> > > > - Changing/checking link status through testpmd.
> > > > - RTE flow and flow_director tests.
> > > > - Some RSS tests.
> > > > - VLAN stripping and insertion tests.
> > > > - Checksum and TSO tests.
> > > > - ptype tests.
> > > > - Multi-process tests.
> > > >
> > > > Testing matrix:
> > > > - NIC: ConnectX-4 Lx / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-
> > > > 4.7-
> > > > 3.2.9.0 / Firmware: 14.26.4012
> > > > - NIC: ConnectX-5 / OS: RHEL7.4 / Driver: MLNX_OFED_LINUX-4.7-
> > > > 3.2.9.0
> > > > / Firmware: 16.26.4012
> > > >
> > > > Current issues:
> > > > - The message “PMD: Failed to send netlink message: Bad file
> > > > descriptor (9)” is shown at testpmd startup. Still didn’t debug
> > > > its
> > > > effect on functionality.
> > > > - ibv_icmd_create_cq () from /lib64/libibverbs.so.1 segfaults
> > > > after
> > > > setting the number of queues and restarting the ports.
> > > > - Enabling MLX4_PMD causes a build failure with OFED-4.7-
> > > > 3.2.9.0
> > > > (“error: unknown type name ‘off_t’”).
> > > > - Count action isn’t supported in rte_flow.
> > > > - Error creating rte_flow rules with vlan items.
> > > >
> > > >
> > > > Regards,
> > > > Ali
> > >
> > > Thanks - are these new regressions since 17.11.9?
>
> Only the segfault is caused by new changes. It is caused by commit:
> https://git.dpdk.org/dpdk-stable/commit/?h=17.11&id=a10a7083d164fb48cc3978067820c3d256899f3e
> (Added Xueming Li to CC list.)
> I should've reported the rest of the issues earlier, my bad.
No problem! Let's wait and see if Xueming can help us.
> > It was mentioned in the release meeting call that they are new
> > regressions
> > since 17.11.10-rc1 - these are the commits that were backported
> > that
> > changed drivers/mlx*:
> >
>
> <removed>
> > They seem quite innocous to me, but I might be wrong. Authors and
> > reviewers CC'ed.
> >
> > - there's no "off_t" reference in MLX4
>
> This started reproducing since driver (MLNX_OFED_LINUX) version 4.6-
> 1.0.1.1. Full error:
> """
> In file included from /path/to/drivers/net/mlx4/mlx4_rxq.c:49:0:
> /usr/include/infiniband/mlx4dv.h:176:2: error: unknown type name
> ‘off_t’
> off_t uar_mmap_offset;
> ^
> make[6]: *** [mlx4_rxq.o] Error 1
> make[6]: *** Waiting for unfinished jobs....
> """
If I am understanding this correctly, newer OFED versions aren't
compatible with this LTS? Is that it? Is there anything we can do about
it?
> > - the only flow change was this commit and it seems unrelated:
> >
>
> <removed>
> > So I'd imagine it could be some unrelated change, in the core
> > libraries, that
> > causes these issues. Would you be able to bisect the tree to see
> > exactly
> > which commits cause issues?
>
> Regarding issues:
> - Count action isn’t supported in rte_flow.
> - Error creating rte_flow rules with vlan items.
> These are related to support in the OFED driver, and we still need to
> do more investigation.
Thank you, I will wait for an update.
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-13 12:47 [dpdk-dev] 17.11.10 (LTS) patches review and test Luca Boccassi
` (2 preceding siblings ...)
2020-01-22 16:06 ` [dpdk-dev] " Ali Alnubani
@ 2020-01-30 10:08 ` Luca Boccassi
2020-01-30 15:01 ` Stokes, Ian
2020-02-15 11:21 ` Luca Boccassi
3 siblings, 2 replies; 18+ messages in thread
From: Luca Boccassi @ 2020-01-30 10:08 UTC (permalink / raw)
To: dpdk stable, dev
On Mon, 2020-01-13 at 12:47 +0000, Luca Boccassi wrote:
> Hi all,
>
> Here is a list of patches targeted for LTS release 17.11.10.
> This will be the last 17.11 release.
>
> The planned date for the final release is the 30th of January.
>
> 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=v17.11.10-rc1
>
>
> These patches are located at branch 17.11 of dpdk-stable repo:
>
> https://dpdk.org/browse/dpdk-stable/
>
>
> Thanks.
>
> Luca Boccassi
Hi,
We are still waiting to solve 2 issues highlighted during validation,
so we need to postpone.
New proposed target is February the 14th.
Apologies for any issue it might cause.
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-30 10:08 ` Luca Boccassi
@ 2020-01-30 15:01 ` Stokes, Ian
2020-01-30 15:25 ` Luca Boccassi
2020-02-15 11:21 ` Luca Boccassi
1 sibling, 1 reply; 18+ messages in thread
From: Stokes, Ian @ 2020-01-30 15:01 UTC (permalink / raw)
To: Luca Boccassi, dpdk stable, dev
On 1/30/2020 10:08 AM, Luca Boccassi wrote:
> On Mon, 2020-01-13 at 12:47 +0000, Luca Boccassi wrote:
>> Hi all,
>>
>> Here is a list of patches targeted for LTS release 17.11.10.
>> This will be the last 17.11 release.
>>
>> The planned date for the final release is the 30th of January.
>>
>> 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=v17.11.10-rc1
>>
>>
>> These patches are located at branch 17.11 of dpdk-stable repo:
>>
>> https://dpdk.org/browse/dpdk-stable/
>>
>>
>> Thanks.
>>
>> Luca Boccassi
>
> Hi,
>
> We are still waiting to solve 2 issues highlighted during validation,
> so we need to postpone.
>
> New proposed target is February the 14th.
>
> Apologies for any issue it might cause.
Hi Luca,
no problem on the delay, on the OVS DPDK side we've validated the
following to date
OVS testing against head OVS 2.10 and 2.19 with VSPERF
Tested NICs i40e (X710) & ixgbe (82599ES)
P2P
PVP
Hotplug
Multiqueue
Vhostuserclient reconnect
Vhost cross-NUMA awareness
Jumbo frames
Rate limiting
QoS policer
If theres a need to re-test for the follow on patches let me know.
Regards
Ian
>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-30 15:01 ` Stokes, Ian
@ 2020-01-30 15:25 ` Luca Boccassi
0 siblings, 0 replies; 18+ messages in thread
From: Luca Boccassi @ 2020-01-30 15:25 UTC (permalink / raw)
To: Stokes, Ian, dpdk stable, dev
On Thu, 2020-01-30 at 15:01 +0000, Stokes, Ian wrote:
>
> On 1/30/2020 10:08 AM, Luca Boccassi wrote:
> > On Mon, 2020-01-13 at 12:47 +0000, Luca Boccassi wrote:
> > > Hi all,
> > >
> > > Here is a list of patches targeted for LTS release 17.11.10.
> > > This will be the last 17.11 release.
> > >
> > > The planned date for the final release is the 30th of January.
> > >
> > > 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=v17.11.10-rc1
> > >
> > >
> > >
> > > These patches are located at branch 17.11 of dpdk-stable repo:
> > >
> > > https://dpdk.org/browse/dpdk-stable/
> > >
> > >
> > >
> > > Thanks.
> > >
> > > Luca Boccassi
> >
> > Hi,
> >
> > We are still waiting to solve 2 issues highlighted during
> > validation,
> > so we need to postpone.
> >
> > New proposed target is February the 14th.
> >
> > Apologies for any issue it might cause.
>
> Hi Luca,
>
> no problem on the delay, on the OVS DPDK side we've validated the
> following to date
>
> OVS testing against head OVS 2.10 and 2.19 with VSPERF
>
> Tested NICs i40e (X710) & ixgbe (82599ES)
>
> P2P
> PVP
> Hotplug
> Multiqueue
> Vhostuserclient reconnect
> Vhost cross-NUMA awareness
> Jumbo frames
> Rate limiting
> QoS policer
>
> If theres a need to re-test for the follow on patches let me know.
>
> Regards
> Ian
Thank you!
--
Kind regards,
Luca Boccassi
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-01-30 10:08 ` Luca Boccassi
2020-01-30 15:01 ` Stokes, Ian
@ 2020-02-15 11:21 ` Luca Boccassi
2020-02-20 10:00 ` Luca Boccassi
1 sibling, 1 reply; 18+ messages in thread
From: Luca Boccassi @ 2020-02-15 11:21 UTC (permalink / raw)
To: dpdk stable, dev
On Thu, 30 Jan 2020 at 10:08, Luca Boccassi <bluca@debian.org> wrote:
>
> On Mon, 2020-01-13 at 12:47 +0000, Luca Boccassi wrote:
> > Hi all,
> >
> > Here is a list of patches targeted for LTS release 17.11.10.
> > This will be the last 17.11 release.
> >
> > The planned date for the final release is the 30th of January.
> >
> > 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=v17.11.10-rc1
> >
> >
> > These patches are located at branch 17.11 of dpdk-stable repo:
> >
> > https://dpdk.org/browse/dpdk-stable/
> >
> >
> > Thanks.
> >
> > Luca Boccassi
>
> Hi,
>
> We are still waiting to solve 2 issues highlighted during validation,
> so we need to postpone.
>
> New proposed target is February the 14th.
>
> Apologies for any issue it might cause.
Hi,
The MLX issue has been fixed, but we are still waiting for a
resolution for the QAT issue.
The release is postponed to the 21st at the earliest.
Apologies for any issue it might cause.
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [dpdk-dev] 17.11.10 (LTS) patches review and test
2020-02-15 11:21 ` Luca Boccassi
@ 2020-02-20 10:00 ` Luca Boccassi
2020-02-20 11:37 ` Trahe, Fiona
0 siblings, 1 reply; 18+ messages in thread
From: Luca Boccassi @ 2020-02-20 10:00 UTC (permalink / raw)
To: dpdk stable, dev
On Sat, 15 Feb 2020 at 11:21, Luca Boccassi <bluca@debian.org> wrote:
>
> On Thu, 30 Jan 2020 at 10:08, Luca Boccassi <bluca@debian.org> wrote:
> >
> > On Mon, 2020-01-13 at 12:47 +0000, Luca Boccassi wrote:
> > > Hi all,
> > >
> > > Here is a list of patches targeted for LTS release 17.11.10.
> > > This will be the last 17.11 release.
> > >
> > > The planned date for the final release is the 30th of January.
> > >
> > > 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=v17.11.10-rc1
> > >
> > >
> > > These patches are located at branch 17.11 of dpdk-stable repo:
> > >
> > > https://dpdk.org/browse/dpdk-stable/
> > >
> > >
> > > Thanks.
> > >
> > > Luca Boccassi
> >
> > Hi,
> >
> > We are still waiting to solve 2 issues highlighted during validation,
> > so we need to postpone.
> >
> > New proposed target is February the 14th.
> >
> > Apologies for any issue it might cause.
>
> Hi,
>
> The MLX issue has been fixed, but we are still waiting for a
> resolution for the QAT issue.
>
> The release is postponed to the 21st at the earliest.
>
> Apologies for any issue it might cause.
Hi,
Postponing again by one week to give more time to look into the QAT issue.
Apologies again for any issue it might cause.
^ permalink raw reply [flat|nested] 18+ messages in thread