Thanks for the patch.
Acked-by: Daniele Di Proietto <ddiproie...@vmware.com>
Pravin, would you mind pushing this? I suggest one minor change, inline.
On 10/13/14, 7:17 AM, "maryam.tahhan" <maryam.tah...@intel.com> wrote:
>This patch updates the documentation to reflect that DPDK 1.7.1 is
>supported.
>Travis scripts have also been updated to reflect this. DPDK phy and ring
>ports
>were validated against DPDK 1.7.1.
>(note: ring ports were validated with the upcoming multi-queue patch fix).
>
>Reviewed-by: Mark D. Gray <mark.d.g...@intel.com>
>Signed-off-by: Maryam Tahhan <maryam.tah...@intel.com>
>---
> .travis.yml | 2 +-
> .travis/build.sh | 6 +++---
> INSTALL.DPDK | 4 ++--
> acinclude.m4 | 2 +-
> 4 files changed, 7 insertions(+), 7 deletions(-)
>
>diff --git a/.travis.yml b/.travis.yml
>index 4dfe15e..cd6e623 100644
>--- a/.travis.yml
>+++ b/.travis.yml
>@@ -8,7 +8,7 @@ before_install: ./.travis/prepare.sh
> env:
> - OPTS="--disable-ssl"
> - TESTSUITE=1 KERNEL=1 OPTS="--with-linux=./linux-3.16.2"
>- - KERNEL=1 DPDK=1 OPTS="--with-dpdk=./dpdk-1.7.0/build"
>+ - KERNEL=1 DPDK=1 OPTS="--with-dpdk=./dpdk-1.7.1/build"
>
> script: ./.travis/build.sh $OPTS
>
>diff --git a/.travis/build.sh b/.travis/build.sh
>index 3872893..5dba4fe 100755
>--- a/.travis/build.sh
>+++ b/.travis/build.sh
>@@ -19,9 +19,9 @@ function install_kernel()
>
> function install_dpdk()
> {
>- wget
>https://urldefense.proofpoint.com/v1/url?u=http://www.dpdk.org/browse/dpdk
>/snapshot/dpdk-1.7.0.tar.gz&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=MV9BdLjt
>FIdhBDBaw5z%2BU6SSA2gAfY4L%2F1HCy3VjlKU%3D%0A&m=GQUPdW%2FL5fNkvOTC0RUDpJXz
>qsWaHRpGLnSKLH4paEI%3D%0A&s=c8dc9c6bb356802da33c5b91e88404ee67d81653868939
>c51b7e2cc4beb688d5
>- tar xzvf dpdk-1.7.0.tar.gz > /dev/null
>- cd dpdk-1.7.0
>+ wget
>https://urldefense.proofpoint.com/v1/url?u=http://www.dpdk.org/browse/dpdk
>/snapshot/dpdk-1.7.1.tar.gz&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=MV9BdLjt
>FIdhBDBaw5z%2BU6SSA2gAfY4L%2F1HCy3VjlKU%3D%0A&m=GQUPdW%2FL5fNkvOTC0RUDpJXz
>qsWaHRpGLnSKLH4paEI%3D%0A&s=840ce9c562b5a4da31596fa459d50ec10b38066f49231c
>85f71abe9fb7b6ac81
>+ tar xzvf dpdk-1.7.1.tar.gz > /dev/null
>+ cd dpdk-1.7.1
> find ./ -type f | xargs sed -i
>'s/max-inline-insns-single=100/max-inline-insns-single=400/'
> sed -ri 's,(CONFIG_RTE_BUILD_COMBINE_LIBS=).*,\1y,'
>config/common_linuxapp
> make config CC=gcc T=x86_64-native-linuxapp-gcc
>diff --git a/INSTALL.DPDK b/INSTALL.DPDK
>index d9a77c9..7484b4b 100644
>--- a/INSTALL.DPDK
>+++ b/INSTALL.DPDK
>@@ -14,10 +14,10 @@ and "make".
> Building and Installing:
> ------------------------
>
>-Required DPDK 1.7.
>+Required DPDK 1.7.1
We do not require DPDK 1.7.1. OVS works even with 1.7.0. The reason I'm
saying this is that some distro might have packaged DPDK 1.7.0.
>
> DPDK:
>-Set dir i.g.: export DPDK_DIR=/usr/src/dpdk-1.7.0
>+Set dir i.g.: export DPDK_DIR=/usr/src/dpdk-1.7.1
> cd $DPDK_DIR
> update config/common_linuxapp so that dpdk generate single lib file.
> (modification also required for IVSHMEM build)
>diff --git a/acinclude.m4 b/acinclude.m4
>index 9a7f809..f9ee2ec 100644
>--- a/acinclude.m4
>+++ b/acinclude.m4
>@@ -206,7 +206,7 @@ AC_DEFUN([OVS_CHECK_DPDK], [
> OVS_LDFLAGS="$OVS_LDFLAGS -L$DPDK_LIB_DIR"
> OVS_CFLAGS="$OVS_CFLAGS -I$DPDK_INCLUDE"
>
>- # DPDK 1.7.0 pmd drivers are not linked unless --whole-archive is
>used.
>+ # DPDK 1.7.1 pmd drivers are not linked unless --whole-archive is
>used.
> #
> # This happens because the rest of the DPDK code doesn't use any
>symbol in
> # the pmd driver objects, and the drivers register themselves using
>an
>--
>1.9.0
>
>_______________________________________________
>dev mailing list
>dev@openvswitch.org
>https://urldefense.proofpoint.com/v1/url?u=http://openvswitch.org/mailman/
>listinfo/dev&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=MV9BdLjtFIdhBDBaw5z%2BU
>6SSA2gAfY4L%2F1HCy3VjlKU%3D%0A&m=GQUPdW%2FL5fNkvOTC0RUDpJXzqsWaHRpGLnSKLH4
>paEI%3D%0A&s=e5effbefba9164425ea4eede02252eaea97dc640a221b2f062e73c57da2c0
>2e7
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
