Re: [dpdk-dev] [PATCH v2 0/3] Remove CRC from byte counters

["Ananyev, Konstantin" <konstantin.ananyev@intel.com>]

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Harry van Haaren
> Sent: Wednesday, November 18, 2015 10:48 AM
> To: dev@dpdk.org
> Cc: shemming@brocade.com
> Subject: [dpdk-dev] [PATCH v2 0/3] Remove CRC from byte counters
> 
> This patchset removes CRC bytes from byte statistics in the following
> PMDs: igb, ixgbe, i40e.
> 
> This brings the reported byte statistics in-line with other NICs,
> providing consistency.
> 
> Removing the CRC from byte counters in i40e resolves a bug, see
> i40e commit message for details.
> 
> v2:
> -Done Stephen Hemmingers suggestion of ETHER_CRC_LEN replacing magic #4
> -Removed fm10k patch, thanks Michael Qiu for reporting - my testpmd usage
>  for packet sizes on fm10k was creating an offset of 4 bytes in testing.
> -Reworded some comments
> 
> 
> Harry van Haaren (3):
>   e1000: remove crc size from all byte counters
>   ixgbe: remove crc size from all byte counters
>   i40e: fix rx/tx size mismatch, remove crc bytes
> 
>  drivers/net/e1000/igb_ethdev.c   | 19 ++++++++++++--
>  drivers/net/i40e/i40e_ethdev.c   | 13 ++++++++--
>  drivers/net/ixgbe/ixgbe_ethdev.c | 55 +++++++++++++++++++++++++++++++---------
>  3 files changed, 71 insertions(+), 16 deletions(-)
> 

Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>

> --
> 1.9.1