[ovs-dev] Re：[PATCH 2/2] ofproto: Do not delete datapath flows on exit by default

[Ben Pfaff]

I applied this to master.

On Sat, Feb 29, 2020 at 10:59:37AM -0800, Ben Pfaff wrote:
> Thanks for testing.
> 
> I think that the "destroy" call should be outside the braces; the simap
> does not "own" the ports, it's just an string-to-integer map that needs
> to get destroyed along with the ofproto.
> 
> On Sat, Feb 29, 2020 at 10:26:42AM +0800, txfh2007 wrote:
> > Hi Ben:
> > 
> >     I have tried, this patch works! Thank you!
> >    One question: should "simap_destroy(&backer->tnl_backers)" be within the close brace ?
> > 
> > Timo  
> > 
> > 
> > 
> > 
> > 
> > Re: [PATCH 2/2] ofproto: Do not delete datapath flows on exit by default
> > 
> > 
> > On Wed, Feb 26, 2020 at 04:40:25PM +0800, txfh2007 wrote:
> > > Hi Ben:
> > >  
> > >     I have read your patch about "not delete datapath flow when daemon exit". I think this patch is really important, It can be used during upgrading without effecting existing traffic. I have test in my env, and found it works! Thanks a lot ! 
> > >     But I have a question about tunnel traffic: I have found if there're vxlan traffic across compute nodes, when daemon exit, the traffic would breakdown. The reason is during "close_dpif_backer"  we would delete tunnel port in datapath, so even if the flows are not delete , but as the tunnel port is deleted , flow action turns to be "drop". So i write this mail to ask if we can add a flag to control this behavior ?
> > 
> > Please test this patch: https://patchwork.ozlabs.org/patch/1246701/
> >