[ovs-dev] [PATCH v2 10/13] ovn-trace: Add support for partial UUID matches for ports and datapaths.
Ben Pfaff
blp at ovn.org
Wed May 3 15:45:57 UTC 2017
This makes ovn-trace commands easier to type.
Signed-off-by: Ben Pfaff <blp at ovn.org>
---
ovn/utilities/ovn-trace.c | 44 ++++++++++++++++++++++++++++++++++++--------
1 file changed, 36 insertions(+), 8 deletions(-)
diff --git a/ovn/utilities/ovn-trace.c b/ovn/utilities/ovn-trace.c
index 860fd4b26be0..7ad7839a4a1a 100644
--- a/ovn/utilities/ovn-trace.c
+++ b/ovn/utilities/ovn-trace.c
@@ -352,6 +352,7 @@ struct ovntrace_datapath {
struct ovntrace_port {
struct ovntrace_datapath *dp;
+ struct uuid uuid;
char *name;
char *type;
uint16_t tunnel_key;
@@ -429,15 +430,15 @@ ovntrace_datapath_find_by_sb_uuid(const struct uuid *sb_uuid)
static const struct ovntrace_datapath *
ovntrace_datapath_find_by_name(const char *name)
{
- struct uuid uuid;
- bool is_uuid = uuid_from_string(&uuid, name);
-
struct ovntrace_datapath *dp;
HMAP_FOR_EACH (dp, sb_uuid_node, &datapaths) {
- if (!strcmp(name, dp->name)
- || (is_uuid
- && (uuid_equals(&uuid, &dp->sb_uuid) ||
- uuid_equals(&uuid, &dp->nb_uuid)))) {
+ if (!strcmp(name, dp->name)) {
+ return dp;
+ }
+ }
+ HMAP_FOR_EACH (dp, sb_uuid_node, &datapaths) {
+ if (uuid_is_partial_match(&dp->sb_uuid, name) >= 4 ||
+ uuid_is_partial_match(&dp->nb_uuid, name) >= 4) {
return dp;
}
}
@@ -558,6 +559,7 @@ read_ports(void)
continue;
}
port->dp = dp;
+ port->uuid = sbpb->header_.uuid;
port->name = xstrdup(port_name);
port->type = xstrdup(sbpb->type);
port->tunnel_key = sbpb->tunnel_key;
@@ -865,6 +867,32 @@ read_db(void)
read_mac_bindings();
}
+static const struct ovntrace_port *
+ovntrace_port_lookup_by_name(const char *name)
+{
+ const struct ovntrace_port *port = shash_find_data(&ports, name);
+ if (port) {
+ return port;
+ }
+
+ if (uuid_is_partial_string(name) >= 4) {
+ struct shash_node *node;
+ SHASH_FOR_EACH (node, &ports) {
+ const struct ovntrace_port *port = node->data;
+
+ struct uuid name_uuid;
+ if (uuid_is_partial_match(&port->uuid, name) >= 4
+ || (uuid_from_string(&name_uuid, port->name)
+ && uuid_is_partial_match(&name_uuid, name) >= 4)) {
+ return port;
+ }
+ }
+
+ }
+
+ return NULL;
+}
+
static bool
ovntrace_lookup_port(const void *dp_, const char *port_name,
unsigned int *portp)
@@ -876,7 +904,7 @@ ovntrace_lookup_port(const void *dp_, const char *port_name,
return true;
}
- const struct ovntrace_port *port = shash_find_data(&ports, port_name);
+ const struct ovntrace_port *port = ovntrace_port_lookup_by_name(port_name);
if (port) {
if (port->dp == dp) {
*portp = port->tunnel_key;
--
2.10.2
More information about the dev
mailing list