When offloading rule, tc should be filled with police index, instead
of meter id. As meter is mapped to police action, and the mapping is
inserted into meter_id_to_police_idx hmap, this hmap is used to find
the police index. Besides, an action cookie is added to save meter id
on rule creation, so meter id can be retrieved from the cookie, and
pass to dpif while dumping rules.
Signed-off-by: Jianbo Liu <jian...@nvidia.com>
---
lib/netdev-offload-tc.c | 16 +++++++++++++++-
lib/tc.c | 25 ++++++++++++++++++++++++-
2 files changed, 39 insertions(+), 2 deletions(-)
diff --git a/lib/netdev-offload-tc.c b/lib/netdev-offload-tc.c
index d6ef5159c..afe566800 100644
--- a/lib/netdev-offload-tc.c
+++ b/lib/netdev-offload-tc.c
@@ -150,6 +150,8 @@ static struct netlink_field set_flower_map[][4] = {
},
};
+static int meter_id_lookup(uint32_t meter_id, uint32_t *police_idx);
+
static struct ovs_mutex ufid_lock = OVS_MUTEX_INITIALIZER;
/**
@@ -1019,7 +1021,8 @@ parse_tc_flower_to_match(struct tc_flower *flower,
}
break;
case TC_ACT_POLICE: {
- /* Not supported yet */
+ nl_msg_put_u32(buf, OVS_ACTION_ATTR_METER,
+ action->police.meter_id);
}
break;
}
@@ -1936,6 +1939,17 @@ netdev_tc_flow_put(struct netdev *netdev, struct match
*match,
action->type = TC_ACT_GOTO;
action->chain = 0; /* 0 is reserved and not used by recirc. */
flower.action_count++;
+ } else if (nl_attr_type(nla) == OVS_ACTION_ATTR_METER) {
+ uint32_t police_index;
+
+ action->type = TC_ACT_POLICE;
+ action->police.meter_id = nl_attr_get_u32(nla);
+ if (meter_id_lookup(action->police.meter_id, &police_index)) {
+ return EOPNOTSUPP;
+ }
+
+ action->police.index = police_index;
+ flower.action_count++;
} else {
VLOG_DBG_RL(&rl, "unsupported put action type: %d",
nl_attr_type(nla));
diff --git a/lib/tc.c b/lib/tc.c
index 16d4ae3da..ecdcb676d 100644
--- a/lib/tc.c
+++ b/lib/tc.c
@@ -2421,6 +2421,22 @@ nl_msg_put_act_gact(struct ofpbuf *request, uint32_t
chain)
nl_msg_end_nested(request, offset);
}
+static void
+nl_msg_put_act_police_index(struct ofpbuf *request, uint32_t police_idx)
+{
+ struct tc_police police;
+ size_t offset;
+
+ memset(&police, 0, sizeof police);
+ police.index = police_idx;
+
+ nl_msg_put_string(request, TCA_ACT_KIND, "police");
+ offset = nl_msg_start_nested(request, TCA_ACT_OPTIONS);
+ nl_msg_put_unspec(request, TCA_POLICE_TBF, &police, sizeof police);
+ nl_msg_put_u32(request, TCA_POLICE_RESULT, TC_ACT_PIPE);
+ nl_msg_end_nested(request, offset);
+}
+
static void
nl_msg_put_act_ct(struct ofpbuf *request, struct tc_action *action)
{
@@ -2911,7 +2927,14 @@ nl_msg_put_flower_acts(struct ofpbuf *request, struct
tc_flower *flower)
}
break;
case TC_ACT_POLICE: {
- /* Not supported yet */
+ struct tc_cookie act_cookie;
+
+ act_offset = nl_msg_start_nested(request, act_index++);
+ nl_msg_put_act_police_index(request, action->police.index);
+ act_cookie.data = &action->police.meter_id;
+ act_cookie.len = sizeof(action->police.meter_id);
+ nl_msg_put_act_cookie(request, &act_cookie);
+ nl_msg_end_nested(request, act_offset);
}
break;
}
--
2.26.2
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
