-----Original Message----- > Date: Wed, 3 Oct 2018 10:41:33 +0300 > From: Andrew Rybchenko <[email protected]> > To: Jerin Jacob <[email protected]>, Wenzhuo Lu > <[email protected]>, Jingjing Wu <[email protected]>, Bernard > Iremonger <[email protected]>, John McNamara > <[email protected]>, Marko Kovacevic <[email protected]>, > Thomas Monjalon <[email protected]>, Ferruh Yigit > <[email protected]>, Olivier Matz <[email protected]> > CC: [email protected], [email protected] > Subject: Re: [dpdk-dev] [PATCH v2 2/4] ethdev: add Tx offload outer UDP > checksum definition > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 > Thunderbird/60.0 > > External Email > > On 10/2/18 10:24 PM, Jerin Jacob wrote: > > Introduced DEV_TX_OFFLOAD_OUTER_UDP_CKSUM offload flags and > PKT_TX_OUTER_TCP_CKSUM mbuf ol_flags to enable Tx outer UDP > checksum offload. > > PKT_TX_OUTER_UDP_CKSUM, not TCP
OK > > > > > To use hardware Tx outer UDP checksum offload, the user needs to, > > - enable following in mbuff: > > > mbuf OK > > > a) fill outer_l2_len and outer_l3_len in mbuf > b) set the PKT_TX_OUTER_UDP_CKSUM flag > c) set the flag PKT_TX_OUTER_IPV4 or PKT_TX_OUTER_IPV6 > > - configure DEV_TX_OFFLOAD_OUTER_UDP_CKSUM offload flags in slow path > > Signed-off-by: Jerin Jacob > <[email protected]><mailto:[email protected]> > > The question about mbuf and ethdev changes separation is applicable here as > well. I have no strong opinion, but I think it would be good to follow. I don't have strong opinion on this. If there are no other objection, I will split the patch further as mbuf and ethdev as you pointed out.
