[ovs-dev] [PATCH ovn 1/4] Make is_switch() in lflow.c a util function
numans at ovn.org
numans at ovn.org
Fri Jan 24 11:02:28 UTC 2020
From: Numan Siddique <numans at ovn.org>
This patch renames is_switch() to datapath_is_switch() and moves to
lib/ovn-util.c.
Upcoming patch will make use of it.
Signed-off-by: Numan Siddique <numans at ovn.org>
---
controller/lflow.c | 11 ++---------
lib/ovn-util.c | 6 ++++++
lib/ovn-util.h | 3 ++-
3 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/controller/lflow.c b/controller/lflow.c
index 9a3c1eb5b..3ddb0571e 100644
--- a/controller/lflow.c
+++ b/controller/lflow.c
@@ -149,13 +149,6 @@ is_chassis_resident_cb(const void *c_aux_, const char *port_name)
}
}
-static bool
-is_switch(const struct sbrec_datapath_binding *ldp)
-{
- return smap_get(&ldp->external_ids, "logical-switch") != NULL;
-
-}
-
void
lflow_resource_init(struct lflow_resource_ref *lfrr)
{
@@ -794,7 +787,7 @@ consider_logical_flow(
.lookup_port = lookup_port_cb,
.tunnel_ofport = tunnel_ofport_cb,
.aux = &aux,
- .is_switch = is_switch(ldp),
+ .is_switch = datapath_is_switch(ldp),
.group_table = group_table,
.meter_table = meter_table,
.lflow_uuid = lflow->header_.uuid,
@@ -818,7 +811,7 @@ consider_logical_flow(
if (m->match.wc.masks.conj_id) {
m->match.flow.conj_id += *conj_id_ofs;
}
- if (is_switch(ldp)) {
+ if (datapath_is_switch(ldp)) {
unsigned int reg_index
= (ingress ? MFF_LOG_INPORT : MFF_LOG_OUTPORT) - MFF_REG0;
int64_t port_id = m->match.flow.regs[reg_index];
diff --git a/lib/ovn-util.c b/lib/ovn-util.c
index e67c3c073..6e0bba2c5 100644
--- a/lib/ovn-util.c
+++ b/lib/ovn-util.c
@@ -423,3 +423,9 @@ ovn_logical_flow_hash(const struct uuid *logical_datapath,
hash = hash_string(match, hash);
return hash_string(actions, hash);
}
+
+bool
+datapath_is_switch(const struct sbrec_datapath_binding *ldp)
+{
+ return smap_get(&ldp->external_ids, "logical-switch") != NULL;
+}
diff --git a/lib/ovn-util.h b/lib/ovn-util.h
index b58208e9b..422d69e5b 100644
--- a/lib/ovn-util.h
+++ b/lib/ovn-util.h
@@ -23,6 +23,7 @@ struct sbrec_logical_flow;
struct uuid;
struct eth_addr;
struct sbrec_port_binding;
+struct sbrec_datapath_binding;
struct ipv4_netaddr {
ovs_be32 addr; /* 192.168.10.123 */
@@ -86,5 +87,5 @@ uint32_t ovn_logical_flow_hash(const struct uuid *logical_datapath,
uint8_t table_id, const char *pipeline,
uint16_t priority,
const char *match, const char *actions);
-
+bool datapath_is_switch(const struct sbrec_datapath_binding *);
#endif
--
2.24.1
More information about the dev
mailing list