On Tue, Nov 19, 2013 at 11:03:04AM +0200, Lori Jakab wrote:
> On 11/19/13 12:51 AM, Ben Pfaff wrote:
> >On Mon, Nov 18, 2013 at 02:45:46PM -0800, Ben Pfaff wrote:
> >>On Mon, Nov 18, 2013 at 10:27:48PM +0200, Lorand Jakab wrote:
> >>>Add member is_layer3 to struct ofport_dpif to mark layer 3 ports.  Set
> >>>it to "true" for the only layer 3 port we support for now: lisp.
> >>>
> >>>Additionally, prevent flooding to layer 3 ports.  A later patch will
> >>>also prevent MAC learning.
> >>>
> >>>Signed-off-by: Lorand Jakab <loja...@cisco.com>
> >>vswitch.xml was missing a closing </p>.  I fixed it and will apply
> >>this to master soon.
> >>
> >>Thanks!
> >Actually, no, I take that back, this somehow makes tests 7 and 710
> >reliably fail for me; if I revert it they work again.  It's weird, but
> >I've tried it both ways a couple of times.
> 
> Stupid mistake on my part.  In port_construct() I initialized
> port->is_layer3 *after* the patch port check, which returns on true
> and leaves is_layer uninitialized.  Will send out a fixed patch.

That explains it, thanks for fixing the problem.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to