[ovs-dev] [PATCH V1] Add offload packets statistics

2019-10-10 Thread zhaozhanxu
Add argument '-m' for command ovs-appctl bridge/dump-flows to display the offloaded packets statistics. The commands display as below: orignal command: ovs-appctl bridge/dump-flows br0 duration=243589s, n_packets=214752, n_bytes=223673384, priority=0,actions=NORMAL table_id=254,

Re: [ovs-dev] [PATCH ovn] pinctrl: Update icmp6 header structures.

2019-10-10 Thread Han Zhou
Thanks Ilya and Mark. I verified the compiling and tests, and applied to master. On Thu, Oct 10, 2019 at 1:23 PM Mark Michelson wrote: > Thanks, Ilya, I just ran into this problem when trying to OVN :) > > Acked-by: Mark Michelson > > On 10/10/19 5:43 AM, Ilya Maximets wrote: > > commit

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-10-10 Thread 0-day Robot
Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 109 characters long (recommended limit is 79) #1018 FILE:

[ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-10-10 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- Documentation/automake.mk | 7 + Documentation/conf.py | 14 + Documentation/ref/index.rst | 35 +- Documentation/ref/ovs-appctl.8.rst | 332 + Documentation/ref/ovs-ctl.8.rst

Re: [ovs-dev] [PATCH] tests: Allow valgrind check for afxdp testsuite.

2019-10-10 Thread William Tu
On Wed, Oct 09, 2019 at 04:18:38PM -0700, William Tu wrote: > On Wed, Oct 09, 2019 at 11:20:00AM -0700, Ben Pfaff wrote: > > On Wed, Oct 09, 2019 at 04:36:49PM +0200, Ilya Maximets wrote: > > > New 'make' target 'check-afxdp-valgrind'. > > > > > > Signed-off-by: Ilya Maximets > > > > Makes

Re: [ovs-dev] [PATCH ovn] pinctrl: Update icmp6 header structures.

2019-10-10 Thread Mark Michelson
Thanks, Ilya, I just ran into this problem when trying to OVN :) Acked-by: Mark Michelson On 10/10/19 5:43 AM, Ilya Maximets wrote: commit 361a47d66 ("flow: Fix using pointer to member of packed struct icmp6_hdr.") in OVS updated 'struct icmp6_error_header' with renaming it to 'struct

[ovs-dev] [PATCH] ovs-vlan-bug-workaround: Remove.

2019-10-10 Thread Ben Pfaff
This workaround only applied to kernels earlier than 2.6.37, but OVS only supports 3.10 and later. As the original author of this code, I won't miss it. Signed-off-by: Ben Pfaff --- Documentation/faq/vlan.rst| 40 - Documentation/ref/index.rst | 4 -

Re: [ovs-dev] Fwd: [PATCH ovn v2] Utilities: add ovn-sim

2019-10-10 Thread Aaron Conole
Flavio Fernandes writes: > Hi Aaron, > > I think the robot is wrong in thinking there is an issue here; but then again > I may the one who is wrong. :^) You're not wrong, but you're not right :) There is an issue with checkpatch.py since the regex on line 415 won't match anything (in this

Re: [ovs-dev] [PATCH] netdev-afxdp: Update memory locking limits unconditionally.

2019-10-10 Thread William Tu
On Wed, Oct 09, 2019 at 04:23:31PM +0200, Ilya Maximets wrote: > Any type of AF_XDP socket in all modes implies creation of BPF map of > type BPF_MAP_TYPE_XSKMAP. This leads to BPF_MAP_CREATE syscall and > subsequently 'xsk_map_alloc()' function that will charge required > memory from the memlock

Re: [ovs-dev] [PATCH] netdev-afxdp: Fix umem creation failure due to uninitialized config.

2019-10-10 Thread William Tu
On Wed, Oct 09, 2019 at 04:17:58PM +0200, Ilya Maximets wrote: > Later version of 'struct xsk_umem_config' contains additional field > 'flags'. OVS doesn't use that field passing uninitialized stack > memory to the 'xsk_umem__create()' call that could fail with > 'Invalid argument' if 'flags' are

Re: [ovs-dev] [PATCH] cirrus: Use latest stable FreeBSD images.

2019-10-10 Thread Ilya Maximets
On 10.10.2019 16:31, Aaron Conole wrote: Ilya Maximets writes: CirrusCI recently introduced [1] new feature to use image families instead of bare image names for gCloud based instances. This allows us to use most recent stable builds. All the stable builds are in the same image family in

Re: [ovs-dev] [PATCH] cirrus: Use latest stable FreeBSD images.

2019-10-10 Thread Aaron Conole
Ilya Maximets writes: > CirrusCI recently introduced [1] new feature to use image families > instead of bare image names for gCloud based instances. > This allows us to use most recent stable builds. All the stable builds > are in the same image family in gCloud and it will run instances using >

[ovs-dev] [PATCH] cirrus: Use latest stable FreeBSD images.

2019-10-10 Thread Ilya Maximets
CirrusCI recently introduced [1] new feature to use image families instead of bare image names for gCloud based instances. This allows us to use most recent stable builds. All the stable builds are in the same image family in gCloud and it will run instances using the most recent one. This also

Re: [ovs-dev] [PATCH 1/1] Add offload packets statistics

2019-10-10 Thread 0-day Robot
Bleep bloop. Greetings zhaozhanxu, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 82 characters long (recommended limit is 79) #139 FILE:

[ovs-dev] [PATCH 1/1] Add offload packets statistics

2019-10-10 Thread zhaozhanxu
Add argument '-m' for command ovs-appctl bridge/dump-flows to display the offloaded packets statistics. The commands display as below: orignal command: ovs-appctl bridge/dump-flows br0 duration=243589s, n_packets=214752, n_bytes=223673384, priority=0,actions=NORMAL table_id=254,

[ovs-dev] [PATCH 0/1] Add offload packets statistics

2019-10-10 Thread zhaozhanxu
Add argument '-m' for command ovs-appctl bridge/dump-flows to display the offloaded packets statistics. The commands display as below: orignal command: ovs-appctl bridge/dump-flows br0 duration=243589s, n_packets=214752, n_bytes=223673384, priority=0,actions=NORMAL table_id=254,

Re: [ovs-dev] [PATCH ovn v2] Utilities: add ovn-sim

2019-10-10 Thread 0-day Robot
Bleep bloop. Greetings Flavio Fernandes, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: New doc ovn-sim.1.rst not listed in Documentation/automake.mk Lines checked: 813,

[ovs-dev] [PATCH ovn v2] Utilities: add ovn-sim

2019-10-10 Thread Flavio Fernandes
Adding a wrapper to ovs-sim called ovn-sim, which re-introduces the primitives for OVN in the simulator that were lost as part of the ovn-org/ovn split. Also resurrected the man page used to document ovn-sim usage. Lastly, there were some leftover ovs-sim references that were not proper. This

[ovs-dev] Call Tasha For More Informations

2019-10-10 Thread Trusty Cash Loan (Pty) Ltd
Attention, Open Attached Files And Send Your Documents Back To Us As We Are Giving Out Loans At 3% Fixed Interest Rate, We Have Personal Loans, Home Loan, Debt Consolidation Loan And Business Loan. Apply while offer last. Regards, Team Trusty Cash Loan

[ovs-dev] [PATCH ovn] pinctrl: Update icmp6 header structures.

2019-10-10 Thread Ilya Maximets
commit 361a47d66 ("flow: Fix using pointer to member of packed struct icmp6_hdr.") in OVS updated 'struct icmp6_error_header' with renaming it to 'struct icmp6_data_header'. OVN should update names of a structure and constants accordingly. Signed-off-by: Ilya Maximets --- This is a copy-paste

Re: [ovs-dev] [PATCH] flow: Fix using pointer to member of packed struct icmp6_hdr.

2019-10-10 Thread Ilya Maximets
On 09.10.2019 20:18, Ben Pfaff wrote: On Wed, Oct 09, 2019 at 06:54:29PM +0200, Ilya Maximets wrote: On 08.10.2019 18:55, William Tu wrote: On Tue, Oct 01, 2019 at 08:04:00PM +0300, Ilya Maximets wrote: OVS has no structure definition for ICMPv6 header with additional data. More precisely, it

[ovs-dev] [RFC ovn] Add CoPP (Control Plane Protection).

2019-10-10 Thread Dumitru Ceara
Add new 'Copp' (Control plane protection) table to OVN Northbound DB: - this stores mappings between control plane protocol names and meters that should be used to rate limit controller-destined traffic for those protocols. Add new 'copp' columns to the following OVN Northbound DB tables: -