This seems fine, I would go slightly further though.
> + if (s->balance == BM_SLB && port->bridge->cfg->n_flood_vlans) {
> + VLOG_WARN("port %s: SLB bonds are incompatible with flood_vlans, "
> + "please use another bond type or disable flood_vlans",
> + port->name);
> + }
I would change this warning to an error, and actually fail to create
the port in this case. I'm afraid people are going to ignore the
warning in the log. If you disagree, go ahead and merge.
Ethan
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
