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

Reply via email to