From: "Stokes, Ian" <ian.stokes@intel.com>
To: Kevin Traynor <ktraynor@redhat.com>,
"Govindharajan, Hariprasad" <hariprasad.govindharajan@intel.com>,
"stable@dpdk.org" <stable@dpdk.org>,
"Loftus, Ciara" <ciara.loftus@intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>,
Abhishek Marathe <Abhishek.Marathe@microsoft.com>,
Akhil Goyal <akhil.goyal@nxp.com>,
"Ali Alnubani" <alialnu@mellanox.com>,
"Walker, Benjamin" <benjamin.walker@intel.com>,
David Christensen <drc@linux.vnet.ibm.com>,
Hemant Agrawal <hemant.agrawal@nxp.com>,
Jerin Jacob <jerinj@marvell.com>,
"Mcnamara, John" <john.mcnamara@intel.com>,
Ju-Hyoung Lee <juhlee@microsoft.com>,
Luca Boccassi <bluca@debian.org>, Pei Zhang <pezhang@redhat.com>,
"Yu, PingX" <pingx.yu@intel.com>,
"Xu, Qian Q" <qian.q.xu@intel.com>,
Raslan Darawsheh <rasland@mellanox.com>,
"Thomas Monjalon" <thomas@monjalon.net>,
"Peng, Yuan" <yuan.peng@intel.com>,
"Chen, Zhaoyan" <zhaoyan.chen@intel.com>
Subject: Re: [dpdk-dev] 18.11.9 (LTS) patches review and test
Date: Tue, 30 Jun 2020 16:32:44 +0000 [thread overview]
Message-ID: <52156197ADF84D4FB1B836FA75938A8918E9B19F@IRSMSX103.ger.corp.intel.com> (raw)
In-Reply-To: <7b8f79ea-2cea-3222-1a3b-d84dd7247329@redhat.com>
> -----Original Message-----
> From: Kevin Traynor <ktraynor@redhat.com>
> Sent: Tuesday, June 30, 2020 5:21 PM
> To: Govindharajan, Hariprasad <hariprasad.govindharajan@intel.com>;
> stable@dpdk.org; Stokes, Ian <ian.stokes@intel.com>; Loftus, Ciara
> <ciara.loftus@intel.com>
> Cc: dev@dpdk.org; Abhishek Marathe <Abhishek.Marathe@microsoft.com>;
> Akhil Goyal <akhil.goyal@nxp.com>; Ali Alnubani <alialnu@mellanox.com>;
> Walker, Benjamin <benjamin.walker@intel.com>; David Christensen
> <drc@linux.vnet.ibm.com>; Hemant Agrawal <hemant.agrawal@nxp.com>;
> Jerin Jacob <jerinj@marvell.com>; Mcnamara, John
> <john.mcnamara@intel.com>; Ju-Hyoung Lee <juhlee@microsoft.com>; Luca
> Boccassi <bluca@debian.org>; Pei Zhang <pezhang@redhat.com>; Yu, PingX
> <pingx.yu@intel.com>; Xu, Qian Q <qian.q.xu@intel.com>; Raslan Darawsheh
> <rasland@mellanox.com>; Thomas Monjalon <thomas@monjalon.net>; Peng,
> Yuan <yuan.peng@intel.com>; Chen, Zhaoyan <zhaoyan.chen@intel.com>
> Subject: Re: [dpdk-dev] 18.11.9 (LTS) patches review and test
>
> On 30/06/2020 16:38, Govindharajan, Hariprasad wrote:
> >
> >
> >> -----Original Message-----
> >> From: dev <dev-bounces@dpdk.org> On Behalf Of Kevin Traynor
> >> Sent: Friday, June 26, 2020 1:53 PM
> >> To: stable@dpdk.org
> >> Cc: dev@dpdk.org; Abhishek Marathe <Abhishek.Marathe@microsoft.com>;
> >> Akhil Goyal <akhil.goyal@nxp.com>; Ali Alnubani
> >> <alialnu@mellanox.com>; Walker, Benjamin <benjamin.walker@intel.com>;
> >> David Christensen <drc@linux.vnet.ibm.com>; Hemant Agrawal
> >> <hemant.agrawal@nxp.com>; Stokes, Ian <ian.stokes@intel.com>; Jerin
> >> Jacob <jerinj@marvell.com>; Mcnamara, John <john.mcnamara@intel.com>;
> >> Ju-Hyoung Lee <juhlee@microsoft.com>; Kevin Traynor
> >> <ktraynor@redhat.com>; Luca Boccassi <bluca@debian.org>; Pei Zhang
> >> <pezhang@redhat.com>; Yu, PingX <pingx.yu@intel.com>; Xu, Qian Q
> >> <qian.q.xu@intel.com>; Raslan Darawsheh <rasland@mellanox.com>;
> >> Thomas Monjalon <thomas@monjalon.net>; Peng, Yuan
> >> <yuan.peng@intel.com>; Chen, Zhaoyan <zhaoyan.chen@intel.com>
> >> Subject: [dpdk-dev] 18.11.9 (LTS) patches review and test
> >>
> >> Hi all,
> >>
> >> Here is a list of patches targeted for LTS release 18.11.9.
> >>
> >> The planned date for the final release is 3rd July.
> >>
> >> Please help with testing and validation of your use cases and report
> >> any issues/results with reply-all to this mail. For the final release
> >> the fixes and reported validations will be added to the release notes.
> >>
> >> A release candidate tarball can be found at:
> >>
> >> https://dpdk.org/browse/dpdk-stable/tag/?id=v18.11.9-rc2
> >>
> >> These patches are located at branch 18.11 of dpdk-stable repo:
> >> https://dpdk.org/browse/dpdk-stable/
> >>
> >> rc2:
> >> Creating this to include important VF fix.
> >> http://inbox.dpdk.org/dev/20200625035046.19820-1-
> >> haiyue.wang@intel.com/
> >>
> >> New patches in rc2:
> >> 145bb0e780 bus/pci: fix VF memory access
> >> 420a437247 net/tap: fix mbuf double free when writev fails
> >>
> >> Thanks.
> >>
> >> Kevin.
> > [Govindharajan, Hariprasad] Hi Kevin,
> >
>
> Hi G Hariprasad,
>
> Thanks for testing.
>
> > The following performance and functional tests carried out with ixgbe,
> > i40e and vhost devices
> >
> > DPDK 18.11.9 RC2 with OvS 2.11 and OvS 2.12
> >
> > P2p throughput
> > P2p multi queue
> > PVP rss
> > vHost reconnect
> > Flow control
> >
> > Zero copy feature could not be tested.
> > From ovs, zero copy is used only vhost user client. So this particular patch
> 0309429b198800f4d17d578aad650672476ba054 is breaking the zero copy
> support in ovs.
> > so for ovs to move to support 18.11.9, it should probably remove support for
> zero-copy in the ovs code base?
> >
>
> The code says that zero copy is incompatible with vhost client mode.
> It seems that the check for client/server was incorrect, and now it is fixed and
> correctly detects this incompatible combination and returns an error to OVS.
>
> I don't think we can call that breaking. It was previously incompatible also, it just
> didn't return an error as there was a bug.
>
> I don't see an issue with having this patch and the new behaviour.
>
> Ian/Ciara, let me know if I'm missing something.
I think OVS only supports zero copy for vhostuserclient devices. In this case it looks like that’s no longer supported. I guess the real question becomes should the support in OVS be removed now? It is not broken as you said, just no longer supported in the OVS use case.
I don’t think it needs to block the release of this, but it's a follow up for the OVS. I believe you'd see the same behavior for OVS master and 2.13.
BR
Ian
>
> thanks,
> Kevin.
>
> > Regards
> > G Hariprasad
> >>
> >> ---
> >> Adam Dybkowski (2):
> >> app/crypto-perf: fix display of sample test vector
> >> cryptodev: fix SHA-1 digest enum comment
> >>
> >> Ajit Khaparde (3):
> >> net/bnxt: fix FW version query
> >> net/bnxt: fix error log for command timeout
> >> net/bnxt: fix using RSS config struct
> >>
> >> Alex Kiselev (1):
> >> lpm6: fix size of tbl8 group
> >>
> >> Alexander Kozyrev (1):
> >> net/mlx5: fix Tx queue release debug log timing
> >>
> >> Alvin Zhang (1):
> >> net/e1000: fix port hotplug for multi-process
> >>
> >> Amit Gupta (1):
> >> net/octeontx: fix meson build for disabled drivers
> >>
> >> Andrew Rybchenko (1):
> >> net/sfc: fix reported promiscuous/multicast mode
> >>
> >> Apeksha Gupta (1):
> >> bus/fslmc: fix dereferencing null pointer
> >>
> >> Arek Kusztal (1):
> >> crypto/qat: fix cipher descriptor for ZUC and SNOW
> >>
> >> Bing Zhao (1):
> >> mem: fix overflow on allocation
> >>
> >> Bruce Richardson (1):
> >> drivers: fix log type variables for -fno-common
> >>
> >> Cheng Peng (1):
> >> net/iavf: fix stats query error code
> >>
> >> Darek Stojaczyk (1):
> >> pci: accept 32-bit domain numbers
> >>
> >> David Christensen (2):
> >> pci: fix build on ppc
> >> eal/ppc: fix build with gcc 9.3
> >>
> >> David Marchand (4):
> >> mem: mark pages as not accessed when reserving VA
> >> eal: fix typo in endian conversion macros
> >> remove references to private PCI probe function
> >> doc: prefer https when pointing to dpdk.org
> >>
> >> Dekel Peled (2):
> >> net/mlx5: fix mask used for IPv6 item validation
> >> net/mlx5: fix CVLAN tag set in IP item translation
> >>
> >> Eugeny Parshutin (1):
> >> ethdev: fix build when vtune profiling is on
> >>
> >> Fan Zhang (1):
> >> vhost/crypto: add missing user protocol flag
> >>
> >> Ferruh Yigit (3):
> >> net/null: fix secondary burst function selection
> >> net/null: remove redundant check
> >> examples/kni: fix crash during MTU set
> >>
> >> Gaetan Rivet (4):
> >> doc: fix number of failsafe sub-devices
> >> net/ring: fix device pointer on allocation
> >> pci: reject negative values in PCI id
> >> kvargs: fix strcmp helper documentation
> >>
> >> Gavin Hu (1):
> >> net/i40e: relax barrier in Tx for NEON
> >>
> >> Guinan Sun (2):
> >> net/ixgbe: fix statistics in flow control mode
> >> net/ixgbe: check driver type in MACsec API
> >>
> >> Haifeng Lin (1):
> >> eal/arm64: fix precise TSC
> >>
> >> Haiyue Wang (1):
> >> bus/pci: fix VF memory access
> >>
> >> Harry van Haaren (3):
> >> service: fix crash on exit
> >> examples/eventdev: fix crash on exit
> >> test/flow_classify: enable multi-sockets system
> >>
> >> Hemant Agrawal (3):
> >> drivers: add crypto as dependency for event drivers
> >> bus/fslmc: fix size of qman fq descriptor
> >> mempool/dpaa2: install missing header with meson
> >>
> >> Honnappa Nagarahalli (2):
> >> service: fix race condition for MT unsafe service
> >> service: fix identification of service running on other lcore
> >>
> >> Igor Chauskin (2):
> >> net/ena/base: make allocation macros thread-safe
> >> net/ena/base: prevent allocation of zero sized memory
> >>
> >> Igor Romanov (7):
> >> net/sfc: fix initialization error path
> >> net/sfc: fix Rx queue start failure path
> >> net/sfc/base: reduce filter priorities to implemented only
> >> net/sfc/base: reject automatic filter creation by users
> >> net/sfc/base: refactor filter lookup loop in EF10
> >> net/sfc/base: handle manual and auto filter clashes in EF10
> >> net/sfc/base: fix manual filter delete in EF10
> >>
> >> Itsuro Oda (2):
> >> net/vhost: fix potential memory leak on close
> >> vhost: make IOTLB cache name unique among processes
> >>
> >> Ivan Dyukov (3):
> >> net/virtio-user: fix devargs parsing
> >> app: remove extra new line after link duplex
> >> examples: remove extra new line after link duplex
> >>
> >> Jasvinder Singh (3):
> >> net/softnic: fix memory leak for thread
> >> net/softnic: fix resource leak for pipeline
> >> examples/ip_pipeline: remove check of null response
> >>
> >> Jeff Guo (2):
> >> net/i40e: fix setting L2TAG
> >> net/iavf: fix setting L2TAG
> >>
> >> Jim Harris (1):
> >> contigmem: cleanup properly when load fails
> >>
> >> Junyu Jiang (2):
> >> examples/vmdq: fix output of pools/queues
> >> examples/vmdq: fix RSS configuration
> >>
> >> Kalesh AP (2):
> >> net/bnxt: fix VLAN add when port is stopped
> >> app/testpmd: fix memory failure handling for i40e DDP
> >>
> >> Kevin Traynor (14):
> >> kni: fix ethtool build with kernel 5.6
> >> net/avp: fix gcc 10 maybe-uninitialized warning
> >> eal/x86: ignore gcc 10 stringop-overflow warnings
> >> crypto/kasumi: fix extern declaration
> >> build: disable gcc 10 zero-length-bounds warning
> >> kni: fix ethtool dev_open build error
> >> kni: fix ethtool pointer type build error
> >> drivers/crypto: disable gcc 10 fno-common errors
> >> kni: fix gcc 10 ethtool build error
> >> kni: fix ethtool maybe-uninitialized warnings
> >> crypto/octeontx: fix gcc 10 -fno-common build errors
> >> app/testpmd: disable gcc 10 -fno-common build errors
> >> examples/vm_power: fix build because of missing include
> >> version: 18.11.9-rc1
> >>
> >> Konstantin Ananyev (1):
> >> security: fix crash at accessing non-implemented ops
> >>
> >> Li Feng (1):
> >> mem: mark pages as not accessed when freeing memory
> >>
> >> Louise Kilheeney (1):
> >> examples/l2fwd-keepalive: fix mbuf pool size
> >>
> >> Luca Boccassi (2):
> >> fix various typos found by Lintian
> >> usertools: check for pci.ids in /usr/share/misc
> >>
> >> Lukasz Wojciechowski (5):
> >> drivers/crypto: fix log type variables for -fno-common
> >> security: fix verification of parameters
> >> security: fix return types in documentation
> >> security: fix session counter
> >> test: remove redundant macro
> >>
> >> Mattias Rönnblom (3):
> >> event/dsw: remove redundant control ring poll
> >> event/dsw: remove unnecessary read barrier
> >> event/dsw: avoid reusing previously recorded events
> >>
> >> Michael Baum (1):
> >> net/mlx4: fix drop queue error handling
> >>
> >> Michael Haeuptle (1):
> >> vfio: fix race condition with sysfs
> >>
> >> Michal Krawczyk (5):
> >> net/ena/base: fix testing for supported hash function
> >> net/ena/base: fix documentation of functions
> >> net/ena/base: fix indentation in CQ polling
> >> net/ena/base: fix indentation of multiple defines
> >> net/ena: set IO ring size to valid value
> >>
> >> Muhammad Bilal (2):
> >> fix same typo in multiple places
> >> doc: fix typo in contributors guide
> >>
> >> Nicolas Chautru (1):
> >> bbdev: fix doxygen comments
> >>
> >> Nithin Dabilpuram (1):
> >> devtools: fix symbol map change check
> >>
> >> Olivier Matz (3):
> >> test/kvargs: fix to consider empty elements as valid
> >> test/kvargs: fix invalid cases check
> >> kvargs: fix invalid token parsing on FreeBSD
> >>
> >> Pablo de Lara (3):
> >> test/crypto: fix flag check
> >> crypto/openssl: fix out-of-place encryption
> >> doc: add NASM installation steps
> >>
> >> Pavan Nikhilesh (2):
> >> eventdev: fix probe and remove for secondary process
> >> app/eventdev: check Tx adapter service ID
> >>
> >> Phil Yang (1):
> >> service: remove rte prefix from static functions
> >>
> >> Qiming Yang (3):
> >> net/i40e: fix X722 performance
> >> doc: fix multicast filter feature announcement
> >> net/i40e: fix queue related exception handling
> >>
> >> Rasesh Mody (3):
> >> net/qede: fix link state configuration
> >> net/qede: fix port reconfiguration
> >> examples/kni: fix MTU change to setup Tx queue
> >>
> >> Raslan Darawsheh (3):
> >> net/mlx5: fix validation of VXLAN/VXLAN-GPE specs
> >> app/testpmd: add parsing for QinQ VLAN headers
> >> net/mlx5: fix matching for UDP tunnels with Verbs
> >>
> >> Rohit Raj (1):
> >> net/dpaa2: fix 10G port negotiation
> >>
> >> Roland Qi (1):
> >> vhost: fix peer close check
> >>
> >> Ruifeng Wang (1):
> >> test: skip some subtests in no-huge mode
> >>
> >> Sarosh Arif (1):
> >> doc: fix typo in contributors guide
> >>
> >> Somnath Kotur (1):
> >> bus/pci: fix devargs on probing again
> >>
> >> Stephen Hemminger (21):
> >> ethdev: fix spelling
> >> net/mvneta: do not use PMD log type
> >> net/virtio: do not use PMD log type
> >> net/tap: do not use PMD log type
> >> net/bnxt: do not use PMD log type
> >> net/dpaa: use dynamic log type
> >> net/thunderx: use dynamic log type
> >> net/netvsc: propagate descriptor limits from VF
> >> net/netvsc: handle Rx packets during multi-channel setup
> >> net/netvsc: split send buffers from Tx descriptors
> >> net/netvsc: fix memory free on device close
> >> net/netvsc: remove process event optimization
> >> net/netvsc: handle Tx completions based on burst size
> >> net/netvsc: avoid possible live lock
> >> lpm6: fix comments spelling
> >> eal: fix comments spelling
> >> net/netvsc: fix comment spelling
> >> bus/vmbus: fix comment spelling
> >> net/tap: fix crash in flow destroy
> >> eal: fix C++17 compilation
> >> net/vmxnet3: handle bad host framing
> >>
> >> Sunil Kumar Kori (1):
> >> eal: fix log message print for regex
> >>
> >> Thomas Monjalon (11):
> >> doc: fix sphinx compatibility
> >> log: fix level picked with globbing on type register
> >> doc: fix matrix CSS for recent sphinx
> >> net/mlx4: fix build with -fno-common
> >> app: fix usage help of options separated by dashes
> >> net/mvpp2: fix build with gcc 10
> >> examples/vm_power: drop Unix path limit redefinition
> >> doc: fix build with doxygen 1.8.18
> >> doc: fix API index
> >> mlx5: fix build with -fno-common
> >> examples/vm_power: fix build with -fno-common
> >>
> >> Timothy Redaelli (3):
> >> app/pipeline: fix build with gcc 10
> >> examples/eventdev: fix build with gcc 10
> >> crypto/octeontx: fix build with gcc 10
> >>
> >> Ting Xu (1):
> >> app/testpmd: fix DCB set
> >>
> >> Vijaya Mohan Guvva (1):
> >> bus/pci: fix UIO resource access from secondary process
> >>
> >> Wei Hu (Xavier) (2):
> >> vfio: fix use after free with multiprocess
> >> app/testpmd: fix statistics after reset
> >>
> >> Wei Zhao (4):
> >> net/i40e: fix flow director for ARP packets
> >> doc: add i40e limitation for flow director
> >> net/i40e: fix flush of flow director filter
> >> net/i40e: fix wild pointer
> >>
> >> Xiaolong Ye (3):
> >> net/virtio: fix outdated comment
> >> vhost: remove unused variable
> >> doc: fix log level example in Linux guide
> >>
> >> Xiaoyu Min (1):
> >> net/mlx5: fix RSS enablement
> >>
> >> Xiaoyun Li (2):
> >> net/ixgbe/base: update copyright
> >> net/i40e/base: update copyright
> >>
> >> Xuan Ding (2):
> >> vhost: prevent zero-copy with incompatible client mode
> >> vhost: fix zero-copy server mode
> >>
> >> Yunjian Wang (14):
> >> kvargs: fix buffer overflow when parsing list
> >> net/tap: remove unused assert
> >> net/tap: fix mbuf and mem leak during queue release
> >> net/tap: fix check for mbuf number of segment
> >> net/tap: fix file close on remove
> >> net/tap: fix fd leak on creation failure
> >> net/tap: fix unexpected link handler
> >> net/tap: fix queues fd check before close
> >> net/octeontx: fix dangling pointer on init failure
> >> crypto/ccp: fix fd leak on probe failure
> >> net/failsafe: fix fd leak
> >> crypto/caam_jr: fix check of file descriptors
> >> crypto/caam_jr: fix IRQ functions return type
> >> net/tap: fix mbuf double free when writev fails
> >>
> >> Yuri Chipchev (1):
> >> event/dsw: fix enqueue burst return value
> >>
> >> Zhihong Peng (1):
> >> net/ixgbe: fix link status synchronization on BSD
> >
next prev parent reply other threads:[~2020-06-30 16:32 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-26 12:53 Kevin Traynor
2020-06-28 3:51 ` Chen, BoX C
2020-06-29 9:03 ` Kevin Traynor
2020-06-28 5:42 ` Pei Zhang
2020-06-29 9:04 ` Kevin Traynor
2020-06-30 8:54 ` Ali Alnubani
2020-06-30 9:52 ` Kevin Traynor
2020-06-30 10:02 ` Ali Alnubani
2020-07-10 8:53 ` Kevin Traynor
2020-06-30 15:38 ` Govindharajan, Hariprasad
2020-06-30 16:21 ` Kevin Traynor
2020-06-30 16:32 ` Stokes, Ian [this message]
2020-06-30 17:03 ` Kevin Traynor
2020-07-06 15:57 ` Kevin Traynor
2020-07-03 17:26 ` Kevin Traynor
2020-07-10 17:28 ` [dpdk-dev] [EXTERNAL] " Abhishek Marathe
2020-07-10 17:51 ` Kevin Traynor
-- strict thread matches above, loose matches on Subject: below --
2020-06-20 12:52 [dpdk-dev] " Kevin Traynor
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52156197ADF84D4FB1B836FA75938A8918E9B19F@IRSMSX103.ger.corp.intel.com \
--to=ian.stokes@intel.com \
--cc=Abhishek.Marathe@microsoft.com \
--cc=akhil.goyal@nxp.com \
--cc=alialnu@mellanox.com \
--cc=benjamin.walker@intel.com \
--cc=bluca@debian.org \
--cc=ciara.loftus@intel.com \
--cc=dev@dpdk.org \
--cc=drc@linux.vnet.ibm.com \
--cc=hariprasad.govindharajan@intel.com \
--cc=hemant.agrawal@nxp.com \
--cc=jerinj@marvell.com \
--cc=john.mcnamara@intel.com \
--cc=juhlee@microsoft.com \
--cc=ktraynor@redhat.com \
--cc=pezhang@redhat.com \
--cc=pingx.yu@intel.com \
--cc=qian.q.xu@intel.com \
--cc=rasland@mellanox.com \
--cc=stable@dpdk.org \
--cc=thomas@monjalon.net \
--cc=yuan.peng@intel.com \
--cc=zhaoyan.chen@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).