[ovs-dev] [PATCH] vswitchd: Report actual port number, not -1, in "added interface" message.

[Kyle Mestery (kmestery)]

On Apr 24, 2012, at 12:44 PM, Ben Pfaff wrote:

> CC: Ethan Jackson <ethan at nicira.com>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> vswitchd/bridge.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
> index 077d9ee..833973a 100644
> --- a/vswitchd/bridge.c
> +++ b/vswitchd/bridge.c
> @@ -1306,7 +1306,7 @@ iface_create(struct bridge *br, struct if_cfg *if_cfg, int ofp_port)
>         error = ofproto_port_add(br->ofproto, iface->netdev, &new_ofp_port);
>         if (!error) {
>             VLOG_INFO("bridge %s: added interface %s (%d)", br->name,
> -                      iface->name, ofp_port);
> +                      iface->name, new_ofp_port);
>             iface_set_ofp_port(iface, new_ofp_port);
>         } else {
>             netdev_close(iface->netdev);
> -- 
> 1.7.2.5
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev

Nice catch, will be good to have this in the logs accurately!

Thanks,
Kyle