Re: [ovs-dev] weekly OVN report

2016-03-05 Thread Russell Bryant
Yes, we have a weekly OVN IRC meeting in #openvswitch on Freenode on Thursdays at 10:15 Pacific / 1:15 Eastern. On Saturday, March 5, 2016, Dan Mihai Dumitriu wrote: > Thanks for the explanation and the link Russell. > > Is there a regular meeting where this topic might be discussed? > On Mar 5,

Re: [ovs-dev] [PATCH 2/4] ofp-actions: Fix use-after-free in bundle action.

2016-03-05 Thread William Tu
Hi Joe, On Thu, Mar 3, 2016 at 12:22 AM, Joe Stringer wrote: > If the actions list in an incoming flow mod is long enough, and there is > a bundle() action with 3 or more slaves, then it is possible for a > index ae961f6cc5bb..fe1424f137a1 100644 > --- a/lib/ofp-actions.c > +++ b/lib/ofp-actions

Re: [ovs-dev] [PATCH 1/4] tests: Add bundle action test with buffer realloc.

2016-03-05 Thread Ben Pfaff
On Thu, Mar 03, 2016 at 09:22:48PM +1300, Joe Stringer wrote: > Add a test which causes internal reallocation of the ofpacts buffer, > followed by a large bundle action which should cause a subsequent > reallocation while decoding slave ports. Running this test under > valgrind reveals the issue be

Re: [ovs-dev] weekly OVN report

2016-03-05 Thread Dan Mihai Dumitriu
Cool, see you there. On Mar 5, 2016 20:38, "Russell Bryant" wrote: > Yes, we have a weekly OVN IRC meeting in #openvswitch on Freenode on > Thursdays at 10:15 Pacific / 1:15 Eastern. > > On Saturday, March 5, 2016, Dan Mihai Dumitriu wrote: > >> Thanks for the explanation and the link Russell. >

Re: [ovs-dev] weekly OVN report

2016-03-05 Thread Ben Pfaff
There are basically two possible paths here. One path is to enhance OVSDB. The other is to switch to a different distributed database. Of course, in the latter case the question is "which one?" Until recently, we weren't seeing much performance or availability pressure on OVSDB, so it made sens

Re: [ovs-dev] [PATCH 2/4] ofp-actions: Fix use-after-free in bundle action.

2016-03-05 Thread Joe Stringer
On 5 March 2016 at 09:13, William Tu wrote: > Hi Joe, > > On Thu, Mar 3, 2016 at 12:22 AM, Joe Stringer wrote: >> >> If the actions list in an incoming flow mod is long enough, and there is >> a bundle() action with 3 or more slaves, then it is possible for a >> index ae961f6cc5bb..fe1424f137a1 1

Re: [ovs-dev] [PATCH 3/4] ofp-actions: Prevent integer overflow in decode.

2016-03-05 Thread Ben Pfaff
On Thu, Mar 03, 2016 at 09:22:50PM +1300, Joe Stringer wrote: > When decoding a variable-length action, if the length of the action > exceeds the length storable in a uint16_t then something has gone > terribly wrong. Assert that this is not the case. > > Signed-off-by: Joe Stringer > --- > lib/

Re: [ovs-dev] [PATCH] netdev-dpdk: Fix memory leak in netdev_dpdk_vhost_destruct().

2016-03-05 Thread Daniele Di Proietto
Thanks, I applied this to master and branch-2.5 On 03/03/2016 04:50, "Flavio Leitner" wrote: >On Thu, 03 Mar 2016 11:30:06 +0300 >Ilya Maximets wrote: > >> Fixes: 4573fbd38fa1 ("netdev-dpdk: Add vhost-user multiqueue support") >> Signed-off-by: Ilya Maximets >> --- >> lib/netdev-dpdk.c | 1 +

Re: [ovs-dev] weekly OVN report

2016-03-05 Thread Dan Mihai Dumitriu
Understood Ben. If the central (NB and/or SB) OVSDB were to be replaced, do you think it would still be preferable to keep the OVSDB protocol between the local ovn-controller and the central control cluster? Or would it be reasonable to consider something like XMPP? Assuming the OVSDB protocol is