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

2020-02-29 Thread 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

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

2020-02-29 Thread Ben Pfaff
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,

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

2020-02-28 Thread txfh2007 via dev
Hi Ben: I have tried, this patch works! Thank you! One question: should "simap_destroy(>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: >