On May 31, 2011, at 4:15 PM, Ben Pfaff wrote:
> + if (cfg->wildcards) {
> + uint32_t ofpfw;
> +
> + if (cfg->wildcards == UINT32_MAX) {
> + /* No wildcard defined, try to wildcard as many fields
> + * as possible, however we cannot wildcard all fields.
> + * We need in_port to detect moves.
> + * We need both MAC SA and DA to do L2 learning. */
> + ofpfw = (OFPFW_DL_TYPE | OFPFW_NW_SRC_ALL | OFPFW_NW_DST_ALL
> + | OFPFW_NW_TOS | OFPFW_NW_PROTO
> + | OFPFW_TP_SRC | OFPFW_TP_DST);
As we discussed in person, it may be worth wildcarding the VLAN PCP values,
too. It's not more correct, but it will reduce the amount of traffic going up
to the controller and number of flows in the switch.
--Justin
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev