<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-GB" link="#467886" vlink="#96607D" style="word-wrap:break-word"> <div class="WordSection1"> The previous code incorrectly parsed the VLAN ID and priority.<o:p></o:p> If the 16-bits of VLAN ID and priority/CFI on the wire was<o:p></o:p> 0123456789ABCDEF the code parsed it as 456789ABCDEF3012. There<o:p></o:p> were macros defined to handle this conversion but they were not<o:p></o:p> used.<o:p></o:p> <o:p> </o:p> This fix takes an approach similar to the Linux netvsc driver and<o:p></o:p> defines an explicit structure to use for parsing.<o:p></o:p> <o:p> </o:p> Fixes: 4e9c73e96e83 ("net/netvsc: add Hyper-V network device")<o:p></o:p> Cc: sthemmin@microsoft.com<o:p></o:p> Cc: stable@dpdk.org<o:p></o:p> <o:p> </o:p> Signed-off-by: Alan Elder <alan.elder@microsoft.com><o:p></o:p> ---<o:p></o:p> .mailmap | 1 +<o:p></o:p> drivers/net/netvsc/hn_rxtx.c | 23 +++++++++++++++++------<o:p></o:p> drivers/net/netvsc/ndis.h | 23 +++++++++++++----------<o:p></o:p> 3 files changed, 31 insertions(+), 16 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/.mailmap b/.mailmap<o:p></o:p> index a0756974e2..eca02318d6 100644<o:p></o:p> --- a/.mailmap<o:p></o:p> +++ b/.mailmap<o:p></o:p> @@ -33,6 +33,7 @@ Alain Leon <xerebz@gmail.com><o:p></o:p> Alan Brady <alan.brady@intel.com><o:p></o:p> Alan Carew <alan.carew@intel.com><o:p></o:p> Alan Dewar <alan.dewar@att.com> <adewar@brocade.com><o:p></o:p> +Alan Elder <alan.elder@microsoft.com><o:p></o:p> Alan Liu <zaoxingliu@gmail.com><o:p></o:p> Alan Winkowski <walan@marvell.com><o:p></o:p> Alejandro Lucero <alejandro.lucero@netronome.com><o:p></o:p> diff --git a/drivers/net/netvsc/hn_rxtx.c b/drivers/net/netvsc/hn_rxtx.c<o:p></o:p> index e4f5015aa3..e3b9899f1e 100644<o:p></o:p> --- a/drivers/net/netvsc/hn_rxtx.c<o:p></o:p> +++ b/drivers/net/netvsc/hn_rxtx.c<o:p></o:p> @@ -42,8 +42,13 @@<o:p></o:p> #define HN_TXD_CACHE_SIZE 32 /* per cpu tx_descriptor pool cache */<o:p></o:p> #define HN_RXQ_EVENT_DEFAULT 2048<o:p></o:p> <o:p></o:p> +#define HN_VLAN_PRIO_MASK 0xe000 /* Priority Code Point */<o:p></o:p> +#define HN_VLAN_PRIO_SHIFT 13<o:p></o:p> +#define HN_VLAN_CFI_MASK 0x1000 /* Canonical Format Indicator / Drop Eligible Indicator */<o:p></o:p> +#define HN_VLAN_VID_MASK 0x0fff /* VLAN Identifier */<o:p></o:p> +<o:p></o:p> struct hn_rxinfo {<o:p></o:p> - uint32_t vlan_info;<o:p></o:p> + struct ndis_pkt_vlan_info vlan_info;<o:p></o:p> uint32_t csum_info;<o:p></o:p> uint32_t hash_info;<o:p></o:p> uint32_t hash_value;<o:p></o:p> @@ -477,7 +482,7 @@ hn_rndis_rxinfo(const void *info_data, unsigned int info_dlen,<o:p></o:p> case NDIS_PKTINFO_TYPE_VLAN:<o:p></o:p> if (unlikely(dlen < NDIS_VLAN_INFO_SIZE))<o:p></o:p> return -EINVAL;<o:p></o:p> - info->vlan_info = *((const uint32_t *)data);<o:p></o:p> + info->vlan_info = *((const struct ndis_pkt_vlan_info *)data);<o:p></o:p> mask |= HN_RXINFO_VLAN;<o:p></o:p> break;<o:p></o:p> <o:p></o:p> @@ -611,8 +616,10 @@ static void hn_rxpkt(struct hn_rx_queue *rxq, struct hn_rx_bufinfo *rxb,<o:p></o:p> RTE_PTYPE_L3_MASK |<o:p></o:p> RTE_PTYPE_L4_MASK);<o:p></o:p> <o:p></o:p> - if (info->vlan_info != HN_NDIS_VLAN_INFO_INVALID) {<o:p></o:p> - m->vlan_tci = info->vlan_info;<o:p></o:p> + if (info->vlan_info.value != HN_NDIS_VLAN_INFO_INVALID) {<o:p></o:p> + m->vlan_tci = info->vlan_info.vlanid |<o:p></o:p> + (info->vlan_info.pri << HN_VLAN_PRIO_SHIFT) |<o:p></o:p> + (info->vlan_info.cfi ? HN_VLAN_CFI_MASK : 0);<o:p></o:p> m->ol_flags |= RTE_MBUF_F_RX_VLAN_STRIPPED | RTE_MBUF_F_RX_VLAN;<o:p></o:p> <o:p></o:p> /* NDIS always strips tag, put it back if necessary */<o:p></o:p> @@ -669,7 +676,7 @@ static void hn_rndis_rx_data(struct hn_rx_queue *rxq,<o:p></o:p> unsigned int pktinfo_off, pktinfo_len;<o:p></o:p> const struct rndis_packet_msg *pkt = data;<o:p></o:p> struct hn_rxinfo info = {<o:p></o:p> - .vlan_info = HN_NDIS_VLAN_INFO_INVALID,<o:p></o:p> + .vlan_info.value = HN_NDIS_VLAN_INFO_INVALID,<o:p></o:p> .csum_info = HN_NDIS_RXCSUM_INFO_INVALID,<o:p></o:p> .hash_info = HN_NDIS_HASH_INFO_INVALID,<o:p></o:p> };<o:p></o:p> @@ -1332,7 +1339,11 @@ static void hn_encap(struct rndis_packet_msg *pkt,<o:p></o:p> if (m->ol_flags & RTE_MBUF_F_TX_VLAN) {<o:p></o:p> pi_data = hn_rndis_pktinfo_append(pkt, NDIS_VLAN_INFO_SIZE,<o:p></o:p> NDIS_PKTINFO_TYPE_VLAN);<o:p></o:p> - *pi_data = m->vlan_tci;<o:p></o:p> + struct ndis_pkt_vlan_info *vlan = (struct ndis_pkt_vlan_info *)pi_data;<o:p></o:p> + vlan->value = 0;<o:p></o:p> + vlan->vlanid = (m->vlan_tci & HN_VLAN_VID_MASK);<o:p></o:p> + vlan->cfi = (!!(m->vlan_tci & HN_VLAN_CFI_MASK));<o:p></o:p> + vlan->pri = ((m->vlan_tci & HN_VLAN_PRIO_MASK) >> HN_VLAN_PRIO_SHIFT);<o:p></o:p> }<o:p></o:p> <o:p></o:p> if (m->ol_flags & RTE_MBUF_F_TX_TCP_SEG) {<o:p></o:p> diff --git a/drivers/net/netvsc/ndis.h b/drivers/net/netvsc/ndis.h<o:p></o:p> index d97a397a86..a1e587c738 100644<o:p></o:p> --- a/drivers/net/netvsc/ndis.h<o:p></o:p> +++ b/drivers/net/netvsc/ndis.h<o:p></o:p> @@ -316,16 +316,19 @@ struct ndis_offload {<o:p></o:p> */<o:p></o:p> <o:p></o:p> /* VLAN */<o:p></o:p> -#define NDIS_VLAN_INFO_SIZE sizeof(uint32_t)<o:p></o:p> -#define NDIS_VLAN_INFO_PRI_MASK 0x0007<o:p></o:p> -#define NDIS_VLAN_INFO_CFI_MASK 0x0008<o:p></o:p> -#define NDIS_VLAN_INFO_ID_MASK 0xfff0<o:p></o:p> -#define NDIS_VLAN_INFO_MAKE(id, pri, cfi) \<o:p></o:p> - (((pri) & NDIS_VLAN_INFO_PRI_MASK) | \<o:p></o:p> - (((cfi) & 0x1) << 3) | (((id) & 0xfff) << 4))<o:p></o:p> -#define NDIS_VLAN_INFO_ID(inf) (((inf) & NDIS_VLAN_INFO_ID_MASK) >> 4)<o:p></o:p> -#define NDIS_VLAN_INFO_CFI(inf) (((inf) & NDIS_VLAN_INFO_CFI_MASK) >> 3)<o:p></o:p> -#define NDIS_VLAN_INFO_PRI(inf) ((inf) & NDIS_VLAN_INFO_PRI_MASK)<o:p></o:p> +struct ndis_pkt_vlan_info {<o:p></o:p> + union {<o:p></o:p> + struct {<o:p></o:p> + uint32_t pri:3; /* User Priority */<o:p></o:p> + uint32_t cfi:1; /* Canonical Format ID / DEI */<o:p></o:p> + uint32_t vlanid:12; /* VLAN ID */<o:p></o:p> + uint32_t reserved:16;<o:p></o:p> + };<o:p></o:p> + uint32_t value;<o:p></o:p> + };<o:p></o:p> +};<o:p></o:p> +<o:p></o:p> +#define NDIS_VLAN_INFO_SIZE sizeof(struct ndis_pkt_vlan_info)<o:p></o:p> <o:p></o:p> /* Reception checksum */<o:p></o:p> #define NDIS_RXCSUM_INFO_SIZE sizeof(uint32_t)<o:p></o:p> -- <o:p></o:p> 2.25.1<o:p></o:p> <o:p> </o:p> </div> </body> </html>