On Thu, Sep 8, 2016 at 2:47 PM, Guru Shetty <g...@ovn.org> wrote: > On 1 September 2016 at 10:02, Ben Pfaff <b...@ovn.org> wrote: > > > Nothing freed 'reply'. This fixes the problem. > > > > Most of this patch is moving coding around. The essential change is that > > breaking the code that works with 'reply' out into a separate function > > makes it possible to catch all paths out of the function so that 'reply' > > can be freed in one place. > > > > Reported-by: Ryan Moats <rmo...@us.ibm.com> > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > > > For me, this commit breaks multi-host networking. table 32 no longer has > NXM_NX_TUN_METADATA related flows. > > It would be nice if someone else can confirm. >
I used flaviof's just-ovn-nodes with a new build from a new cloned repo. Multi-host networking had no issues. Table 32 has NXM_NX_TUN_METADATA flows for remote switch ports. Pings between namespaces on different VMs work. Mickey > > <snip> > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
