On Wed, Jul 04, 2012 at 05:50:44PM +0900, Simon Horman wrote: > + ovs_be16 vid_and_cfi = flow->vlan_tci & htons(VLAN_PCP_MASK | VLAN_CFI); This cannot be right, the variable name says VID but the expression says PCP.