[ovs-dev] [bridge 08/15] bridge: Get rid of "port_ifidx" in struct iface. Fix bonding hash.

Ben Pfaff blp at nicira.com
Mon Mar 21 20:25:32 UTC 2011


On Mon, Mar 21, 2011 at 01:22:58PM -0700, Ethan Jackson wrote:
> > ? ? } else {
> > ? ? ? ? struct bond_entry *e = lookup_bond_entry(port, flow, vlan);
> > - ? ? ? ?if (e->iface_idx < 0 || e->iface_idx >= port->n_ifaces
> > - ? ? ? ? ? ?|| !port->ifaces[e->iface_idx]->enabled) {
> > + ? ? ? ?if (!e->iface) {
> 
> I think we still want to check if !e->iface->enabled here.

Good catch, thank you!



More information about the dev mailing list