[ovs-dev] [PATCH v2] ovn-controller: Provide the option to set Encap.options:csum
Russell Bryant
russell at ovn.org
Thu Feb 2 15:40:31 UTC 2017
On Thu, Feb 2, 2017 at 10:32 AM, Numan Siddique <nusiddiq at redhat.com> wrote:
> On Wed, Feb 1, 2017 at 4:31 AM, Ben Pfaff <blp at ovn.org> wrote:
>
> > On Sun, Jan 15, 2017 at 12:36:09PM +0530, Numan Siddique wrote:
> > > ovn-controller by default enables tunnel encapsulation checksums
> > > for geneve tunnels. With this patch user can set the desired value
> > > in Open_vSwitch.external_ids:ovn_encap_csum.
> > >
> > > This option will be useful in cases where enabling tunnel
> > > encapsulation checksums incur significant performance loss due to
> > > limitations in checksum offloading capabilities of the nics.
> > >
> > > Signed-off-by: Numan Siddique <nusiddiq at redhat.com>
> >
> > Thanks. I applied this to master.
> >
>
> Thanks Ben for the review and applying it. Can this be applied to branch
> 2.7 as well ?
>
I'm supportive of backporting it.
Just to elaborate on why we care ...
With checksums enabled, we are not able to make use of the
tx-udp_tnl-segmentation offload. We will be able to leave checksums
enabled once we also have support for tx-udp_tnl-csum-segmentation, which
is present in the upstream kernel but not the RHEL kernel yet. So, right
now to get the best overall performance, we have to disable checksums (for
current CentOS / RHEL, anyway).
--
Russell Bryant
More information about the dev
mailing list