frank.blaschka@de.ibm.com wrote:

 >                        tag = (u16 *)(new_skb->data + 12);
 >                        *tag = __constant_htons(ETH_P_8021Q);
 >                        *(tag + 1) = htons(vlan_tx_tag_get(new_skb));
 >-                       VLAN_TX_SKB_CB(new_skb)->magic = 0;

The intention appears to be to invalidate the tag (its used
later on again if present), so that doesn't look correct.