Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-11 Thread Darrell Ball
; To: Chandran, Sugesh ; Ilya Maximets > ; ovs-dev@openvswitch.org; Ben Pfaff > > Subject: Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx > chceksum offload. > > Hi Sugesh > > For future reference, the naming guidelines are

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-11 Thread Chandran, Sugesh
uly 10, 2017 5:53 PM > > To: Ilya Maximets ; ovs- > d...@openvswitch.org; > > Ben Pfaff > > Cc: Chandran, Sugesh ; Traynor, Kevin > > > > Subject: Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx > > chceksum offload. > > >

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-10 Thread Darrell Ball
Ball [mailto:db...@vmware.com] > Sent: Monday, July 10, 2017 5:53 PM > To: Ilya Maximets ; ovs-dev@openvswitch.org; > Ben Pfaff > Cc: Chandran, Sugesh ; Traynor, Kevin > > Subject: Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx > chceksum offloa

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-10 Thread Chandran, Sugesh
[ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx > chceksum offload. > > > > On 7/10/17, 1:37 AM, "Ilya Maximets" wrote: > > > ‘chceksum’ is misspelled > > > > Since these patches really only affect ‘dpdk’, the module name ‘dpdk’

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-10 Thread Darrell Ball
On 7/10/17, 1:37 AM, "Ilya Maximets" wrote: > ‘chceksum’ is misspelled > > Since these patches really only affect ‘dpdk’, the module name ‘dpdk’ may more accurately > reflect the real effect of these patches. Please, don't do that. Only patches that changes lib/dpdk.{

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-10 Thread Darrell Ball
rrell Ball ; Ben Pfaff > > Cc: Chandran, Sugesh ; Traynor, Kevin > > Subject: Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx > chceksum offload. > > > ‘chceksum’ is misspelled > > [Sugesh] Will correct in next version. &g

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-10 Thread Chandran, Sugesh
Regards _Sugesh > -Original Message- > From: Ilya Maximets [mailto:i.maxim...@samsung.com] > Sent: Monday, July 10, 2017 9:37 AM > To: ovs-dev@openvswitch.org; Darrell Ball ; Ben Pfaff > > Cc: Chandran, Sugesh ; Traynor, Kevin > > Subject: Re: [ovs-dev] [PATCH

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-10 Thread Ilya Maximets
> ‘chceksum’ is misspelled > > Since these patches really only affect ‘dpdk’, the module name ‘dpdk’ may > more accurately > reflect the real effect of these patches. Please, don't do that. Only patches that changes lib/dpdk.{c,h} should have 'dpdk' prefix in subject line. All other patches shou

Re: [ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-07-09 Thread Darrell Ball
‘chceksum’ is misspelled Since these patches really only affect ‘dpdk’, the module name ‘dpdk’ may more accurately reflect the real effect of these patches. btw, I think there have been significant changes b/w V2 and V3… On 6/27/17, 7:39 AM, "Sugesh Chandran" wrote: Conntrack need not v

[ovs-dev] [PATCH v3 0/2] conntrack : Add support for rx chceksum offload.

2017-06-27 Thread Sugesh Chandran
Conntrack need not verify the checksum of incoming packets if its validated by DPDK physical NIC ports. Also make use the DPDK rx checksum mask bits along with flags while validating the reported hardware checksum state. v2->v3 - Rebased on latest master. - Updated the existing checksum validat