lkml.org 
[lkml]   [2020]   [Jul]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v3] net: hyperv: add support for vlans in netvsc driver
Date
    On 22/07/20, 3:48 AM, "Haiyang Zhang" <haiyangz@microsoft.com> 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 suggestion because the "regular" vlan packet length was
> counted by bytes without the VLAN_HLEN(4) -- the vlan tag is
> in the skb metadata, separately from the ethernet header. I want the
> statistical data for AF_PACKET mode consistent with the regular case.

> struct hv_netvsc_packet {
> /* Bookkeeping stuff */
> u8 cp_partial; /* partial copy into send buffer */

> u8 rmsg_size; /* RNDIS header and PPI size */
> u8 rmsg_pgcnt; /* page count of RNDIS header and PPI */
> u8 page_buf_cnt;

> u16 q_idx;
> u16 total_packets;

> u32 total_bytes;
> u32 send_buf_index;
> u32 total_data_buflen;
> };

> Thanks
> - Haiyang

Sorry my bad, found out that the drop was a testbed issue and not related to the change. Patch v5 contains the recommendation.

Thanks,
Sriram

\
 
 \ /
  Last update: 2020-07-22 12:45    [W:0.097 / U:0.668 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site