Go to file
Olivier Matz 59cfa76a22 i40e: remove the use of UDP tunnel flag
The definition of the flag PKT_TX_UDP_TUNNEL_PKT in rte_mbuf.h was:
  TX packet is an UDP tunneled packet. It must be specified when using
  outer checksum offload (PKT_TX_OUTER_IP_CKSUM)

This flag was used to tell the NIC that the offload type is UDP
(I40E_TXD_CTX_UDP_TUNNELING flag). In the datasheet, it says it's
required to specify the tunnel type in the register. However, some tests
(see [1]) showed that it also works without this flag.

Moreover, it is not explained how the hardware use this
information. From a network perspective, this information is useless for
calculating the outer IP checksum as it does not depend on the payload.

Having this flag in the API would force the application to specify the
tunnel type for something that looks only useful for this PMD. It will
limit the number of possible tunnel types (we would need a flag for each
tunnel type) and therefore prevent to support outer IP checksum for
proprietary tunnels.

Finally, if a hardware advertises "I support outer IP checksum", it must
be supported for any payload types.

This has been validated by [2], knowing that the ipip test case was fixed
after this test report [3].

[1] http://dpdk.org/ml/archives/dev/2015-January/011380.html
[2] http://dpdk.org/ml/archives/dev/2015-January/011475.html
[3] http://dpdk.org/ml/archives/dev/2015-January/011610.html

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Jijiang Liu <jijiang.liu@intel.com>
2015-02-16 19:21:17 +01:00
app mbuf: enhance the API documentation of offload flags 2015-02-16 19:21:17 +01:00
config acl: remove standalone header 2015-02-02 12:30:33 +01:00
doc doc: add ABI documentation 2015-02-03 16:56:59 +01:00
examples examples/vhost: fix crash when linking device to VMDQ fails 2015-01-30 10:41:08 +01:00
lib i40e: remove the use of UDP tunnel flag 2015-02-16 19:21:17 +01:00
mk mk: fix vhost linking 2015-02-12 16:59:36 +01:00
pkg version: 1.8.0 2014-12-20 00:38:39 +01:00
scripts scripts: find areas without maintainer 2015-02-09 15:15:04 +01:00
tools tools: fix setup script for Fedora 21 2014-10-10 17:50:31 +02:00
.gitignore init DPDK repository 2013-03-07 10:57:42 +01:00
GNUmakefile remove trailing whitespaces 2014-06-11 00:29:34 +02:00
LICENSE.GPL doc: GPL/LGPL licenses 2013-07-25 14:43:06 +02:00
LICENSE.LGPL doc: fix file format (dos to unix) 2013-09-06 11:43:07 +02:00
MAINTAINERS maintainers: claim review for virtio and vhost 2015-02-09 16:33:22 +01:00
Makefile remove trailing whitespaces 2014-06-11 00:29:34 +02:00