[dpdk-dev] [RFC 00/16] enhance checksum offload API

2015-01-23 Thread Olivier MATZ
Hi Stephen, On 01/22/2015 02:01 AM, Stephen Hemminger wrote: > On Thu, 22 Jan 2015 00:36:19 +0100 > Olivier Matz wrote: > >> The goal of this series is to clarify and simplify the mbuf offload API. >> Several issues are solved: > > If you are doing this could you invert the meaning of the check

[dpdk-dev] [RFC 00/16] enhance checksum offload API

2015-01-23 Thread Liu, Jijiang
> > - remove the PKT_TX_UDP_TUNNEL_PKT flag: it is useless from an API point > of view. It was added because i40e need this info for some reason. We > have 3 solutions: > > - remove the flag and adapt the driver to the API (the choice I made > for this series). > I'm checking the L4T

[dpdk-dev] [RFC 00/16] enhance checksum offload API

2015-01-22 Thread Thomas Monjalon
2015-01-22 00:41, Olivier MATZ: > We use the attached scapy script (dpdk-cksum-test.py) for testing. Attaching the python script which was filtered out. -- next part -- A non-text attachment was scrubbed... Name: dpdk-cksum-test.py Type: text/x-python Size: 2407 bytes Desc:

[dpdk-dev] [RFC 00/16] enhance checksum offload API

2015-01-22 Thread Olivier MATZ
Test done on testpmd on x86_64-native-linuxapp-gcc platform: Tester (linux) <> DUT (DPDK) ixgbe6 port0 (i40e or ixgbe) Run testpmd on DUT: cd dpdk.org/ make install T=x86_64-native-linuxapp-gcc cd x86_64-native-linuxapp-gcc/ modprobe uio insmod kmod/igb_

[dpdk-dev] [RFC 00/16] enhance checksum offload API

2015-01-22 Thread Olivier Matz
The goal of this series is to clarify and simplify the mbuf offload API. Several issues are solved: - simplify the definitions of PKT_TX_IP_CKSUM and PKT_TX_IPV4, each flag has now only one meaning. No impact on the code. - add a feature flag for OUTER_IP_CHECKSUM (from Jijiang's patches) - re

[dpdk-dev] [RFC 00/16] enhance checksum offload API

2015-01-21 Thread Stephen Hemminger
On Thu, 22 Jan 2015 00:36:19 +0100 Olivier Matz wrote: > The goal of this series is to clarify and simplify the mbuf offload API. > Several issues are solved: > > - simplify the definitions of PKT_TX_IP_CKSUM and PKT_TX_IPV4, each > flag has now only one meaning. No impact on the code. > > -