On Thu, Sep 8, 2016 at 2:47 PM, Guru Shetty <[email protected]> wrote: > On 1 September 2016 at 10:02, Ben Pfaff <[email protected]> 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 <[email protected]> > > Signed-off-by: Ben Pfaff <[email protected]> > > > > 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 > [email protected] > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
