From: Hariprasad Shenai <[email protected]>
Date: Wed, 16 Dec 2015 13:16:25 +0530
> @@ -66,7 +66,7 @@ struct l2t_data {
>
> static inline unsigned int vlan_prio(const struct l2t_entry *e)
> {
> - return e->vlan >> 13;
> + return (e->vlan & VLAN_PRIO_MASK) >> VLAN_PRIO_SHIFT;
e->vlan is a u16, the vlan priotity is the top 3 bits of the 16-bit
value, and finally the right shift will be non-signed.
Therefore this change is absolutely not necessary.
Please remove this patch from the series and resend.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
