Re: [ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-28 Thread Flavio Leitner via dev
On Wed, Aug 28, 2019 at 11:29:54AM -0700, Ben Pfaff wrote: > On Tue, Aug 13, 2019 at 01:34:04PM -0300, Flavio Leitner via dev wrote: > > When a packet needs to be encapsulated in userspace, the endpoint > > address needs to be resolved to fill in the headers. If it is not, > > then currently OvS

Re: [ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-28 Thread Flavio Leitner via dev
On Wed, Aug 28, 2019 at 11:29:54AM -0700, Ben Pfaff wrote: > On Tue, Aug 13, 2019 at 01:34:04PM -0300, Flavio Leitner via dev wrote: > > When a packet needs to be encapsulated in userspace, the endpoint > > address needs to be resolved to fill in the headers. If it is not, > > then currently OvS

Re: [ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-28 Thread Ben Pfaff
On Tue, Aug 13, 2019 at 01:34:04PM -0300, Flavio Leitner via dev wrote: > When a packet needs to be encapsulated in userspace, the endpoint > address needs to be resolved to fill in the headers. If it is not, > then currently OvS sends either a Neighbor Solicitation (IPv6) > or an ARP Query (IPv4)

Re: [ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-28 Thread Flavio Leitner via dev
Hi Ben and Ilya, This patch has two reviews, so any chance for you to take a look soon as well? Thanks in advance! fbl On Tue, Aug 13, 2019 at 01:34:04PM -0300, Flavio Leitner via dev wrote: > When a packet needs to be encapsulated in userspace, the endpoint > address needs to be resolved to

Re: [ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-26 Thread David Marchand
On Tue, Aug 13, 2019 at 6:35 PM Flavio Leitner via dev wrote: > > When a packet needs to be encapsulated in userspace, the endpoint > address needs to be resolved to fill in the headers. If it is not, > then currently OvS sends either a Neighbor Solicitation (IPv6) > or an ARP Query (IPv4) to

Re: [ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-14 Thread Vasu Dasari
Looks good Flavio. Acked-By: Vasu Dasari Thanks -Vasu *Vasu Dasari* On Tue, Aug 13, 2019 at 12:45 PM Flavio Leitner via dev < ovs-dev@openvswitch.org> wrote: > On Tue, Aug 13, 2019 at 01:34:04PM -0300, Flavio Leitner via dev wrote: > > When a packet needs to be encapsulated in userspace,

Re: [ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-13 Thread Flavio Leitner via dev
On Tue, Aug 13, 2019 at 01:34:04PM -0300, Flavio Leitner via dev wrote: > When a packet needs to be encapsulated in userspace, the endpoint > address needs to be resolved to fill in the headers. If it is not, > then currently OvS sends either a Neighbor Solicitation (IPv6) > or an ARP Query (IPv4)

[ovs-dev] [PATCH] tnl-neigh: Use outgoing ofproto version.

2019-08-13 Thread Flavio Leitner via dev
When a packet needs to be encapsulated in userspace, the endpoint address needs to be resolved to fill in the headers. If it is not, then currently OvS sends either a Neighbor Solicitation (IPv6) or an ARP Query (IPv4) to resolve it. The problem is that the NS/ARP packet will go through the flow