Re: [ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-14 Thread Yifeng Sun
Thanks Darrell, that is very helpful. Come up with v2 soon. Yifeng On Mon, Aug 13, 2018 at 9:50 AM, Darrell Ball wrote: > clarification: the running time is per test. > > > On Mon, Aug 13, 2018 at 9:47 AM, Darrell Ball wrote: > >> With a few minor tweaks, below, the running time was brought

Re: [ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-13 Thread Darrell Ball
clarification: the running time is per test. On Mon, Aug 13, 2018 at 9:47 AM, Darrell Ball wrote: > With a few minor tweaks, below, the running time was brought down from > 1.5-2 minutes to 15-25 seconds per test. > > I just changed the tcpdump filter (many variations are possible though) >

Re: [ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-13 Thread Darrell Ball
With a few minor tweaks, below, the running time was brought down from 1.5-2 minutes to 15-25 seconds. I just changed the tcpdump filter (many variations are possible though) and eliminated some redundant tcpdump checks since they are implied. Thanks Darrell dball@ubuntu:~/ovs$ git diff

Re: [ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-10 Thread Yifeng Sun
Hi Darrell, Thanks for the review. Will do. Best, Yifeng On Fri, Aug 10, 2018 at 3:12 PM, Darrell Ball wrote: > Thanks Yifeng > > Thanks for switching to use ‘packet-out’. > Patches 4-8 seem like they can be part of the same patch, since they are > closely related. > As discussed, none of the

Re: [ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-10 Thread Darrell Ball
Thanks Yifeng Thanks for switching to use ‘packet-out’. Patches 4-8 seem like they can be part of the same patch, since they are closely related. As discussed, none of the tests pass for Userspace datapath, so maybe you want to check them out or just disable them for now I tested with “Linux

Re: [ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-09 Thread William Tu
On Thu, Aug 9, 2018 at 12:46 PM, Yifeng Sun wrote: > Yes, in original test, ADD_NATIVE_TUNNEL will try to load upstream > gre modules. But since openvswitch is running in compatible model, > there is a conflict and the gre tests will always fail. So this new series > of tests abandoned using

Re: [ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-09 Thread Yifeng Sun
Yes, in original test, ADD_NATIVE_TUNNEL will try to load upstream gre modules. But since openvswitch is running in compatible model, there is a conflict and the gre tests will always fail. So this new series of tests abandoned using ADD_NATIVE_TUNNEL and doesn't need native gre modules any more.

[ovs-dev] [PATCH 4/8] system-traffic.at: Add gre tunnel test that doesn't depend on upstream gre module

2018-08-08 Thread Yifeng Sun
Introduce a new test that doesn't setup native gre tunnels but sends simulated raw packets. This test is supposed to only run for kernel version from 4.4.x to 4.15.x. Signed-off-by: Yifeng Sun --- tests/system-traffic.at | 47 +++ 1 file changed, 47