> Seems reasonable. Should we mention this in NEWS?
I think it makes sense to mention it in the next patch.
I added this:
- Additional protocols are not mirrored and dropped when forward-bpdu is
false. For a full list, see the ovs-vswitchd.conf.db man page.
>
> I'd mark the following structure "const" also:
>> + static struct masked_eth_addr mea[] = {
>> + { /* STP, IEEE pause frames, and other reserved protocols. */
>> + {0x01, 0x08, 0xc2, 0x00, 0x00, 0x00},
>> + {0xff, 0xff, 0xff, 0xff, 0xff, 0xf0}},
>> +
>
> Otherwise looks good, thank you.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev