Re: [PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-22 Thread Sriram Krishnan (srirakr2)
On 22/07/20, 3:48 AM, "Haiyang Zhang" wrote: > If you make this change, did you see any drop in a live test? The > "packet->total_bytes" in struct hv_netvsc_packet is for book keeping > only, which is used for stats info, and not visible by the host at all. > I made this

RE: [PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-21 Thread Haiyang Zhang
M > M (nimm) ; xe-linux-external(mailer list) exter...@cisco.com>; k...@kernel.org; linux-hyp...@vger.kernel.org; > net...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v3] net: hyperv: add support for vlans in netvsc driver > > > > On 21/07/20, 4:57

Re: [PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-21 Thread Sriram Krishnan (srirakr2)
On 21/07/20, 4:57 AM, "David Miller" wrote: From: Sriram Krishnan Date: Mon, 20 Jul 2020 22:15:51 +0530 > + if (skb->protocol == htons(ETH_P_8021Q)) { > + u16 vlan_tci = 0; > + skb_reset_mac_header(skb); > Please place an empty line between basic

Re: [PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-20 Thread David Miller
From: Sriram Krishnan Date: Mon, 20 Jul 2020 22:15:51 +0530 > + if (skb->protocol == htons(ETH_P_8021Q)) { > + u16 vlan_tci = 0; > + skb_reset_mac_header(skb); Please place an empty line between basic block local variable declarations and actual code. > +

RE: [PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-20 Thread Haiyang Zhang
Miller ; Jakub Kicinski > ; linux-hyp...@vger.kernel.org; net...@vger.kernel.org; > linux-kernel@vger.kernel.org > Subject: [PATCH v3] net: hyperv: add support for vlans in netvsc driver Also netvsc already supports vlan in "regular" cases. Please be more specific in the subject

RE: [PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-20 Thread Haiyang Zhang
Miller ; Jakub Kicinski > ; linux-hyp...@vger.kernel.org; net...@vger.kernel.org; > linux-kernel@vger.kernel.org > Subject: [PATCH v3] net: hyperv: add support for vlans in netvsc driver > > Vlan tagged packets are getting dropped when used with DPDK that uses > the AF_PACKET interfac

Re: [PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-20 Thread Stephen Hemminger
On Mon, 20 Jul 2020 22:15:51 +0530 Sriram Krishnan wrote: > > + /* When using AF_PACKET we need to remove VLAN from frame > + * and indicate VLAN information in SKB so HOST OS will > + * transmit the VLAN frame > + */ > + if (skb->protocol == htons(ETH_P_8021Q)) { > +

[PATCH v3] net: hyperv: add support for vlans in netvsc driver

2020-07-20 Thread Sriram Krishnan
Vlan tagged packets are getting dropped when used with DPDK that uses the AF_PACKET interface on a hyperV guest. The packet layer uses the tpacket interface to communicate the vlans information to the upper layers. On Rx path, these drivers can read the vlan info from the tpacket header but on