On Mon, Jun 27, 2011 at 01:41:25PM -0700, Ethan Jackson wrote: > Currently, OVS accepts incoming traffic on all slaves participating > in a bond. In Linux active-backup bonding, all traffic which comes > in on backup slaves is dropped. This patch causes OVS to do the > same. > > Bug #6125.
It looks correct but I think that it shoves too much into a single 'if' block. I think it would be clearer to have separate 'if' statements for the SLB and AB cases. At least, the new variables can be inserted inside the 'if' instead of being at top level. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
