[ovs-dev] [monitor 2/4] ofproto: Discontinue use of netdev_monitor.

Ethan Jackson ethan at nicira.com
Tue May 31 21:32:43 UTC 2011


On Fri, May 27, 2011 at 09:33, Ben Pfaff <blp at nicira.com> 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



More information about the dev mailing list