[ovs-dev] [PATCH v2] ofproto: Delete all groups and meters when (un)configuring a controller.

Ben Pfaff blp at ovn.org
Mon Jan 8 21:17:47 UTC 2018


On Tue, Dec 26, 2017 at 11:12:02AM -0800, Gregory Rose wrote:
> On 12/26/2017 9:33 AM, Gregory Rose wrote:
> >On 12/22/2017 2:09 PM, Ben Pfaff wrote:
> >>On Tue, Nov 07, 2017 at 07:04:02PM -0800, Ben Pfaff wrote:
> >>>Open vSwitch has always deleted all flows from the flow table whenever
> >>>a
> >>>controller is configured or whenever all the controllers are
> >>>unconfigured.
> >>>After this commit, OVS additionally deletes all OpenFlow groups and
> >>>meters.
> >>>
> >>>Suggested-by: Periyasamy Palanisamy
> >>><periyasamy.palanisamy at ericsson.com>
> >>>Suggested-by: Jan Scheurich <jan.scheurich at ericsson.com>
> >>>Signed-off-by: Ben Pfaff <blp at ovn.org>
> >>>---
> >>>v1->v2: Clear the meter table too (thanks Jan).
> >>Anyone want to review this?
> 
> I can't seem to find the original patch in my mailbox but I applied it from
> patchworks and tested it
> with 'make check' and 'make check-kmod'.  The patch itself looks good to me
> and it passes checkpatch.
> 
> Tested-by: Greg Rose <gvrose8192 at gmail.com>
> Reviewed-by: Greg Rose <gvrose8192 at gmail.com>
> 

Thanks for the review.  I applied this to master.


More information about the dev mailing list