Re: [ovs-dev] [PATCH] travis: support versioning of DPDK and options to specify git repo and tag

2015-01-03 Thread Thomas Graf
On 01/02/15 at 11:14pm, Shu Shen wrote: > Use DPDK_VER to specify the version of DPDK, and DPDK_GIT to specify > the URL of the DPDK git repository. > > Both flags require "DPDK=1" and "--with-dpdk" shall no longer be > specified as part of "OPTS" environment. > > Example uses: > > - skip both f

[ovs-dev] [PATCH] dpif-netdev: Fix rare flow add race condition.

2015-01-03 Thread Ethan Jackson
Before this patch, dp_netdev_flow_add() inserted newly minted flows in the "flow_table" cmap before inserting them into the per core "dpcls" classifier. Since dpcls_insert() initializes 'flow->cr.mask', there's a brief window where the flow is accessible from the cmap, but has a bogus mask value.

Re: [ovs-dev] [PATCH] dpif-netdev: Fix rare flow add race condition.

2015-01-03 Thread Ethan Jackson
I think this was the crash that sunk us last time. We're in a really good position at this point. The automated test runner is pretty stable and I'm getting some really nice consistent numbers. I should have some graphs shortly. Also btw with the perf stuff. Since the NSDI deadline the datapat

[ovs-dev] [PATCH] freebsd: Fix build failure.

2015-01-03 Thread Alex Wang
This commit fixes an include dependency for header ip6.h, on FreeBSD. Without this commit, the gmake of ovs master on FreeBSD will result in the following error. /usr/include/netinet/ip6.h:82: error: field 'ip6_src' has incomplete type /usr/include/netinet/ip6.h:83: error: field 'ip6_dst' has inc

[ovs-dev] [PATCH 2/2] ovs-command-compgen: Avoid using GNU sed extensions.

2015-01-03 Thread Alex Wang
This commit changes '\?' to '\{0,1\}' to avoid using GNU sed extension. Found by using the script in FreeBSD. Signed-off-by: Alex Wang --- utilities/ovs-command-compgen.bash |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-command-compgen.bash b/utilities/ov

[ovs-dev] [PATCH 1/2] FreeBSD: Fix build failure.

2015-01-03 Thread Alex Wang
This commit fixes an include dependency for header ip6.h, on FreeBSD. Without this commit, the gmake of ovs master on FreeBSD will result in the following error. /usr/include/netinet/ip6.h:82: error: field 'ip6_src' has incomplete type /usr/include/netinet/ip6.h:83: error: field 'ip6_dst' has inc

Re: [ovs-dev] [PATCH V2] ovs-command-completion: Autotest integration.

2015-01-03 Thread Alex Wang
Upon testing the script in my FreeBSD vm, I found many of the csh bash syntax issues can be avoided by using the m4 macros (e.g. array definition for 'for loop' statement). So, I'd keep the 'm4_define's and using 'foreach'. Sent out new patch here: http://openvswitch.org/pipermail/dev/2015-Januar