On Fri, May 27, 2011 at 09:33, Ben Pfaff <[email protected]> wrote: > The change to ofproto_run() looks like it can't be right, because it > uses 'error' and 'devname' without ever setting them. (Once that's > fixed, it can call update_port() directly instead of going through > process_port_change(), and then process_port_change() can be inlined > into ofproto_run() since it only has one caller left.)
Oops, pretty silly. I'll just call update_port() directly using netdev_get_name() instead of going through process_port_change(). Thanks, for the review. Ethan _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
