On 8 August 2017 at 18:16, Andy Zhou <[email protected]> wrote: > On Tue, Aug 8, 2017 at 5:10 PM, Joe Stringer <[email protected]> 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 <[email protected]> >> Signed-off-by: Joe Stringer <[email protected]> > > A few minor comments inline. Otherwise, > > Acked-by: Andy Zhou <[email protected]>
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 [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
