On Wed, Aug 10, 2016 at 10:09:07AM +0000, Kavanagh, Mark B wrote: > > > >On 09/08/2016 09:08, "Thadeu Lima de Souza Cascardo" <casca...@redhat.com> > >wrote: > > > >>On Tue, Aug 09, 2016 at 05:01:14PM +0100, Mark Kavanagh wrote: > >>> From: Daniele Di Proietto <diproiet...@vmware.com> > >>> > >>> Interfaces with type "internal" end up having a netdev with type "tap" > >>> in the dpif-netdev datapath, so a strcmp will fail to match internal > >>> interfaces. > >>> > >>> We can translate the types with ofproto_port_open_type() before calling > >>> strcmp to fix this. > >>> > >>> This fixes a minor issue where internal interfaces are considered > >>> non-internal in the userspace datapath for the purpose of adjusting the > >>> MTU. > >>> > >>> Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com> > >> > >>Acked-by: Thadeu Lima de Souza Cascardo <casca...@redhat.com> > > > >Thanks guys, I pushed this to master > > > >> > >>Hi, Mark. > >> > >>Can you keep my Ack in further submissions in case there are no changes to > >>this > >>patch? > > > Hi Cascardo, > > Apologies - I performed the rework for the entire series on my local branch, > so your Ack wasn't present. > > Thanks to Daniele for re-adding it. > > Cheers, > Mark
Hi, Mark. No apologies needed. Just thought it could be missed and adding it to further submissions could help with that. Thanks. Cascardo. > > >> > >>Thanks. > >>Cascardo. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev