On Wed, Apr 12, 2017 at 01:21:30PM -0700, Joe Stringer wrote: > On 12 April 2017 at 12:43, Eric Garver <[email protected]> wrote: > > On Wed, Apr 12, 2017 at 04:12:51PM +0000, Alin Serdean wrote: > >> Thanks a lot for fixing the leak Eric! > >> > >> We should apply on 2.7 as well, but needs a rebase. > >> > >> Is it okay if we add the function call on 2.7 as well or should we limit > >> it too WIN32? > > > > I don't think it would hurt anything. If others disagree I can submit a > > 2.7 only patch. > > > > Joe, > > Do you have an opinion? > > Hi Eric, > > I think that if you split this patch up and simplified it then it > would make it easier to review, apply, and backport. Really the > refactor belongs with the rtnetlink series.
You're right. > > Since I was already looking at this, I went ahead and did it: > https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/330743.html > https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/330741.html > https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/330742.html Great. > > I'll apply the first two to master and branch-2.7 shortly. Feel free > to take the third patch into your rtnetlink series when you resubmit. Sounds like a plan. Thanks. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
