Re: [ovs-dev] [PATCH] ofproto: Always delete rules before deleting a meter.

2016-10-06 Thread Petr Machata
Jarno Rajahalme writes: > Pushed to master, but not sure if need to backport to earlier > branches, as meters are not really supported by any release yet? Nope, we only hit this because our own ofproto provider actually does support meters. Thanks for this! Petr ___

Re: [ovs-dev] [PATCH] ofproto: Always delete rules before deleting a meter.

2016-10-05 Thread Ben Pfaff
On Tue, Oct 04, 2016 at 06:17:08PM -0700, Jarno Rajahalme wrote: > > > On Oct 4, 2016, at 2:32 PM, Ben Pfaff wrote: > > > > On Fri, Sep 30, 2016 at 11:08:19AM -0700, Jarno Rajahalme wrote: > >> When deleting a bridge it is currently possible to delete a mater > >> without deleting the rules usin

Re: [ovs-dev] [PATCH] ofproto: Always delete rules before deleting a meter.

2016-10-04 Thread Jarno Rajahalme
> On Oct 4, 2016, at 2:32 PM, Ben Pfaff wrote: > > On Fri, Sep 30, 2016 at 11:08:19AM -0700, Jarno Rajahalme wrote: >> When deleting a bridge it is currently possible to delete a mater >> without deleting the rules using the meter first. Fix this by moving >> the meter's rule deletion to meter_

Re: [ovs-dev] [PATCH] ofproto: Always delete rules before deleting a meter.

2016-10-04 Thread Ben Pfaff
On Fri, Sep 30, 2016 at 11:08:19AM -0700, Jarno Rajahalme wrote: > When deleting a bridge it is currently possible to delete a mater > without deleting the rules using the meter first. Fix this by moving > the meter's rule deletion to meter_delete(). > > Signed-off-by: Jarno Rajahalme > Reported

[ovs-dev] [PATCH] ofproto: Always delete rules before deleting a meter.

2016-09-30 Thread Jarno Rajahalme
When deleting a bridge it is currently possible to delete a mater without deleting the rules using the meter first. Fix this by moving the meter's rule deletion to meter_delete(). Signed-off-by: Jarno Rajahalme Reported-by: Petr Machata --- ofproto/ofproto.c | 35 +++---