On Mon, 22 Oct 2018 at 09:16, Roi Dayan <[email protected]> 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 <[email protected]> > >> > >> 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 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
