[dpdk-dev] [PATCH 1/4] ethdev: add SCTP Rx checksum offload support

Ferruh Yigit ferruh.yigit at intel.com
Tue Oct 2 11:13:29 CEST 2018

On 10/1/2018 4:59 PM, Jerin Jacob wrote:
> -----Original Message-----
>> Date: Mon, 1 Oct 2018 14:46:39 +0100
>> From: Ferruh Yigit <ferruh.yigit at intel.com>
>> To: Jerin Jacob <jerin.jacob at caviumnetworks.com>, Wenzhuo Lu
>>  <wenzhuo.lu at intel.com>, Jingjing Wu <jingjing.wu at intel.com>, Bernard
>>  Iremonger <bernard.iremonger at intel.com>, John McNamara
>>  <john.mcnamara at intel.com>, Marko Kovacevic <marko.kovacevic at intel.com>,
>>  Thomas Monjalon <thomas at monjalon.net>, Andrew Rybchenko
>>  <arybchenko at solarflare.com>
>> CC: dev at dpdk.org
>> Subject: Re: [dpdk-dev] [PATCH 1/4] ethdev: add SCTP Rx checksum offload
>>  support
>> User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
>>  Thunderbird/52.9.1
>> On 9/13/2018 2:47 PM, Jerin Jacob wrote:
>>> Signed-off-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>
>> Overall set looks good to me, I put some comments on individual patches.
>> And can you please rebase on top of latest head?
> Sure.
> Regarding space issue mentioned in other email in this thread.
> It looks like similar space added in other offloads.
> example: http://git.dpdk.org/dpdk/tree/app/test-pmd/config.c#n571
> So, I expect no change in this patch other than rebase to latest head.
> If not, let me know.

As commented to the patch, can you also check "csum show", "csum set" functions
in testpmd, I think they are affected and need to be updated with your patch.

More information about the dev mailing list