On Thu, 23 Apr 2015 20:58:51 +0000
"Kavanagh, Mark B" <mark.b.kavan...@intel.com> wrote:

> Hi all,
> 
> Just a quick poll: are the resolutions to review comments in this
> patch acceptable to everyone?
> 
> If I've missed anything, or are there any additional opens that need
> to be addressed before it can be merged, just let me know.
> 
> Thanks in advance,
> Mark
> 
> >
> >Update relevant artifacts to add support for DPDK v2.0.0
> > - INSTALL.DPDK.md
> > - travis build script
> > - acinclude.m4: add 'mssse3' flag to OVS_CFLAGS
> > - netdev-dpdk: fix build with unified offload types in DPDK v2.0.0
> >
> >Note that this breaks compatibility with DPDK v1.8.0
> >
> >v1: - update DPDK version & build instructions in INSTALL.DPDK.md
> >    - update DPDK version and remove compile flags in travis/build.sh
> >    - fix build with unified offload types in DPDK v2.0.0
> >
> >v2: - add mssse3 flag to OVS_CFLAGS in acinclude.m4
> >    - reinstate '-Wno-cast-align' compile flag for clang
> >    - add details of vhost user support limitations to
> > INSTALL.DPDK.md
> >    - refactor travis/build.sh to reflect these changes
> >
> >v3: -correct minor typos in commit message
> >
> >Signed-off-by: Mark Kavanagh <mark.b.kavan...@intel.com>
> >Signed-off-by: Panu Matilainen <pmati...@redhat.com>
> >---
> > .travis/build.sh  | 11 +++++++----
> > INSTALL.DPDK.md   | 13 +++++++++----
> > acinclude.m4      |  2 +-
> > lib/netdev-dpdk.c |  3 +--
> > 4 files changed, 18 insertions(+), 11 deletions(-)
> >

Patch looks good and works for me.
Reviewed-by: Flavio Leitner <f...@sysclose.org>

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to