Re: [ovs-dev] [PATCH v2 05/26] ofproto: Use ofproto_mutex for groups and keep track of referring flows.

2016-07-29 Thread Jarno Rajahalme
> On Jul 29, 2016, at 1:41 PM, Ben Pfaff wrote: > > On Fri, Jul 29, 2016 at 01:19:03PM -0700, Jarno Rajahalme wrote: >> >>> On Jul 29, 2016, at 12:33 PM, Ben Pfaff wrote: >>> >>> On Thu, Jul 28, 2016 at 05:55:57PM -0700, Jarno Rajahalme wrote: Adding groups

Re: [ovs-dev] [PATCH v2 05/26] ofproto: Use ofproto_mutex for groups and keep track of referring flows.

2016-07-29 Thread Ben Pfaff
On Fri, Jul 29, 2016 at 01:19:03PM -0700, Jarno Rajahalme wrote: > > > On Jul 29, 2016, at 12:33 PM, Ben Pfaff wrote: > > > > On Thu, Jul 28, 2016 at 05:55:57PM -0700, Jarno Rajahalme wrote: > >> Adding groups support for bundles is simpler if also groups are > >> modified under

Re: [ovs-dev] [PATCH v2 05/26] ofproto: Use ofproto_mutex for groups and keep track of referring flows.

2016-07-29 Thread Jarno Rajahalme
> On Jul 29, 2016, at 12:33 PM, Ben Pfaff wrote: > > On Thu, Jul 28, 2016 at 05:55:57PM -0700, Jarno Rajahalme wrote: >> Adding groups support for bundles is simpler if also groups are >> modified under ofproto_mutex. >> >> Eliminate the search for rules when deleting a group so

Re: [ovs-dev] [PATCH v2 05/26] ofproto: Use ofproto_mutex for groups and keep track of referring flows.

2016-07-29 Thread Ben Pfaff
On Thu, Jul 28, 2016 at 05:55:57PM -0700, Jarno Rajahalme wrote: > Adding groups support for bundles is simpler if also groups are > modified under ofproto_mutex. > > Eliminate the search for rules when deleting a group so that we will > not keep the mutex for too long. > > Signed-off-by: Jarno

[ovs-dev] [PATCH v2 05/26] ofproto: Use ofproto_mutex for groups and keep track of referring flows.

2016-07-28 Thread Jarno Rajahalme
Adding groups support for bundles is simpler if also groups are modified under ofproto_mutex. Eliminate the search for rules when deleting a group so that we will not keep the mutex for too long. Signed-off-by: Jarno Rajahalme --- include/openvswitch/ofp-actions.h | 40 +++