[dpdk-dev] [PATCH 10/12] testpmd: rework csum forward engine
Olivier MATZ
olivier.matz at 6wind.com
Tue Nov 11 10:55:37 CET 2014
Hi Jijiang,
On 11/11/2014 09:35 AM, Liu, Jijiang wrote:
> The PKT_TX_VXLAN_CKSUM was not set in the patch, and VXLAN TX checksum offload would not work.
Thank you for reporting this. Indeed, there is an issue. See below.
>> +/* Calculate the checksum of outer header (only vxlan is supported,
>> + * meaning IP + UDP). The caller already checked that it's a vxlan
>> + * packet */
>> +static uint64_t
>> +process_outer_cksums(void *outer_l3_hdr, uint16_t outer_ethertype,
>> + uint16_t outer_l3_len, uint16_t testpmd_ol_flags) {
>> + struct ipv4_hdr *ipv4_hdr = outer_l3_hdr;
>> + struct ipv6_hdr *ipv6_hdr = outer_l3_hdr;
>> + struct udp_hdr *udp_hdr;
>> + uint64_t ol_flags = 0;
>> +
>> + if (testpmd_ol_flags & TESTPMD_TX_OFFLOAD_VXLAN_CKSUM)
>> + ol_flags |= PKT_TX_IP_CKSUM;
Here it should be: ol_flags |= PKT_TX_VXLAN_CKSUM
I'll fix that in the next version.
Regards,
Olivier
More information about the dev
mailing list