Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Szczerbik, PrzemyslawX]

> -Original Message-
> From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-
> boun...@openvswitch.org] On Behalf Of Timothy M. Redaelli
> Sent: Tuesday, August 1, 2017 11:37 AM
> To: Weglicki, MichalX <michalx.wegli...@intel.com>; d...@openvswitch.org
> Subject: Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
> DPDK 17.05.1.
> 
> On 07/25/2017 02:34 PM, Michal Weglicki wrote:
> > Upgrading to DPDK 17.05.1 stable release adds new
> > significant features relevant to OVS, including,
> > but not limited to:
> > - tun/tap PMD,
> > - VFIO hotplug support,
> > - Generic flow API.
> >
> > Following changes are applied:
> > - netdev-dpdk: Changes required by DPDK API modifications.
> > - doc: Because of DPDK API changes, backward compatibility
> >with previous DPDK releases will be broken, thus all
> >relevant documentation entries are updated.
> > - .travis: DPDK version change from 16.11.1 to 17.05.1.
> > - rhel/openvswitch-fedora.spec.in: DPDK version change
> >from 16.11 to 17.05.1
> >
> > v1->v2: Patch rebase.
> > v2->v3: Fixed wrong formating after v2 patch rebase.
> > v3->v4: Minor documentation changes.
> >
> > Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
> > Reviewed-by: Aaron Conole <acon...@redhat.com>
> > ---
> >   .travis/linux-build.sh   |   2 +-
> >   Documentation/faq/releases.rst   |   1 +
> >   Documentation/howto/dpdk.rst |   6 +-
> >   Documentation/intro/install/dpdk.rst |  14 +--
> >   Documentation/topics/dpdk/vhost-user.rst |  12 +--
> >   NEWS |   1 +
> >   lib/netdev-dpdk.c| 144 
> > +++
> >   rhel/openvswitch-fedora.spec.in  |   2 +-
> >   tests/dpdk/ring_client.c |   6 +-
> >   9 files changed, 114 insertions(+), 74 deletions(-)
> >
> [...]
> > diff --git a/rhel/openvswitch-fedora.spec.in 
> > b/rhel/openvswitch-fedora.spec.in
> > index 3a045d3..2bb7102 100644
> > --- a/rhel/openvswitch-fedora.spec.in
> > +++ b/rhel/openvswitch-fedora.spec.in
> > @@ -84,7 +84,7 @@ BuildRequires: libcap-ng libcap-ng-devel
> >   %endif
> >   %if %{with dpdk}
> >   BuildRequires: libpcap-devel numactl-devel
> > -BuildRequires: dpdk-devel >= 16.11
> > +BuildRequires: dpdk-devel >= 17.05.1
> >   Provides: %{name}-dpdk = %{version}-%{release}
> >   %endif
> >
> 
> 
> Hi,
> on Fedora there is no dpdk-devel 17.05.1 package, the last rawhide
> version is dpdk-devel 17.05 (and usually Fedora doesn't package minor
> releases of dpdk).
> 
> I tried to build openvswitch with the dpdk 17.05 package from rawhide.
> It build and it works (tested with dpdkvhostuserclient) so, unless you
> need a specific fix from 17.05.1, I suggest to change the BuildRequires
> to 17.05 instead of 17.05.1 or the package cannot be built on Fedora,
> unless you are using an additional repository (copr).
Hi,

If I'm not mistaken we moved to 17.05.1 version, because it includes fix for 
'vhost: fix crash on NUMA', which was causing ovs-vswitchd to terminate.
More details here: http://dpdk.org/dev/patchwork/patch/25012/

Regards,
Przemek
--
Intel Research and Development Ireland Limited
Registered in Ireland
Registered Office: Collinstown Industrial Park, Leixlip, County Kildare
Registered Number: 308263


This e-mail and any attachments may contain confidential material for the sole
use of the intended recipient(s). Any review or distribution by others is
strictly prohibited. If you are not the intended recipient, please contact the
sender and delete all copies.

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Timothy M. Redaelli]

On 07/25/2017 02:34 PM, Michal Weglicki wrote:

Upgrading to DPDK 17.05.1 stable release adds new
significant features relevant to OVS, including,
but not limited to:
- tun/tap PMD,
- VFIO hotplug support,
- Generic flow API.

Following changes are applied:
- netdev-dpdk: Changes required by DPDK API modifications.
- doc: Because of DPDK API changes, backward compatibility
   with previous DPDK releases will be broken, thus all
   relevant documentation entries are updated.
- .travis: DPDK version change from 16.11.1 to 17.05.1.
- rhel/openvswitch-fedora.spec.in: DPDK version change
   from 16.11 to 17.05.1

v1->v2: Patch rebase.
v2->v3: Fixed wrong formating after v2 patch rebase.
v3->v4: Minor documentation changes.

Signed-off-by: Michal Weglicki 
Reviewed-by: Aaron Conole 
---
  .travis/linux-build.sh   |   2 +-
  Documentation/faq/releases.rst   |   1 +
  Documentation/howto/dpdk.rst |   6 +-
  Documentation/intro/install/dpdk.rst |  14 +--
  Documentation/topics/dpdk/vhost-user.rst |  12 +--
  NEWS |   1 +
  lib/netdev-dpdk.c| 144 +++
  rhel/openvswitch-fedora.spec.in  |   2 +-
  tests/dpdk/ring_client.c |   6 +-
  9 files changed, 114 insertions(+), 74 deletions(-)


[...]

diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
index 3a045d3..2bb7102 100644
--- a/rhel/openvswitch-fedora.spec.in
+++ b/rhel/openvswitch-fedora.spec.in
@@ -84,7 +84,7 @@ BuildRequires: libcap-ng libcap-ng-devel
  %endif
  %if %{with dpdk}
  BuildRequires: libpcap-devel numactl-devel
-BuildRequires: dpdk-devel >= 16.11
+BuildRequires: dpdk-devel >= 17.05.1
  Provides: %{name}-dpdk = %{version}-%{release}
  %endif
 



Hi,
on Fedora there is no dpdk-devel 17.05.1 package, the last rawhide 
version is dpdk-devel 17.05 (and usually Fedora doesn't package minor 
releases of dpdk).


I tried to build openvswitch with the dpdk 17.05 package from rawhide.
It build and it works (tested with dpdkvhostuserclient) so, unless you 
need a specific fix from 17.05.1, I suggest to change the BuildRequires 
to 17.05 instead of 17.05.1 or the package cannot be built on Fedora, 
unless you are using an additional repository (copr).

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Darrell Ball]

-Original Message-
From: Aaron Conole <acon...@redhat.com>
Date: Sunday, July 30, 2017 at 8:22 AM
To: Darrell Ball <db...@vmware.com>
Cc: Michal Weglicki <michalx.wegli...@intel.com>, "d...@openvswitch.org" 
<d...@openvswitch.org>
Subject: Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for 
DPDK 17.05.1.

Darrell Ball <db...@vmware.com> writes:

> -Original Message-
> From: <ovs-dev-boun...@openvswitch.org> on behalf of Michal Weglicki 
<michalx.wegli...@intel.com>
> Date: Tuesday, July 25, 2017 at 5:34 AM
> To: "d...@openvswitch.org" <d...@openvswitch.org>
> Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support 
forDPDK 17.05.1.
>
> Upgrading to DPDK 17.05.1 stable release adds new
> significant features relevant to OVS, including,
> but not limited to:
> - tun/tap PMD,
> - VFIO hotplug support,
> - Generic flow API.
> 
> Following changes are applied:
> - netdev-dpdk: Changes required by DPDK API modifications.
> - doc: Because of DPDK API changes, backward compatibility
>   with previous DPDK releases will be broken, thus all
>   relevant documentation entries are updated.
> - .travis: DPDK version change from 16.11.1 to 17.05.1.
> - rhel/openvswitch-fedora.spec.in: DPDK version change
>   from 16.11 to 17.05.1
> 
> v1->v2: Patch rebase.
> v2->v3: Fixed wrong formating after v2 patch rebase.
> v3->v4: Minor documentation changes.
> 
> Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
> Reviewed-by: Aaron Conole <acon...@redhat.com>
> ---
>  .travis/linux-build.sh   |   2 +-
>  Documentation/faq/releases.rst   |   1 +
>  Documentation/howto/dpdk.rst |   6 +-
>  Documentation/intro/install/dpdk.rst |  14 +--
>  Documentation/topics/dpdk/vhost-user.rst |  12 +--
>  NEWS |   1 +
>  lib/netdev-dpdk.c| 144 
+++
>  rhel/openvswitch-fedora.spec.in  |   2 +-
>  tests/dpdk/ring_client.c |   6 +-
>  9 files changed, 114 insertions(+), 74 deletions(-)
> 
> diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
> index f66b534..efccdf1 100755
> --- a/.travis/linux-build.sh
> +++ b/.travis/linux-build.sh
> @@ -80,7 +80,7 @@ fi
>  
>  if [ "$DPDK" ]; then
>  if [ -z "$DPDK_VER" ]; then
> -DPDK_VER="16.11.2"
> +DPDK_VER="17.05.1"
>  fi
>  install_dpdk $DPDK_VER
>  if [ "$CC" = "clang" ]; then
> diff --git a/Documentation/faq/releases.rst 
b/Documentation/faq/releases.rst
> index 707834b..2ecc24c 100644
> --- a/Documentation/faq/releases.rst
> +++ b/Documentation/faq/releases.rst
> @@ -161,6 +161,7 @@ Q: What DPDK version does each Open vSwitch 
release work with?
>  2.5.x2.2
>  2.6.x16.07.2
>  2.7.x16.11.2
> +2.8.x17.05.1
>   ===
>  
>  Q: I get an error like this when I configure Open vSwitch:
> diff --git a/Documentation/howto/dpdk.rst 
b/Documentation/howto/dpdk.rst
> index af01d3e..1756c8c 100644
> --- a/Documentation/howto/dpdk.rst
> +++ b/Documentation/howto/dpdk.rst
> @@ -528,15 +528,15 @@ described in :ref:`dpdk-testpmd`. Once 
compiled, run the application::
>  
>  When you finish testing, bind the vNICs back to kernel::
>  
> -$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:03.0
> -$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:04.0
> +$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci 
:00:03.0
> +$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci 
:00:04.0
>  
>  .. note::
>  
>Valid PCI IDs must be passed in above example. The PCI IDs can be 
retrieved
>like so::
>  
> -  $ $DPDK_DIR/tools/dpdk-devbind.py --status
> +  $ $DPDK_DIR/usertools/dpdk-devbind.py --status
>  
>  More information on the dpdk

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Aaron Conole]

Darrell Ball  writes:

> -Original Message-
> From:  on behalf of Michal Weglicki 
> 
> Date: Tuesday, July 25, 2017 at 5:34 AM
> To: "d...@openvswitch.org" 
> Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
> DPDK 17.05.1.
>
> Upgrading to DPDK 17.05.1 stable release adds new
> significant features relevant to OVS, including,
> but not limited to:
> - tun/tap PMD,
> - VFIO hotplug support,
> - Generic flow API.
> 
> Following changes are applied:
> - netdev-dpdk: Changes required by DPDK API modifications.
> - doc: Because of DPDK API changes, backward compatibility
>   with previous DPDK releases will be broken, thus all
>   relevant documentation entries are updated.
> - .travis: DPDK version change from 16.11.1 to 17.05.1.
> - rhel/openvswitch-fedora.spec.in: DPDK version change
>   from 16.11 to 17.05.1
> 
> v1->v2: Patch rebase.
> v2->v3: Fixed wrong formating after v2 patch rebase.
> v3->v4: Minor documentation changes.
> 
> Signed-off-by: Michal Weglicki 
> Reviewed-by: Aaron Conole 
> ---
>  .travis/linux-build.sh   |   2 +-
>  Documentation/faq/releases.rst   |   1 +
>  Documentation/howto/dpdk.rst |   6 +-
>  Documentation/intro/install/dpdk.rst |  14 +--
>  Documentation/topics/dpdk/vhost-user.rst |  12 +--
>  NEWS |   1 +
>  lib/netdev-dpdk.c| 144 
> +++
>  rhel/openvswitch-fedora.spec.in  |   2 +-
>  tests/dpdk/ring_client.c |   6 +-
>  9 files changed, 114 insertions(+), 74 deletions(-)
> 
> diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
> index f66b534..efccdf1 100755
> --- a/.travis/linux-build.sh
> +++ b/.travis/linux-build.sh
> @@ -80,7 +80,7 @@ fi
>  
>  if [ "$DPDK" ]; then
>  if [ -z "$DPDK_VER" ]; then
> -DPDK_VER="16.11.2"
> +DPDK_VER="17.05.1"
>  fi
>  install_dpdk $DPDK_VER
>  if [ "$CC" = "clang" ]; then
> diff --git a/Documentation/faq/releases.rst 
> b/Documentation/faq/releases.rst
> index 707834b..2ecc24c 100644
> --- a/Documentation/faq/releases.rst
> +++ b/Documentation/faq/releases.rst
> @@ -161,6 +161,7 @@ Q: What DPDK version does each Open vSwitch release 
> work with?
>  2.5.x2.2
>  2.6.x16.07.2
>  2.7.x16.11.2
> +2.8.x17.05.1
>   ===
>  
>  Q: I get an error like this when I configure Open vSwitch:
> diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst
> index af01d3e..1756c8c 100644
> --- a/Documentation/howto/dpdk.rst
> +++ b/Documentation/howto/dpdk.rst
> @@ -528,15 +528,15 @@ described in :ref:`dpdk-testpmd`. Once compiled, 
> run the application::
>  
>  When you finish testing, bind the vNICs back to kernel::
>  
> -$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:03.0
> -$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:04.0
> +$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:03.0
> +$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:04.0
>  
>  .. note::
>  
>Valid PCI IDs must be passed in above example. The PCI IDs can be 
> retrieved
>like so::
>  
> -  $ $DPDK_DIR/tools/dpdk-devbind.py --status
> +  $ $DPDK_DIR/usertools/dpdk-devbind.py --status
>  
>  More information on the dpdkvhostuser ports can be found in
>  :doc:`/topics/dpdk/vhost-user`.
>
> [Darrell]
> While dealing with something else, I noticed that this file has an outdated 
> reference,
> as of OVS 2.8/DPDK 17.05.1, to VFIO not supporting hotplug. Specifically, the 
> text 
>
> “This feature is not supported with VFIO” under “Port Hotplug”
>
> It will be removed unless someone objects.
>
> //

That makes sense.

OTOH, how is it going to work as far as applying / pulling / etc. goes?

I know that the DPDK related work is planning on going with a
pull-request model.  Does that mean this patch is applied to your branch
and you will submit a pull request with it at some point?  How about
work that builds on this patch?  As an example,
  https://mail.openvswitch.org/pipermail/ovs-dev/2017-July/335250.html

I'd like to submit my latest work for this series, but not sure where or
how.

-Aaron
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Darrell Ball]

-Original Message-
From:  on behalf of Michal Weglicki 

Date: Tuesday, July 25, 2017 at 5:34 AM
To: "d...@openvswitch.org" 
Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for  
DPDK 17.05.1.

Upgrading to DPDK 17.05.1 stable release adds new
significant features relevant to OVS, including,
but not limited to:
- tun/tap PMD,
- VFIO hotplug support,
- Generic flow API.

Following changes are applied:
- netdev-dpdk: Changes required by DPDK API modifications.
- doc: Because of DPDK API changes, backward compatibility
  with previous DPDK releases will be broken, thus all
  relevant documentation entries are updated.
- .travis: DPDK version change from 16.11.1 to 17.05.1.
- rhel/openvswitch-fedora.spec.in: DPDK version change
  from 16.11 to 17.05.1

v1->v2: Patch rebase.
v2->v3: Fixed wrong formating after v2 patch rebase.
v3->v4: Minor documentation changes.

Signed-off-by: Michal Weglicki 
Reviewed-by: Aaron Conole 
---
 .travis/linux-build.sh   |   2 +-
 Documentation/faq/releases.rst   |   1 +
 Documentation/howto/dpdk.rst |   6 +-
 Documentation/intro/install/dpdk.rst |  14 +--
 Documentation/topics/dpdk/vhost-user.rst |  12 +--
 NEWS |   1 +
 lib/netdev-dpdk.c| 144 
+++
 rhel/openvswitch-fedora.spec.in  |   2 +-
 tests/dpdk/ring_client.c |   6 +-
 9 files changed, 114 insertions(+), 74 deletions(-)

diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
index f66b534..efccdf1 100755
--- a/.travis/linux-build.sh
+++ b/.travis/linux-build.sh
@@ -80,7 +80,7 @@ fi
 
 if [ "$DPDK" ]; then
 if [ -z "$DPDK_VER" ]; then
-DPDK_VER="16.11.2"
+DPDK_VER="17.05.1"
 fi
 install_dpdk $DPDK_VER
 if [ "$CC" = "clang" ]; then
diff --git a/Documentation/faq/releases.rst b/Documentation/faq/releases.rst
index 707834b..2ecc24c 100644
--- a/Documentation/faq/releases.rst
+++ b/Documentation/faq/releases.rst
@@ -161,6 +161,7 @@ Q: What DPDK version does each Open vSwitch release 
work with?
 2.5.x2.2
 2.6.x16.07.2
 2.7.x16.11.2
+2.8.x17.05.1
  ===
 
 Q: I get an error like this when I configure Open vSwitch:
diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst
index af01d3e..1756c8c 100644
--- a/Documentation/howto/dpdk.rst
+++ b/Documentation/howto/dpdk.rst
@@ -528,15 +528,15 @@ described in :ref:`dpdk-testpmd`. Once compiled, run 
the application::
 
 When you finish testing, bind the vNICs back to kernel::
 
-$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:03.0
-$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:04.0
+$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:03.0
+$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:04.0
 
 .. note::
 
   Valid PCI IDs must be passed in above example. The PCI IDs can be 
retrieved
   like so::
 
-  $ $DPDK_DIR/tools/dpdk-devbind.py --status
+  $ $DPDK_DIR/usertools/dpdk-devbind.py --status
 
 More information on the dpdkvhostuser ports can be found in
 :doc:`/topics/dpdk/vhost-user`.

[Darrell]
While dealing with something else, I noticed that this file has an outdated 
reference,
as of OVS 2.8/DPDK 17.05.1, to VFIO not supporting hotplug. Specifically, the 
text 

“This feature is not supported with VFIO” under “Port Hotplug”

It will be removed unless someone objects.

//



diff --git a/Documentation/intro/install/dpdk.rst 
b/Documentation/intro/install/dpdk.rst
index a05aa1a..20baa63 100644
--- a/Documentation/intro/install/dpdk.rst
+++ b/Documentation/intro/install/dpdk.rst
@@ -40,7 +40,7 @@ Build requirements
 In addition to the requirements described in :doc:`general`, building Open
 vSwitch with DPDK will require the following:
 
-- DPDK 16.11
+- DPDK 17.05.1
 
 - A `DPDK supported NIC`_
 
@@ -69,9 +69,9 @@ Install DPDK
 #. Download the `DPDK sources`_, extract the file and set ``DPDK_DIR``::
 
$ cd /usr/src/
-   $ wget 
https://urldefense.proofpoint.com/v2/url?u=http-3A__fast.dpdk.org_rel_dpdk-2D16.11.2.tar.xz=DwICAg=uilaK90D4TOVoH58JNXRgQ=BVhFA09CGX7JQ5Ih-uZnsw=1y2ctE1whdC2J1_P88hUEbLvRvKEC-C8osVeldZgdj4=MP_pr-aH-lrKfYN45gHDyPQ6aLBg5TGAW1UPDDHkAN4=
 
-   $ tar xf dpdk-16.11.2.tar.xz
-   $ export 

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Aaron Conole]

Darrell Ball <db...@vmware.com> writes:

> Hi Aaron
>
> You had mentioned that you would be testing with the patch as well.
> Were you able to try it ?

I did some limited testing, with single queue and VM-to-VM traffic only
(no dpdk physical ports).  That seemed to work, but I trust Ian's and
Kevin's testing more.

Acked-by: Aaron Conole <acon...@redhat.com>

> Thanks Darrell
>
>
>
> -Original Message-
> From: <ovs-dev-boun...@openvswitch.org> on behalf of Kevin Traynor
> <ktray...@redhat.com>
> Organization: Red Hat
> Date: Tuesday, July 25, 2017 at 9:35 AM
> To: Michal Weglicki <michalx.wegli...@intel.com>,
> "d...@openvswitch.org" <d...@openvswitch.org>
> Subject: Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add
> support for DPDK 17.05.1.
>
> On 07/25/2017 01:34 PM, Michal Weglicki wrote:
> > Upgrading to DPDK 17.05.1 stable release adds new
> > significant features relevant to OVS, including,
> > but not limited to:
> > - tun/tap PMD,
> > - VFIO hotplug support,
> > - Generic flow API.
> > 
> > Following changes are applied:
> > - netdev-dpdk: Changes required by DPDK API modifications.
> > - doc: Because of DPDK API changes, backward compatibility
> >   with previous DPDK releases will be broken, thus all
> >   relevant documentation entries are updated.
> > - .travis: DPDK version change from 16.11.1 to 17.05.1.
> > - rhel/openvswitch-fedora.spec.in: DPDK version change
> >   from 16.11 to 17.05.1
> > 
> > v1->v2: Patch rebase.
> > v2->v3: Fixed wrong formating after v2 patch rebase.
> > v3->v4: Minor documentation changes.
> > 
> > Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
> > Reviewed-by: Aaron Conole <acon...@redhat.com>
> > ---
> 
> Thanks Michal. I tried a few things out and working ok for me.
> 
> Acked-by: Kevin Traynor <ktray...@redhat.com>
> ___
> dev mailing list
> d...@openvswitch.org
> 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev=DwICAg=uilaK90D4TOVoH58JNXRgQ=BVhFA09CGX7JQ5Ih-uZnsw=ekKh60DEjO7sJHJqmg8W8agp2GxZlvAzz5bWGN3ux4k=AZOPEg-J3gwSlEr1nLn9aCrmRt_LBA0YK88h9QS1_-g=
>  
> 
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Stokes, Ian]

> -Original Message-
> From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-
> boun...@openvswitch.org] On Behalf Of Kavanagh, Mark B
> Sent: Wednesday, July 26, 2017 4:03 PM
> To: Aaron Conole <acon...@redhat.com>
> Cc: d...@openvswitch.org
> Subject: Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support
> for DPDK 17.05.1.
> 
> >From: Aaron Conole [mailto:acon...@redhat.com]
> >Sent: Wednesday, July 26, 2017 3:25 PM
> >To: Kavanagh, Mark B <mark.b.kavan...@intel.com>
> >Cc: Weglicki, MichalX <michalx.wegli...@intel.com>; d...@openvswitch.org
> >Subject: Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add
> >support for DPDK 17.05.1.
> >
> >"Kavanagh, Mark B" <mark.b.kavan...@intel.com> writes:
> >
> >>>From: Weglicki, MichalX
> >>>Sent: Wednesday, July 26, 2017 10:52 AM
> >>>To: Kavanagh, Mark B <mark.b.kavan...@intel.com>; d...@openvswitch.org
> >>>Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add
> >>>support
> >for
> >>>DPDK 17.05.1.
> >>>
> >>>Hey Mark,
> >>>
> >>>Sorry, I had some problems with rebase, and somehow I just simply
> >>>forgot t
> >to
> >>>do so. Do you want me to re-apply this patch as v5?
> >>
> >> No need Michal - hopefully the maintainer will just roll it in when
> >> applying to master.
> >
> >If you reply with your Acked-by tag, the patchwork instance will
> >re-apply it and the maintainer will not have to remember.
> 
> Thanks for the tip Aaron - I wasn't aware of that!
> 
> Acked-by: Mark Kavanagh <mark.b.kavan...@intel.com>
> 
> -Mark

Adding tested by tag.

Tested-by: Ian Stokes <ian.sto...@intel.com>
> 
> >
> >> Thanks,
> >> Mark
> >>
> >>>
> >>>> -Original Message-
> >>>> From: Kavanagh, Mark B
> >>>> Sent: Wednesday, July 26, 2017 10:04 AM
> >>>> To: Weglicki, MichalX <michalx.wegli...@intel.com>;
> >>>> d...@openvswitch.org
> >>>> Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add
> >>>> support
> >>>for DPDK 17.05.1.
> >>>>
> >>>>
> >>>>
> >>>> >-Original Message-
> >>>> >From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-
> >>>boun...@openvswitch.org]
> >>>> >On Behalf Of Michal Weglicki
> >>>> >Sent: Tuesday, July 25, 2017 1:35 PM
> >>>> >To: d...@openvswitch.org
> >>>> >Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add
> >>>> >support
> >for
> >>>> >DPDK 17.05.1.
> >>>> >
> >>>> >Upgrading to DPDK 17.05.1 stable release adds new significant
> >>>> >features relevant to OVS, including, but not limited to:
> >>>> >- tun/tap PMD,
> >>>> >- VFIO hotplug support,
> >>>> >- Generic flow API.
> >>>> >
> >>>> >Following changes are applied:
> >>>> >- netdev-dpdk: Changes required by DPDK API modifications.
> >>>> >- doc: Because of DPDK API changes, backward compatibility
> >>>> >  with previous DPDK releases will be broken, thus all
> >>>> >  relevant documentation entries are updated.
> >>>> >- .travis: DPDK version change from 16.11.1 to 17.05.1.
> >>>> >- rhel/openvswitch-fedora.spec.in: DPDK version change
> >>>> >  from 16.11 to 17.05.1
> >>>> >
> >>>> >v1->v2: Patch rebase.
> >>>> >v2->v3: Fixed wrong formating after v2 patch rebase.
> >>>> >v3->v4: Minor documentation changes.
> >>>> >
> >>>> >Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
> >>>> >Reviewed-by: Aaron Conole <acon...@redhat.com>
> >>>>
> >>>>
> >>>> Hi Michal,
> >>>>
> >>>> Is there a reason why you haven't added my acked-by to this latest
> patch?
> >>>https://mail.openvswitch.org/pipermail/ovs-dev/2017-
> >>>> July/334893.html
> >>>>
> >>>> Thanks,
> >>>> Mark
> >>>>
> >>>> >---
> >>>> > .travis/linux-build.sh   |   2 +-
> >>>

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Kavanagh, Mark B]

>From: Aaron Conole [mailto:acon...@redhat.com]
>Sent: Wednesday, July 26, 2017 3:25 PM
>To: Kavanagh, Mark B <mark.b.kavan...@intel.com>
>Cc: Weglicki, MichalX <michalx.wegli...@intel.com>; d...@openvswitch.org
>Subject: Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
>DPDK 17.05.1.
>
>"Kavanagh, Mark B" <mark.b.kavan...@intel.com> writes:
>
>>>From: Weglicki, MichalX
>>>Sent: Wednesday, July 26, 2017 10:52 AM
>>>To: Kavanagh, Mark B <mark.b.kavan...@intel.com>; d...@openvswitch.org
>>>Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add support
>for
>>>DPDK 17.05.1.
>>>
>>>Hey Mark,
>>>
>>>Sorry, I had some problems with rebase, and somehow I just simply forgot t
>to
>>>do so. Do you want me to re-apply this patch as v5?
>>
>> No need Michal - hopefully the maintainer will just roll it in when
>> applying to master.
>
>If you reply with your Acked-by tag, the patchwork instance will
>re-apply it and the maintainer will not have to remember.

Thanks for the tip Aaron - I wasn't aware of that!

Acked-by: Mark Kavanagh <mark.b.kavan...@intel.com>

-Mark

>
>> Thanks,
>> Mark
>>
>>>
>>>> -Original Message-
>>>> From: Kavanagh, Mark B
>>>> Sent: Wednesday, July 26, 2017 10:04 AM
>>>> To: Weglicki, MichalX <michalx.wegli...@intel.com>; d...@openvswitch.org
>>>> Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add support
>>>for DPDK 17.05.1.
>>>>
>>>>
>>>>
>>>> >-Original Message-
>>>> >From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-
>>>boun...@openvswitch.org]
>>>> >On Behalf Of Michal Weglicki
>>>> >Sent: Tuesday, July 25, 2017 1:35 PM
>>>> >To: d...@openvswitch.org
>>>> >Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support
>for
>>>> >DPDK 17.05.1.
>>>> >
>>>> >Upgrading to DPDK 17.05.1 stable release adds new
>>>> >significant features relevant to OVS, including,
>>>> >but not limited to:
>>>> >- tun/tap PMD,
>>>> >- VFIO hotplug support,
>>>> >- Generic flow API.
>>>> >
>>>> >Following changes are applied:
>>>> >- netdev-dpdk: Changes required by DPDK API modifications.
>>>> >- doc: Because of DPDK API changes, backward compatibility
>>>> >  with previous DPDK releases will be broken, thus all
>>>> >  relevant documentation entries are updated.
>>>> >- .travis: DPDK version change from 16.11.1 to 17.05.1.
>>>> >- rhel/openvswitch-fedora.spec.in: DPDK version change
>>>> >  from 16.11 to 17.05.1
>>>> >
>>>> >v1->v2: Patch rebase.
>>>> >v2->v3: Fixed wrong formating after v2 patch rebase.
>>>> >v3->v4: Minor documentation changes.
>>>> >
>>>> >Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
>>>> >Reviewed-by: Aaron Conole <acon...@redhat.com>
>>>>
>>>>
>>>> Hi Michal,
>>>>
>>>> Is there a reason why you haven't added my acked-by to this latest patch?
>>>https://mail.openvswitch.org/pipermail/ovs-dev/2017-
>>>> July/334893.html
>>>>
>>>> Thanks,
>>>> Mark
>>>>
>>>> >---
>>>> > .travis/linux-build.sh   |   2 +-
>>>> > Documentation/faq/releases.rst   |   1 +
>>>> > Documentation/howto/dpdk.rst |   6 +-
>>>> > Documentation/intro/install/dpdk.rst |  14 +--
>>>> > Documentation/topics/dpdk/vhost-user.rst |  12 +--
>>>> > NEWS |   1 +
>>>> > lib/netdev-dpdk.c| 144 +++--
>--
>>>---
>>>> >-
>>>> > rhel/openvswitch-fedora.spec.in  |   2 +-
>>>> > tests/dpdk/ring_client.c |   6 +-
>>>> > 9 files changed, 114 insertions(+), 74 deletions(-)
>>>> >
>>>> >diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
>>>> >index f66b534..efccdf1 100755
>>>> >--- a/.travis/linux-build.sh
>>>> >+++ b/.travis/linux-build.sh
>>>> >@@ -80,7 +80,7 @@ fi
>>>> >
>&g

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Aaron Conole]

"Kavanagh, Mark B" <mark.b.kavan...@intel.com> writes:

>>From: Weglicki, MichalX
>>Sent: Wednesday, July 26, 2017 10:52 AM
>>To: Kavanagh, Mark B <mark.b.kavan...@intel.com>; d...@openvswitch.org
>>Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
>>DPDK 17.05.1.
>>
>>Hey Mark,
>>
>>Sorry, I had some problems with rebase, and somehow I just simply forgot t to
>>do so. Do you want me to re-apply this patch as v5?
>
> No need Michal - hopefully the maintainer will just roll it in when
> applying to master.

If you reply with your Acked-by tag, the patchwork instance will
re-apply it and the maintainer will not have to remember.

> Thanks,
> Mark
>
>>
>>> -Original Message-
>>> From: Kavanagh, Mark B
>>> Sent: Wednesday, July 26, 2017 10:04 AM
>>> To: Weglicki, MichalX <michalx.wegli...@intel.com>; d...@openvswitch.org
>>> Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add support
>>for DPDK 17.05.1.
>>>
>>>
>>>
>>> >-Original Message-
>>> >From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-
>>boun...@openvswitch.org]
>>> >On Behalf Of Michal Weglicki
>>> >Sent: Tuesday, July 25, 2017 1:35 PM
>>> >To: d...@openvswitch.org
>>> >Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
>>> >DPDK 17.05.1.
>>> >
>>> >Upgrading to DPDK 17.05.1 stable release adds new
>>> >significant features relevant to OVS, including,
>>> >but not limited to:
>>> >- tun/tap PMD,
>>> >- VFIO hotplug support,
>>> >- Generic flow API.
>>> >
>>> >Following changes are applied:
>>> >- netdev-dpdk: Changes required by DPDK API modifications.
>>> >- doc: Because of DPDK API changes, backward compatibility
>>> >  with previous DPDK releases will be broken, thus all
>>> >  relevant documentation entries are updated.
>>> >- .travis: DPDK version change from 16.11.1 to 17.05.1.
>>> >- rhel/openvswitch-fedora.spec.in: DPDK version change
>>> >  from 16.11 to 17.05.1
>>> >
>>> >v1->v2: Patch rebase.
>>> >v2->v3: Fixed wrong formating after v2 patch rebase.
>>> >v3->v4: Minor documentation changes.
>>> >
>>> >Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
>>> >Reviewed-by: Aaron Conole <acon...@redhat.com>
>>>
>>>
>>> Hi Michal,
>>>
>>> Is there a reason why you haven't added my acked-by to this latest patch?
>>https://mail.openvswitch.org/pipermail/ovs-dev/2017-
>>> July/334893.html
>>>
>>> Thanks,
>>> Mark
>>>
>>> >---
>>> > .travis/linux-build.sh   |   2 +-
>>> > Documentation/faq/releases.rst   |   1 +
>>> > Documentation/howto/dpdk.rst |   6 +-
>>> > Documentation/intro/install/dpdk.rst |  14 +--
>>> > Documentation/topics/dpdk/vhost-user.rst |  12 +--
>>> > NEWS |   1 +
>>> > lib/netdev-dpdk.c| 144 +++
>>---
>>> >-
>>> > rhel/openvswitch-fedora.spec.in  |   2 +-
>>> > tests/dpdk/ring_client.c |   6 +-
>>> > 9 files changed, 114 insertions(+), 74 deletions(-)
>>> >
>>> >diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
>>> >index f66b534..efccdf1 100755
>>> >--- a/.travis/linux-build.sh
>>> >+++ b/.travis/linux-build.sh
>>> >@@ -80,7 +80,7 @@ fi
>>> >
>>> > if [ "$DPDK" ]; then
>>> > if [ -z "$DPDK_VER" ]; then
>>> >-DPDK_VER="16.11.2"
>>> >+DPDK_VER="17.05.1"
>>> > fi
>>> > install_dpdk $DPDK_VER
>>> > if [ "$CC" = "clang" ]; then
>>> >diff --git a/Documentation/faq/releases.rst
>>b/Documentation/faq/releases.rst
>>> >index 707834b..2ecc24c 100644
>>> >--- a/Documentation/faq/releases.rst
>>> >+++ b/Documentation/faq/releases.rst
>>> >@@ -161,6 +161,7 @@ Q: What DPDK version does each Open vSwitch release
>>work
>>> >with?
>>> > 2.5.x2.2
>>> > 2.6.x16.07.2
>>> > 2.7.x   

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Kavanagh, Mark B]

>From: Weglicki, MichalX
>Sent: Wednesday, July 26, 2017 10:52 AM
>To: Kavanagh, Mark B <mark.b.kavan...@intel.com>; d...@openvswitch.org
>Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
>DPDK 17.05.1.
>
>Hey Mark,
>
>Sorry, I had some problems with rebase, and somehow I just simply forgot t to
>do so. Do you want me to re-apply this patch as v5?

No need Michal - hopefully the maintainer will just roll it in when applying to 
master.

Thanks,
Mark

>
>> -Original Message-
>> From: Kavanagh, Mark B
>> Sent: Wednesday, July 26, 2017 10:04 AM
>> To: Weglicki, MichalX <michalx.wegli...@intel.com>; d...@openvswitch.org
>> Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add support
>for DPDK 17.05.1.
>>
>>
>>
>> >-Original Message-
>> >From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-
>boun...@openvswitch.org]
>> >On Behalf Of Michal Weglicki
>> >Sent: Tuesday, July 25, 2017 1:35 PM
>> >To: d...@openvswitch.org
>> >Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
>> >DPDK 17.05.1.
>> >
>> >Upgrading to DPDK 17.05.1 stable release adds new
>> >significant features relevant to OVS, including,
>> >but not limited to:
>> >- tun/tap PMD,
>> >- VFIO hotplug support,
>> >- Generic flow API.
>> >
>> >Following changes are applied:
>> >- netdev-dpdk: Changes required by DPDK API modifications.
>> >- doc: Because of DPDK API changes, backward compatibility
>> >  with previous DPDK releases will be broken, thus all
>> >  relevant documentation entries are updated.
>> >- .travis: DPDK version change from 16.11.1 to 17.05.1.
>> >- rhel/openvswitch-fedora.spec.in: DPDK version change
>> >  from 16.11 to 17.05.1
>> >
>> >v1->v2: Patch rebase.
>> >v2->v3: Fixed wrong formating after v2 patch rebase.
>> >v3->v4: Minor documentation changes.
>> >
>> >Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
>> >Reviewed-by: Aaron Conole <acon...@redhat.com>
>>
>>
>> Hi Michal,
>>
>> Is there a reason why you haven't added my acked-by to this latest patch?
>https://mail.openvswitch.org/pipermail/ovs-dev/2017-
>> July/334893.html
>>
>> Thanks,
>> Mark
>>
>> >---
>> > .travis/linux-build.sh   |   2 +-
>> > Documentation/faq/releases.rst   |   1 +
>> > Documentation/howto/dpdk.rst |   6 +-
>> > Documentation/intro/install/dpdk.rst |  14 +--
>> > Documentation/topics/dpdk/vhost-user.rst |  12 +--
>> > NEWS |   1 +
>> > lib/netdev-dpdk.c| 144 +++
>---
>> >-
>> > rhel/openvswitch-fedora.spec.in  |   2 +-
>> > tests/dpdk/ring_client.c |   6 +-
>> > 9 files changed, 114 insertions(+), 74 deletions(-)
>> >
>> >diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
>> >index f66b534..efccdf1 100755
>> >--- a/.travis/linux-build.sh
>> >+++ b/.travis/linux-build.sh
>> >@@ -80,7 +80,7 @@ fi
>> >
>> > if [ "$DPDK" ]; then
>> > if [ -z "$DPDK_VER" ]; then
>> >-DPDK_VER="16.11.2"
>> >+DPDK_VER="17.05.1"
>> > fi
>> > install_dpdk $DPDK_VER
>> > if [ "$CC" = "clang" ]; then
>> >diff --git a/Documentation/faq/releases.rst
>b/Documentation/faq/releases.rst
>> >index 707834b..2ecc24c 100644
>> >--- a/Documentation/faq/releases.rst
>> >+++ b/Documentation/faq/releases.rst
>> >@@ -161,6 +161,7 @@ Q: What DPDK version does each Open vSwitch release
>work
>> >with?
>> > 2.5.x2.2
>> > 2.6.x16.07.2
>> > 2.7.x16.11.2
>> >+2.8.x17.05.1
>> >  ===
>> >
>> > Q: I get an error like this when I configure Open vSwitch:
>> >diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst
>> >index af01d3e..1756c8c 100644
>> >--- a/Documentation/howto/dpdk.rst
>> >+++ b/Documentation/howto/dpdk.rst
>> >@@ -528,15 +528,15 @@ described in :ref:`dpdk-testpmd`. Once compiled, run
>the
>> >application::
>> >
>> > When you finish testing, bind the vNICs back to kernel::
>> >

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Weglicki, MichalX]

Hey Mark, 

Sorry, I had some problems with rebase, and somehow I just simply forgot t to 
do so. Do you want me to re-apply this patch as v5? 

> -Original Message-
> From: Kavanagh, Mark B
> Sent: Wednesday, July 26, 2017 10:04 AM
> To: Weglicki, MichalX <michalx.wegli...@intel.com>; d...@openvswitch.org
> Subject: RE: [ovs-dev] [PATCH v4] Update relevant artifacts to add support 
> for DPDK 17.05.1.
> 
> 
> 
> >-Original Message-
> >From: ovs-dev-boun...@openvswitch.org 
> >[mailto:ovs-dev-boun...@openvswitch.org]
> >On Behalf Of Michal Weglicki
> >Sent: Tuesday, July 25, 2017 1:35 PM
> >To: d...@openvswitch.org
> >Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
> >DPDK 17.05.1.
> >
> >Upgrading to DPDK 17.05.1 stable release adds new
> >significant features relevant to OVS, including,
> >but not limited to:
> >- tun/tap PMD,
> >- VFIO hotplug support,
> >- Generic flow API.
> >
> >Following changes are applied:
> >- netdev-dpdk: Changes required by DPDK API modifications.
> >- doc: Because of DPDK API changes, backward compatibility
> >  with previous DPDK releases will be broken, thus all
> >  relevant documentation entries are updated.
> >- .travis: DPDK version change from 16.11.1 to 17.05.1.
> >- rhel/openvswitch-fedora.spec.in: DPDK version change
> >  from 16.11 to 17.05.1
> >
> >v1->v2: Patch rebase.
> >v2->v3: Fixed wrong formating after v2 patch rebase.
> >v3->v4: Minor documentation changes.
> >
> >Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
> >Reviewed-by: Aaron Conole <acon...@redhat.com>
> 
> 
> Hi Michal,
> 
> Is there a reason why you haven't added my acked-by to this latest patch? 
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-
> July/334893.html
> 
> Thanks,
> Mark
> 
> >---
> > .travis/linux-build.sh   |   2 +-
> > Documentation/faq/releases.rst   |   1 +
> > Documentation/howto/dpdk.rst |   6 +-
> > Documentation/intro/install/dpdk.rst |  14 +--
> > Documentation/topics/dpdk/vhost-user.rst |  12 +--
> > NEWS |   1 +
> > lib/netdev-dpdk.c| 144 
> > +++---
> >-
> > rhel/openvswitch-fedora.spec.in  |   2 +-
> > tests/dpdk/ring_client.c |   6 +-
> > 9 files changed, 114 insertions(+), 74 deletions(-)
> >
> >diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
> >index f66b534..efccdf1 100755
> >--- a/.travis/linux-build.sh
> >+++ b/.travis/linux-build.sh
> >@@ -80,7 +80,7 @@ fi
> >
> > if [ "$DPDK" ]; then
> > if [ -z "$DPDK_VER" ]; then
> >-DPDK_VER="16.11.2"
> >+DPDK_VER="17.05.1"
> > fi
> > install_dpdk $DPDK_VER
> > if [ "$CC" = "clang" ]; then
> >diff --git a/Documentation/faq/releases.rst b/Documentation/faq/releases.rst
> >index 707834b..2ecc24c 100644
> >--- a/Documentation/faq/releases.rst
> >+++ b/Documentation/faq/releases.rst
> >@@ -161,6 +161,7 @@ Q: What DPDK version does each Open vSwitch release work
> >with?
> > 2.5.x2.2
> > 2.6.x16.07.2
> > 2.7.x16.11.2
> >+2.8.x17.05.1
> >  ===
> >
> > Q: I get an error like this when I configure Open vSwitch:
> >diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst
> >index af01d3e..1756c8c 100644
> >--- a/Documentation/howto/dpdk.rst
> >+++ b/Documentation/howto/dpdk.rst
> >@@ -528,15 +528,15 @@ described in :ref:`dpdk-testpmd`. Once compiled, run 
> >the
> >application::
> >
> > When you finish testing, bind the vNICs back to kernel::
> >
> >-$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:03.0
> >-$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:04.0
> >+$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:03.0
> >+$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:04.0
> >
> > .. note::
> >
> >   Valid PCI IDs must be passed in above example. The PCI IDs can be 
> > retrieved
> >   like so::
> >
> >-  $ $DPDK_DIR/tools/dpdk-devbind.py --status
> >+  $ $DPDK_DIR/usertools/dpdk-devbind.py --status
> >
> > More information on the dpdkvhostuser ports can be found in
> > :doc:`/topics/dpdk/vhost-user`.
> >d

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Kavanagh, Mark B]

>-Original Message-
>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org]
>On Behalf Of Michal Weglicki
>Sent: Tuesday, July 25, 2017 1:35 PM
>To: d...@openvswitch.org
>Subject: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for
>DPDK 17.05.1.
>
>Upgrading to DPDK 17.05.1 stable release adds new
>significant features relevant to OVS, including,
>but not limited to:
>- tun/tap PMD,
>- VFIO hotplug support,
>- Generic flow API.
>
>Following changes are applied:
>- netdev-dpdk: Changes required by DPDK API modifications.
>- doc: Because of DPDK API changes, backward compatibility
>  with previous DPDK releases will be broken, thus all
>  relevant documentation entries are updated.
>- .travis: DPDK version change from 16.11.1 to 17.05.1.
>- rhel/openvswitch-fedora.spec.in: DPDK version change
>  from 16.11 to 17.05.1
>
>v1->v2: Patch rebase.
>v2->v3: Fixed wrong formating after v2 patch rebase.
>v3->v4: Minor documentation changes.
>
>Signed-off-by: Michal Weglicki 
>Reviewed-by: Aaron Conole 


Hi Michal,

Is there a reason why you haven't added my acked-by to this latest patch? 
https://mail.openvswitch.org/pipermail/ovs-dev/2017-July/334893.html

Thanks,
Mark

>---
> .travis/linux-build.sh   |   2 +-
> Documentation/faq/releases.rst   |   1 +
> Documentation/howto/dpdk.rst |   6 +-
> Documentation/intro/install/dpdk.rst |  14 +--
> Documentation/topics/dpdk/vhost-user.rst |  12 +--
> NEWS |   1 +
> lib/netdev-dpdk.c| 144 +++---
>-
> rhel/openvswitch-fedora.spec.in  |   2 +-
> tests/dpdk/ring_client.c |   6 +-
> 9 files changed, 114 insertions(+), 74 deletions(-)
>
>diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
>index f66b534..efccdf1 100755
>--- a/.travis/linux-build.sh
>+++ b/.travis/linux-build.sh
>@@ -80,7 +80,7 @@ fi
>
> if [ "$DPDK" ]; then
> if [ -z "$DPDK_VER" ]; then
>-DPDK_VER="16.11.2"
>+DPDK_VER="17.05.1"
> fi
> install_dpdk $DPDK_VER
> if [ "$CC" = "clang" ]; then
>diff --git a/Documentation/faq/releases.rst b/Documentation/faq/releases.rst
>index 707834b..2ecc24c 100644
>--- a/Documentation/faq/releases.rst
>+++ b/Documentation/faq/releases.rst
>@@ -161,6 +161,7 @@ Q: What DPDK version does each Open vSwitch release work
>with?
> 2.5.x2.2
> 2.6.x16.07.2
> 2.7.x16.11.2
>+2.8.x17.05.1
>  ===
>
> Q: I get an error like this when I configure Open vSwitch:
>diff --git a/Documentation/howto/dpdk.rst b/Documentation/howto/dpdk.rst
>index af01d3e..1756c8c 100644
>--- a/Documentation/howto/dpdk.rst
>+++ b/Documentation/howto/dpdk.rst
>@@ -528,15 +528,15 @@ described in :ref:`dpdk-testpmd`. Once compiled, run the
>application::
>
> When you finish testing, bind the vNICs back to kernel::
>
>-$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:03.0
>-$ $DPDK_DIR/tools/dpdk-devbind.py --bind=virtio-pci :00:04.0
>+$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:03.0
>+$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=virtio-pci :00:04.0
>
> .. note::
>
>   Valid PCI IDs must be passed in above example. The PCI IDs can be retrieved
>   like so::
>
>-  $ $DPDK_DIR/tools/dpdk-devbind.py --status
>+  $ $DPDK_DIR/usertools/dpdk-devbind.py --status
>
> More information on the dpdkvhostuser ports can be found in
> :doc:`/topics/dpdk/vhost-user`.
>diff --git a/Documentation/intro/install/dpdk.rst
>b/Documentation/intro/install/dpdk.rst
>index a05aa1a..20baa63 100644
>--- a/Documentation/intro/install/dpdk.rst
>+++ b/Documentation/intro/install/dpdk.rst
>@@ -40,7 +40,7 @@ Build requirements
> In addition to the requirements described in :doc:`general`, building Open
> vSwitch with DPDK will require the following:
>
>-- DPDK 16.11
>+- DPDK 17.05.1
>
> - A `DPDK supported NIC`_
>
>@@ -69,9 +69,9 @@ Install DPDK
> #. Download the `DPDK sources`_, extract the file and set ``DPDK_DIR``::
>
>$ cd /usr/src/
>-   $ wget http://fast.dpdk.org/rel/dpdk-16.11.2.tar.xz
>-   $ tar xf dpdk-16.11.2.tar.xz
>-   $ export DPDK_DIR=/usr/src/dpdk-stable-16.11.2
>+   $ wget http://fast.dpdk.org/rel/dpdk-17.05.1.tar.xz
>+   $ tar xf dpdk-17.05.1.tar.xz
>+   $ export DPDK_DIR=/usr/src/dpdk-stable-17.05.1
>$ cd $DPDK_DIR
>
> #. (Optional) Configure DPDK as a shared library
>@@ -187,8 +187,8 @@ to the VFIO driver::
> $ modprobe vfio-pci
> $ /usr/bin/chmod a+x /dev/vfio
> $ /usr/bin/chmod 0666 /dev/vfio/*
>-$ $DPDK_DIR/tools/dpdk-devbind.py --bind=vfio-pci eth1
>-$ $DPDK_DIR/tools/dpdk-devbind.py --status
>+$ $DPDK_DIR/usertools/dpdk-devbind.py --bind=vfio-pci eth1
>+$ $DPDK_DIR/usertools/dpdk-devbind.py --status
>
> Setup OVS
> ~
>@@ -569,7 +569,7 @@ 

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Darrell Ball]

Hi Aaron

You had mentioned that you would be testing with the patch as well.
Were you able to try it ?

Thanks Darrell



-Original Message-
From: <ovs-dev-boun...@openvswitch.org> on behalf of Kevin Traynor 
<ktray...@redhat.com>
Organization: Red Hat
Date: Tuesday, July 25, 2017 at 9:35 AM
To: Michal Weglicki <michalx.wegli...@intel.com>, "d...@openvswitch.org" 
<d...@openvswitch.org>
Subject: Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for 
DPDK 17.05.1.

On 07/25/2017 01:34 PM, Michal Weglicki wrote:
> Upgrading to DPDK 17.05.1 stable release adds new
> significant features relevant to OVS, including,
> but not limited to:
> - tun/tap PMD,
> - VFIO hotplug support,
> - Generic flow API.
> 
> Following changes are applied:
> - netdev-dpdk: Changes required by DPDK API modifications.
> - doc: Because of DPDK API changes, backward compatibility
>   with previous DPDK releases will be broken, thus all
>   relevant documentation entries are updated.
> - .travis: DPDK version change from 16.11.1 to 17.05.1.
> - rhel/openvswitch-fedora.spec.in: DPDK version change
>   from 16.11 to 17.05.1
> 
> v1->v2: Patch rebase.
> v2->v3: Fixed wrong formating after v2 patch rebase.
> v3->v4: Minor documentation changes.
> 
> Signed-off-by: Michal Weglicki <michalx.wegli...@intel.com>
> Reviewed-by: Aaron Conole <acon...@redhat.com>
> ---

Thanks Michal. I tried a few things out and working ok for me.

Acked-by: Kevin Traynor <ktray...@redhat.com>
___
dev mailing list
d...@openvswitch.org

https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev=DwICAg=uilaK90D4TOVoH58JNXRgQ=BVhFA09CGX7JQ5Ih-uZnsw=ekKh60DEjO7sJHJqmg8W8agp2GxZlvAzz5bWGN3ux4k=AZOPEg-J3gwSlEr1nLn9aCrmRt_LBA0YK88h9QS1_-g=
 


___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v4] Update relevant artifacts to add support for DPDK 17.05.1.

[Kevin Traynor]

On 07/25/2017 01:34 PM, Michal Weglicki wrote:
> Upgrading to DPDK 17.05.1 stable release adds new
> significant features relevant to OVS, including,
> but not limited to:
> - tun/tap PMD,
> - VFIO hotplug support,
> - Generic flow API.
> 
> Following changes are applied:
> - netdev-dpdk: Changes required by DPDK API modifications.
> - doc: Because of DPDK API changes, backward compatibility
>   with previous DPDK releases will be broken, thus all
>   relevant documentation entries are updated.
> - .travis: DPDK version change from 16.11.1 to 17.05.1.
> - rhel/openvswitch-fedora.spec.in: DPDK version change
>   from 16.11 to 17.05.1
> 
> v1->v2: Patch rebase.
> v2->v3: Fixed wrong formating after v2 patch rebase.
> v3->v4: Minor documentation changes.
> 
> Signed-off-by: Michal Weglicki 
> Reviewed-by: Aaron Conole 
> ---

Thanks Michal. I tried a few things out and working ok for me.

Acked-by: Kevin Traynor 
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev