Re: [ovs-dev] [PATCH v3 1/4] ovn: ovn-ctl support for HA ovn DB servers

2016-10-09 Thread Babu Shanmugam
On Friday 07 October 2016 05:33 AM, Andy Zhou wrote: Babu, Thank you for working on this. At a high level, it is not clear to me the boundary between ocf scripts and the ovn-ctl script -- i.e. which aspect is managed by which entity. For example, 1) which scripts are responsible for starti

Re: [ovs-dev] [PATCH 05/12] dpif-netdev: Clear flow batches inside packet_batch_execute.

2016-10-09 Thread Fischetti, Antonio
Thanks Daniele, in this patch we moved the reset of batches[i].flow->batch from dp_netdev_input__() into packet_batch_per_flow_execute(). So before calling dp_netdev_execute_actions() the flow->batch is already NULL. I think this should prevent recirculation, or am I missing some detail? Antonio

Re: [ovs-dev] [PATCH 02/12] flow: Add comments to mf_get_next_in_map()

2016-10-09 Thread Fischetti, Antonio
Thanks Jarno, will follow your suggestions. That's really a good improvement and will help a lot to explain the function details. Antonio > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Jarno > Rajahalme > Sent: Friday, October 7, 2016 10:08 PM > To: Bo

Re: [ovs-dev] [PATCH v2] vagrant: cleanup before building

2016-10-09 Thread Ben Pfaff
On Sat, Oct 08, 2016 at 07:27:17AM -0300, Thadeu Lima de Souza Cascardo wrote: > Clean the source directory before building, otherwise, build might fail if it > has been configured already. > > Only do it if there is a Makefile present, as suggested by Ben Pfaff. > > Signed-off-by: Thadeu Lima de

Re: [ovs-dev] Hyphens in address set names

2016-10-09 Thread Kevin Lin
Got it, thanks Ben. On Thu, Oct 6, 2016 at 3:09 PM, Ben Pfaff wrote: > On Thu, Oct 06, 2016 at 02:56:44PM -0700, Kevin Lin wrote: > > I’m a developer under Ethan Jackson on Quilt. We’re using address sets > in our ACL match strings, > > but they don’t seem to parse properly for names with hypens

Re: [ovs-dev] [PATCH 05/12] dpif-netdev: Clear flow batches inside packet_batch_execute.

2016-10-09 Thread Daniele Di Proietto
2016-10-09 0:54 GMT-07:00 Fischetti, Antonio : > Thanks Daniele, > in this patch we moved the reset of batches[i].flow->batch > from dp_netdev_input__() into packet_batch_per_flow_execute(). > So before calling dp_netdev_execute_actions() the flow->batch is already > NULL. > batches[0]->flow->bat