Keep ABI consistent after we replacing EBUSY with EEXIST as
return error in datapath.c:ovs_vport_cmd_new. And cry out louder
when failed in any reason.
Signed-off-by: Fan Du <fan...@windriver.com>
---
lib/dpif-linux.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/lib/dpif-linux.c b/lib/dpif-linux.c
index ac20ae7..9fe0e67 100644
--- a/lib/dpif-linux.c
+++ b/lib/dpif-linux.c
@@ -537,7 +537,8 @@ dpif_linux_port_add(struct dpif *dpif_, struct netdev
*netdev,
VLOG_DBG("%s: assigning port %"PRIu32" to netlink pid %"PRIu32,
dpif_name(dpif_), reply.port_no, upcall_pid);
} else {
- if (error == EBUSY && *port_nop != UINT32_MAX) {
+ VLOG_WARN("%s: cannot create port `%s'\n", dpif_name(dpif_), name);
+ if (error == EEXIST && *port_nop != UINT32_MAX) {
VLOG_INFO("%s: requested port %"PRIu32" is in use",
dpif_name(dpif_), *port_nop);
}
--
1.7.1
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
