On 8 August 2017 at 18:16, Andy Zhou <az...@ovn.org> wrote:
> On Tue, Aug 8, 2017 at 5:10 PM, Joe Stringer <j...@ovn.org> wrote:
>> Previously, netdev_ports_insert() would allocate and insert an
>> ifindex->odp_port mapping, but netdev_ports_remove() would never remove
>> the mapping or free the mapping structure. This patch fixes these up.
>>
>> Fixes: 32b77c316d9982("dpif: Save added ports in a port map.")
>> Reported-by: Andy Zhou <az...@ovn.org>
>> Signed-off-by: Joe Stringer <j...@ovn.org>
>
> A few minor comments inline. Otherwise,
>
> Acked-by: Andy Zhou <az...@ovn.org>

Hi Andy,

Thanks for the review. Your comments are good points, and they change
the patch enough that I figured I should just send a v2. I'll follow
up shortly.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to