Re: [ovs-dev] [port reconf 7/7] vswitchd: Clean up iface_create().

2012-04-24 Thread Ben Pfaff
Thanks, I pushed this to master. When I get a clean build from the autobuilder I'll cherry-pick to branch-1.6. On Tue, Apr 24, 2012 at 05:52:00PM -0700, Ethan Jackson wrote: > Much cleaner, looks good, thanks. > > Assuming patch 6/7 is the one I sent out to the list, I consider this > series rev

Re: [ovs-dev] [port reconf 7/7] vswitchd: Clean up iface_create().

2012-04-24 Thread Ethan Jackson
Much cleaner, looks good, thanks. Assuming patch 6/7 is the one I sent out to the list, I consider this series reviewed. Ethan On Tue, Apr 24, 2012 at 17:08, Ben Pfaff wrote: > iface_create() did its work in an order that meant it had to do a lot more > cleanup on error paths than is otherwise

[ovs-dev] [port reconf 7/7] vswitchd: Clean up iface_create().

2012-04-24 Thread Ben Pfaff
iface_create() did its work in an order that meant it had to do a lot more cleanup on error paths than is otherwise needed. This commit reorders the work to avoid this extra cleanup. bridge_ofproto_port_del() is no longer used after the refactoring so this commit deletes it. Signed-off-by: Ben P