From da1253be2ed4bc2ff43d8eb2e2f57c4abdbd5e5f Mon Sep 17 00:00:00 2001
Message-Id: <da1253be2ed4bc2ff43d8eb2e2f57c4abdbd5e5f.1568637354.git.lorenzo.bianconi@redhat.com>
In-Reply-To: <3624362cb395b6bc90f6fd69e12988979db95b7e.1568637354.git.lorenzo.bianconi@redhat.com>
References: <3624362cb395b6bc90f6fd69e12988979db95b7e.1568637354.git.lorenzo.bianconi@redhat.com>
From: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
Date: Tue, 3 Sep 2019 16:22:52 +0200
Subject: [PATCH ovn 2/3] add meter support to trigger_event action
Introduce meter support to trigger_event action in order to not
overload pinctrl thread under heavy load
Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
Signed-off-by: Mark Michelson <mmichels@redhat.com>
Acked-by: Mark Michelson <mmichels@redhat.com>
---
include/ovn/actions.h | 1 +
ovn/lib/actions.c | 43 +++++++++++++++++++++++++++++++++++++------
ovn/ovn-sb.xml | 5 ++++-
tests/ovn.at | 4 ++++
4 files changed, 46 insertions(+), 7 deletions(-)
--- a/include/ovn/actions.h
+++ b/include/ovn/actions.h
@@ -327,6 +327,7 @@ struct ovnact_controller_event {
int event_type; /* controller event type */
struct ovnact_gen_option *options;
size_t n_options;
+ char *meter;
};
/* OVNACT_BIND_VPORT. */
--- a/ovn/lib/actions.c
+++ b/ovn/lib/actions.c
@@ -1273,6 +1273,9 @@ format_TRIGGER_EVENT(const struct ovnact
{
ds_put_format(s, "trigger_event(event = \"%s\"",
event_to_string(event->event_type));
+ if (event->meter) {
+ ds_put_format(s, ", meter = \"%s\"", event->meter);
+ }
for (const struct ovnact_gen_option *o = event->options;
o < &event->options[event->n_options]; o++) {
ds_put_cstr(s, ", ");
@@ -1420,10 +1423,21 @@ encode_TRIGGER_EVENT(const struct ovnact
const struct ovnact_encode_params *ep OVS_UNUSED,
struct ofpbuf *ofpacts)
{
+ uint32_t meter_id = NX_CTLR_NO_METER;
size_t oc_offset;
+ if (event->meter) {
+ meter_id = ovn_extend_table_assign_id(ep->meter_table, event->meter,
+ ep->lflow_uuid);
+ if (meter_id == EXT_TABLE_ID_INVALID) {
+ VLOG_WARN("Unable to assign id for trigger meter: %s",
+ event->meter);
+ return;
+ }
+ }
+
oc_offset = encode_start_controller_op(ACTION_OPCODE_EVENT, false,
- NX_CTLR_NO_METER, ofpacts);
+ meter_id, ofpacts);
ovs_be32 ofs = htonl(event->event_type);
ofpbuf_put(ofpacts, &ofs, sizeof ofs);
@@ -1738,11 +1752,27 @@ parse_trigger_event(struct action_contex
sizeof *event->options);
}
- struct ovnact_gen_option *o = &event->options[event->n_options++];
- memset(o, 0, sizeof *o);
- parse_gen_opt(ctx, o,
- &ctx->pp->controller_event_opts->event_opts[event_type],
- event_to_string(event_type));
+ if (lexer_match_id(ctx->lexer, "meter")) {
+ if (!lexer_force_match(ctx->lexer, LEX_T_EQUALS)) {
+ return;
+ }
+ /* If multiple meters are given, use the most recent. */
+ if (ctx->lexer->token.type == LEX_T_STRING &&
+ strlen(ctx->lexer->token.s)) {
+ free(event->meter);
+ event->meter = xstrdup(ctx->lexer->token.s);
+ } else if (ctx->lexer->token.type != LEX_T_STRING) {
+ lexer_syntax_error(ctx->lexer, "expecting string");
+ return;
+ }
+ lexer_get(ctx->lexer);
+ } else {
+ struct ovnact_gen_option *o = &event->options[event->n_options++];
+ memset(o, 0, sizeof *o);
+ parse_gen_opt(ctx, o,
+ &ctx->pp->controller_event_opts->event_opts[event_type],
+ event_to_string(event_type));
+ }
if (ctx->lexer->error) {
return;
}
@@ -1763,6 +1793,7 @@ static void
ovnact_controller_event_free(struct ovnact_controller_event *event)
{
free_gen_options(event->options, event->n_options);
+ free(event->meter);
}
static void
--- a/ovn/ovn-sb.xml
+++ b/ovn/ovn-sb.xml
@@ -1994,7 +1994,9 @@ tcp.flags = RST;
<p>
This action is used to allow ovs-vswitchd to report CMS related
events writing them in <ref table="Controller_Event"/> table.
- Supported event:
+ It is possible to associate a meter to a each event in order to
+ not overload pinctrl thread under heavy load; each meter is
+ identified though a defined naming convention. Supported events:
</p>
<ul>
@@ -2005,6 +2007,7 @@ tcp.flags = RST;
no configured backend destinations. For this event, the event
info includes the load balancer VIP, the load balancer UUID,
and the transport protocol.
+ Associated meter: <code>event-elb</code>
</p>
</li>
</ul>
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -1363,6 +1363,10 @@ tcp_reset { };
trigger_event(event = "empty_lb_backends", vip = "10.0.0.1:80", protocol = "tcp", load_balancer = "12345678-abcd-9876-fedc-11119f8e7d6c");
encodes as controller(userdata=00.00.00.0f.00.00.00.00.00.00.00.00.00.01.00.0b.31.30.2e.30.2e.30.2e.31.3a.38.30.00.02.00.03.74.63.70.00.03.00.24.31.32.33.34.35.36.37.38.2d.61.62.63.64.2d.39.38.37.36.2d.66.65.64.63.2d.31.31.31.31.39.66.38.65.37.64.36.63)
+trigger_event(event = "empty_lb_backends", meter="event-elb" vip = "10.0.0.1:80", protocol = "tcp", load_balancer = "12345678-abcd-9876-fedc-11119f8e7d6c");
+ formats as trigger_event(event = "empty_lb_backends", meter = "event-elb", vip = "10.0.0.1:80", protocol = "tcp", load_balancer = "12345678-abcd-9876-fedc-11119f8e7d6c");
+ encodes as controller(userdata=00.00.00.0f.00.00.00.00.00.00.00.00.00.01.00.0b.31.30.2e.30.2e.30.2e.31.3a.38.30.00.02.00.03.74.63.70.00.03.00.24.31.32.33.34.35.36.37.38.2d.61.62.63.64.2d.39.38.37.36.2d.66.65.64.63.2d.31.31.31.31.39.66.38.65.37.64.36.63,meter_id=5)
+
# Testing invalid vip results in extra error messages from socket-util.c
trigger_event(event = "empty_lb_backends", vip = "10.0.0.1:80", protocol = "sctp", load_balancer = "12345678-abcd-9876-fedc-11119f8e7d6c");
Load balancer protocol 'sctp' is not 'tcp' or 'udp'