From 046f1f89e6d7716581de207dd0c54421926bc25b Mon Sep 17 00:00:00 2001 From: Ethan Jackson Date: Mon, 21 May 2012 13:20:18 -0700 Subject: [PATCH] bridge: Ignore "null" interfaces as required. Commit bae7208e91a0 (bridge: Refactor bridge_reconfigure().) introduced a regression in which the switch would attempt to instantiate "null" interfaces in the datapath. This would, of course, fail and trigger a warning. Though harmless, these warnings confused users. Signed-off-by: Ethan Jackson --- vswitchd/bridge.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index 13eb80bd..0c78f946 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -2545,11 +2545,12 @@ bridge_add_del_ports(struct bridge *br, for (i = 0; i < port->n_interfaces; i++) { const struct ovsrec_interface *cfg = port->interfaces[i]; struct iface *iface = iface_lookup(br, cfg->name); + const char *type = iface_get_type(cfg, br->cfg); if (iface) { iface->cfg = cfg; - iface->type = iface_get_type(cfg, br->cfg); - } else { + iface->type = type; + } else if (strcmp(type, "null")) { bridge_queue_if_cfg(br, cfg, port); } } -- 2.30.2