On Mon, Sep 26, 2011 at 11:25:51AM -0700, Pravin Shelar wrote: > On Fri, Sep 23, 2011 at 5:17 PM, Ben Pfaff <[email protected]> wrote: > > OFP_VLAN_NONE used to be convenient as a value for struct dst's 'vlan' > > member, because it ended up being used in actions anyway, but now it's > > much better to just use 0. > > I am not sure what is expected type for vlan-id in bonding code, but > atleast bond_unixctl_hash() is still using OFP_VLAN_NONE to generate > bond hash. which is looking inconsistent.
Thanks for pointing that out. I changed that instance to 0 also. I'm doing a few final checks on this series, then I'll push the whole thing. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
