On Nov 17, 2011, at 4:01 PM, Ben Pfaff wrote:

> +            have a 802.1Q header if it has a nonzero VLAN ID.

In other places, "an 802.1Q" is used instead of "a 802.1Q".

> +            Any packet with an 802.1Q header with a nonzero VLAN ID that
> +            ingresses on an access port is dropped, regardless of whether the
> +            VLAN ID in the header is the access port's VLAN ID.
> +            (Priority-tagged frames are allowed on ingress.)

Do you think it's worth forward-referencing the description of priority-tagged 
frames?  Or a parenthetical "(those with a VLAN ID of zero)"?

Otherwise, it looks good.

Thanks,

--Justin


_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to