mir/ovs
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-08-31 06:15:47 +00:00
Code Issues Releases Wiki Activity

odp-util: Make slow_path_reasons mutually exclusive.

Browse Source 
It's no longer possible for a single datapath flow to be slow
pathed for two different reasons.  This patch updates the code to
reflect this fact (marginally simplifying it).

Signed-off-by: Ethan Jackson <ethan@nicira.com>
This commit is contained in:
Ethan Jackson
2013-05-28 11:43:43 -07:00
parent 454a77e5b4
commit 98f0520fb2
3 changed files with 63 additions and 53 deletions
Download Patch File Download Diff File Expand all files Collapse all files

53
lib/odp-util.c
View File

@@ -170,11 +170,9 @@ format_odp_sample_action(struct ds *ds, const struct nlattr *attr)
}
static const char *
slow_path_reason_to_string(uint32_t data)
slow_path_reason_to_string(enum slow_path_reason reason)
{
enum slow_path_reason bit = (enum slow_path_reason) data;
switch (bit) {
switch (reason) {
case SLOW_CFM:
return "cfm";
case SLOW_LACP:
@@ -185,11 +183,26 @@ slow_path_reason_to_string(uint32_t data)
return "bfd";
case SLOW_CONTROLLER:
return "controller";
case __SLOW_MAX:
default:
return NULL;
}
}
static enum slow_path_reason
string_to_slow_path_reason(const char *string)
{
enum slow_path_reason i;
for (i = 1; i < __SLOW_MAX; i++) {
if (!strcmp(string, slow_path_reason_to_string(i))) {
return i;
}
}
return 0;
}
static int
parse_flags(const char *s, const char *(*bit_to_string)(uint32_t),
uint32_t *res)
@@ -284,10 +297,10 @@ format_odp_userspace_action(struct ds *ds, const struct nlattr *attr)
cookie.sflow.output);
} else if (userdata_len == sizeof cookie.slow_path
&& cookie.type == USER_ACTION_COOKIE_SLOW_PATH) {
ds_put_cstr(ds, ",slow_path(");
format_flags(ds, slow_path_reason_to_string,
cookie.slow_path.reason, ',');
ds_put_format(ds, ")");
const char *reason;
reason = slow_path_reason_to_string(cookie.slow_path.reason);
reason = reason ? reason : "";
ds_put_format(ds, ",slow_path(%s)", reason);
} else if (userdata_len == sizeof cookie.flow_sample
&& cookie.type == USER_ACTION_COOKIE_FLOW_SAMPLE) {
ds_put_format(ds, ",flow_sample(probability=%"PRIu16
@@ -498,25 +511,27 @@ parse_odp_action(const char *s, const struct simap *port_names,
odp_put_userspace_action(pid, &cookie, sizeof cookie.sflow,
actions);
return n;
} else if (sscanf(s, "userspace(pid=%lli,slow_path%n", &pid, &n) > 0
} else if (sscanf(s, "userspace(pid=%lli,slow_path(%n", &pid, &n) > 0
&& n > 0) {
union user_action_cookie cookie;
int res;
char reason[32];
if (s[n] == ')' && s[n + 1] == ')') {
reason[0] = '\0';
n += 2;
} else if (sscanf(s + n, "%31[^)]))", reason) > 0) {
n += strlen(reason) + 2;
} else {
return -EINVAL;
}
cookie.type = USER_ACTION_COOKIE_SLOW_PATH;
cookie.slow_path.unused = 0;
cookie.slow_path.reason = 0;
cookie.slow_path.reason = string_to_slow_path_reason(reason);
res = parse_flags(&s[n], slow_path_reason_to_string,
&cookie.slow_path.reason);
if (res < 0) {
return res;
}
n += res;
if (s[n] != ')') {
if (reason[0] && !cookie.slow_path.reason) {
return -EINVAL;
}
n++;
odp_put_userspace_action(pid, &cookie, sizeof cookie.slow_path,
actions);