Re: [ovs-dev] [PATCH ovn] ovn-controller: Add physical/logical translation for containers.

2015-05-18 Thread Ben Pfaff
On Mon, May 18, 2015 at 07:32:01AM -0700, Gurucharan Shetty wrote: > > > >> +int tag = 0; > >> +ofp_port_t ofport; > >> +if (binding->parent_port) { > >> +ofport = u16_to_ofp(simap_get(&lport_to_ofport, > >> + binding->par

Re: [ovs-dev] [PATCH ovn] ovn-controller: Add physical/logical translation for containers.

2015-05-18 Thread Gurucharan Shetty
> >> +int tag = 0; >> +ofp_port_t ofport; >> +if (binding->parent_port) { >> +ofport = u16_to_ofp(simap_get(&lport_to_ofport, >> + binding->parent_port)); >> +if (ofport) { > > I don't think that it is safe to

Re: [ovs-dev] [PATCH ovn] ovn-controller: Add physical/logical translation for containers.

2015-05-17 Thread Ben Pfaff
On Wed, May 13, 2015 at 07:51:21AM -0700, Gurucharan Shetty wrote: > When containers are running inside VMs and the openflow flows > are added in the hypervisor, the physical to logical translation > (and vice versa) needs to handle the VLAN tags that the packet > comes with. Thanks for implementi

[ovs-dev] [PATCH ovn] ovn-controller: Add physical/logical translation for containers.

2015-05-13 Thread Gurucharan Shetty
When containers are running inside VMs and the openflow flows are added in the hypervisor, the physical to logical translation (and vice versa) needs to handle the VLAN tags that the packet comes with. Signed-off-by: Gurucharan Shetty --- ovn/controller/physical.c | 61