[ovs-dev] [PATCH] Implement a new port setting "other-config:priority-tags".

[Justin Pettit]

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