[ovs-dev] [PATCH v5 05/12] dpif-netdev: Remove useless dpif-dummy/delete-port appctl.

Daniele Di Proietto diproiettod at vmware.com
Wed Mar 23 18:37:01 UTC 2016


It is only used in the testsuite and it can be replaced by a dpctl
command.

Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Tested-by: Ilya Maximets <i.maximets at samsung.com>
Acked-by: Ilya Maximets <i.maximets at samsung.com>
---
 lib/dpif-netdev.c | 33 ---------------------------------
 tests/bridge.at   |  4 ++--
 2 files changed, 2 insertions(+), 35 deletions(-)

diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
index c87eed7..342476a 100644
--- a/lib/dpif-netdev.c
+++ b/lib/dpif-netdev.c
@@ -3985,37 +3985,6 @@ exit:
 }
 
 static void
-dpif_dummy_delete_port(struct unixctl_conn *conn, int argc OVS_UNUSED,
-                       const char *argv[], void *aux OVS_UNUSED)
-{
-    struct dp_netdev_port *port;
-    struct dp_netdev *dp;
-
-    ovs_mutex_lock(&dp_netdev_mutex);
-    dp = shash_find_data(&dp_netdevs, argv[1]);
-    if (!dp || !dpif_netdev_class_is_dummy(dp->class)) {
-        ovs_mutex_unlock(&dp_netdev_mutex);
-        unixctl_command_reply_error(conn, "unknown datapath or not a dummy");
-        return;
-    }
-    ovs_refcount_ref(&dp->ref_cnt);
-    ovs_mutex_unlock(&dp_netdev_mutex);
-
-    ovs_mutex_lock(&dp->port_mutex);
-    if (get_port_by_name(dp, argv[2], &port)) {
-        unixctl_command_reply_error(conn, "unknown port");
-    } else if (port->port_no == ODPP_LOCAL) {
-        unixctl_command_reply_error(conn, "can't delete local port");
-    } else {
-        do_del_port(dp, port);
-        unixctl_command_reply(conn, NULL);
-    }
-    ovs_mutex_unlock(&dp->port_mutex);
-
-    dp_netdev_unref(dp);
-}
-
-static void
 dpif_dummy_register__(const char *type)
 {
     struct dpif_class *class;
@@ -4063,8 +4032,6 @@ dpif_dummy_register(enum dummy_level level)
     unixctl_command_register("dpif-dummy/change-port-number",
                              "dp port new-number",
                              3, 3, dpif_dummy_change_port_number, NULL);
-    unixctl_command_register("dpif-dummy/delete-port", "dp port",
-                             2, 2, dpif_dummy_delete_port, NULL);
 }
 
 /* Datapath Classifier. */
diff --git a/tests/bridge.at b/tests/bridge.at
index 0561186..37c55ba 100644
--- a/tests/bridge.at
+++ b/tests/bridge.at
@@ -17,9 +17,9 @@ dummy at ovs-dummy: hit:0 missed:0
 		p2 2/2: (dummy)
 ])
 
-# Delete p1 from the datapath as if by "ovs-dpctl del-port"
+# Delete p1 from the datapath with "ovs-dpctl del-if"
 # and check that it disappeared.
-AT_CHECK([ovs-appctl dpif-dummy/delete-port ovs-dummy p1])
+AT_CHECK([ovs-appctl dpctl/del-if dummy at ovs-dummy p1])
 AT_CHECK([ovs-appctl dpif/show], [0], [dnl
 dummy at ovs-dummy: hit:0 missed:0
 	br0:
-- 
2.1.4




More information about the dev mailing list