Hi Ian,
Thanks for getting back . I agree that the patch Tiago has posted in the latest needs more testing but this is more sort of an improvement. Can we merge the first part of the problem as addressed in https://mail.openvswitch.org/pipermail/ovs-dev/2018-May/346649.html This will atleast prevent silent memory leaks and irrelevant tunnel header pushes . Regards Anju -----Original Message----- From: Stokes, Ian [mailto:ian.sto...@intel.com] Sent: Wednesday, January 16, 2019 5:09 PM To: Lam, Tiago <tiago....@intel.com>; Anju Thomas <anju.tho...@ericsson.com>; Ben Pfaff <b...@ovn.org> Cc: d...@openvswitch.org Subject: RE: [ovs-dev] [PATCH v1] Fix crash due to multiple tnl push action > On 16/01/2019 09:30, Anju Thomas wrote: > > > > Hi Folks, > > > > Are these changes planned to be merged as well? > > > > Regards > > Anju > > Hi Anju, > > Unfortunately, no. An RFC based on the below was proposed to the > mailing list here [1], but no discussion / comments happened after > that. Further discussion and testing would be needed to move this forward... > I wasn't aware of this tbh. It seems like a bug fix so I believe it would be eligible to be merged for the 2.11 release a(and possibly backported also). As it manipulates the dp packets functionality it would need wider testing and discussion among the community. It won't make it upstream this week before we cut for 2.11 I would think but there is 4 weeks allowed until release where bug fixes are allowed. If further work continues it could be merged within that window or afterwards and backported? Ian _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
