-----Original Message----- > Date: Mon, 1 Oct 2018 17:11:50 +0100 > From: Ferruh Yigit <[email protected]> > To: Jerin Jacob <[email protected]> > CC: Wenzhuo Lu <[email protected]>, Jingjing Wu <[email protected]>, > Bernard Iremonger <[email protected]>, John McNamara > <[email protected]>, Marko Kovacevic <[email protected]>, > Thomas Monjalon <[email protected]>, Andrew Rybchenko > <[email protected]>, [email protected] > 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 10/1/2018 4:59 PM, Jerin Jacob wrote: > > -----Original Message----- > >> Date: Mon, 1 Oct 2018 14:46:39 +0100 > >> From: Ferruh Yigit <[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]>, Andrew Rybchenko > >> <[email protected]> > >> CC: [email protected] > >> 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 <[email protected]> > >> > >> 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 > > Hi Jerin, > > This is just detail, the alignment is broken in the output of the log, for > others on/off start from column 56, for this one it is 57, just delete a space > from printf please.
Sure Ferruh. Will add it in v2 > > > > > So, I expect no change in this patch other than rebase to latest head. > > If not, let me know. > > > >> > >> Thanks, > >> ferruh > >> >
