[ovs-dev] [PATCH 0/2] Add support to offload GRE tunnels

Simon Horman simon.horman at netronome.com
Mon Oct 22 07:53:48 UTC 2018


On Mon, 22 Oct 2018 at 09:16, Roi Dayan <roid at mellanox.com> wrote:

>
>
> On 14/10/2018 10:26, Roi Dayan wrote:
> >
> >
> > On 12/10/2018 13:05, Simon Horman wrote:
> >> On Thu, Oct 11, 2018 at 10:06:41AM +0300, Roi Dayan wrote:
> >>> Hi,
> >>>
> >>> The first patch is to add support to offload to GRE tunnels using tc.
> >>> THe second patch is to offload the tunnel csum option correctly.
> >>
> >> Thanks Roi,
> >>
> >> this looks good to me:
> >>
> >> Reviewed-by: Simon Horman <simon.horman at netronome.com>
> >>
> >> Let me see if I can get master clean wrt travis-ci so I can test
> >> your patchset there before applying.
> >>
> >
> > Thanks
> >
> > we also tested using travis and noticed a test failure
> > that reproduced on master branch.
> > we didnt get a chance to deep dive and help there,
> > but looks like our patches dont add more failures.
> >
> > travis log with our patches
> > https://travis-ci.org/roidayan/ovs/builds/439526807
> >
> > travis log with master
> > https://travis-ci.org/roidayan/ovs/builds/439523018
> >
>
>
> Hi Simon,
>
> I noticed the second patch in the series is merged but not the first.
>
> Thanks,
> Roi
>

Hi Roi,

I am a little confused.

In the master branch I see:

bafb398bf6df dpdk: Use DPDK 17.11.4 release.
...
d9677a1f0eaf netdev-tc-offloads: TC csum option is not matched with tunnel
configuration
5e63eaa969a3 netdev-vport: Make gre netdev type to use TC rules


More information about the dev mailing list