d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

net/sfc: improve logging in MAE backend of flow API

Browse Source 
Errors detected during parsing of pattern items and actions
are reflected by setting RTE error, but the name of the bad
element is not disclosed, thus leaving the user to join the
dots themselves. Adjust the code to log missing information.

Signed-off-by: Ivan Malov <ivan.malov@oktetlabs.ru>
Reviewed-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
Reviewed-by: Andy Moreton <amoreton@xilinx.com>
This commit is contained in:
Ivan Malov 2021-07-21 07:15:05 +03:00 committed by Thomas Monjalon
parent f1893715ce
commit 73b91412ca
3 changed files with 62 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
drivers/net/sfc/sfc_flow.c
View File

@ -1128,6 +1128,7 @@ sfc_flow_parse_pppoex(const struct rte_flow_item *item,
static const struct sfc_flow_item sfc_flow_items[] = {
{
.type = RTE_FLOW_ITEM_TYPE_VOID,
.name = "VOID",
.prev_layer = SFC_FLOW_ITEM_ANY_LAYER,
.layer = SFC_FLOW_ITEM_ANY_LAYER,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1135,6 +1136,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_ETH,
.name = "ETH",
.prev_layer = SFC_FLOW_ITEM_START_LAYER,
.layer = SFC_FLOW_ITEM_L2,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1142,6 +1144,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_VLAN,
.name = "VLAN",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L2,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1149,6 +1152,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_PPPOED,
.name = "PPPOED",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L2,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1156,6 +1160,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_PPPOES,
.name = "PPPOES",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L2,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1163,6 +1168,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_IPV4,
.name = "IPV4",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L3,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1170,6 +1176,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_IPV6,
.name = "IPV6",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L3,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1177,6 +1184,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_TCP,
.name = "TCP",
.prev_layer = SFC_FLOW_ITEM_L3,
.layer = SFC_FLOW_ITEM_L4,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1184,6 +1192,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_UDP,
.name = "UDP",
.prev_layer = SFC_FLOW_ITEM_L3,
.layer = SFC_FLOW_ITEM_L4,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1191,6 +1200,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_VXLAN,
.name = "VXLAN",
.prev_layer = SFC_FLOW_ITEM_L4,
.layer = SFC_FLOW_ITEM_START_LAYER,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1198,6 +1208,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_GENEVE,
.name = "GENEVE",
.prev_layer = SFC_FLOW_ITEM_L4,
.layer = SFC_FLOW_ITEM_START_LAYER,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1205,6 +1216,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_NVGRE,
.name = "NVGRE",
.prev_layer = SFC_FLOW_ITEM_L3,
.layer = SFC_FLOW_ITEM_START_LAYER,
.ctx_type = SFC_FLOW_PARSE_CTX_FILTER,
@ -1300,7 +1312,8 @@ sfc_flow_get_item(const struct sfc_flow_item *items,
}
int
sfc_flow_parse_pattern(const struct sfc_flow_item *flow_items,
sfc_flow_parse_pattern(struct sfc_adapter *sa,
const struct sfc_flow_item *flow_items,
unsigned int nb_flow_items,
const struct rte_flow_item pattern[],
struct sfc_flow_parse_ctx *parse_ctx,
@ -1384,8 +1397,11 @@ sfc_flow_parse_pattern(const struct sfc_flow_item *flow_items,
}
rc = item->parse(pattern, parse_ctx, error);
if (rc != 0)
if (rc != 0) {
sfc_err(sa, "failed to parse item %s: %s",
item->name, strerror(-rc));
return rc;
}
if (item->layer != SFC_FLOW_ITEM_ANY_LAYER)
prev_layer = item->layer;
@ -2483,7 +2499,7 @@ sfc_flow_parse_rte_to_filter(struct rte_eth_dev *dev,
ctx.type = SFC_FLOW_PARSE_CTX_FILTER;
ctx.filter = &spec_filter->template;
rc = sfc_flow_parse_pattern(sfc_flow_items, RTE_DIM(sfc_flow_items),
rc = sfc_flow_parse_pattern(sa, sfc_flow_items, RTE_DIM(sfc_flow_items),
pattern, &ctx, error);
if (rc != 0)
goto fail_bad_value;

8
drivers/net/sfc/sfc_flow.h
View File

@ -136,13 +136,17 @@ typedef int (sfc_flow_item_parse)(const struct rte_flow_item *item,
struct sfc_flow_item {
enum rte_flow_item_type type; /* Type of item */
const char *name; /* Item name */
enum sfc_flow_item_layers layer; /* Layer of item */
enum sfc_flow_item_layers prev_layer; /* Previous layer of item */
enum sfc_flow_parse_ctx_type ctx_type; /* Parse context type */
sfc_flow_item_parse *parse; /* Parsing function */
};
int sfc_flow_parse_pattern(const struct sfc_flow_item *flow_items,
struct sfc_adapter;
int sfc_flow_parse_pattern(struct sfc_adapter *sa,
const struct sfc_flow_item *flow_items,
unsigned int nb_flow_items,
const struct rte_flow_item pattern[],
struct sfc_flow_parse_ctx *parse_ctx,
@ -156,8 +160,6 @@ int sfc_flow_parse_init(const struct rte_flow_item *item,
unsigned int size,
struct rte_flow_error *error);
struct sfc_adapter;
void sfc_flow_init(struct sfc_adapter *sa);
void sfc_flow_fini(struct sfc_adapter *sa);
int sfc_flow_start(struct sfc_adapter *sa);

39
drivers/net/sfc/sfc_mae.c
View File

@ -1985,6 +1985,7 @@ sfc_mae_rule_parse_item_tunnel(const struct rte_flow_item *item,
static const struct sfc_flow_item sfc_flow_items[] = {
{
.type = RTE_FLOW_ITEM_TYPE_PORT_ID,
.name = "PORT_ID",
/*
* In terms of RTE flow, this item is a META one,
* and its position in the pattern is don't care.
@ -1996,6 +1997,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_PHY_PORT,
.name = "PHY_PORT",
/*
* In terms of RTE flow, this item is a META one,
* and its position in the pattern is don't care.
@ -2007,6 +2009,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_PF,
.name = "PF",
/*
* In terms of RTE flow, this item is a META one,
* and its position in the pattern is don't care.
@ -2018,6 +2021,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_VF,
.name = "VF",
/*
* In terms of RTE flow, this item is a META one,
* and its position in the pattern is don't care.
@ -2029,6 +2033,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_ETH,
.name = "ETH",
.prev_layer = SFC_FLOW_ITEM_START_LAYER,
.layer = SFC_FLOW_ITEM_L2,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2036,6 +2041,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_VLAN,
.name = "VLAN",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L2,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2043,6 +2049,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_IPV4,
.name = "IPV4",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L3,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2050,6 +2057,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_IPV6,
.name = "IPV6",
.prev_layer = SFC_FLOW_ITEM_L2,
.layer = SFC_FLOW_ITEM_L3,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2057,6 +2065,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_TCP,
.name = "TCP",
.prev_layer = SFC_FLOW_ITEM_L3,
.layer = SFC_FLOW_ITEM_L4,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2064,6 +2073,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_UDP,
.name = "UDP",
.prev_layer = SFC_FLOW_ITEM_L3,
.layer = SFC_FLOW_ITEM_L4,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2071,6 +2081,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_VXLAN,
.name = "VXLAN",
.prev_layer = SFC_FLOW_ITEM_L4,
.layer = SFC_FLOW_ITEM_START_LAYER,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2078,6 +2089,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_GENEVE,
.name = "GENEVE",
.prev_layer = SFC_FLOW_ITEM_L4,
.layer = SFC_FLOW_ITEM_START_LAYER,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2085,6 +2097,7 @@ static const struct sfc_flow_item sfc_flow_items[] = {
},
{
.type = RTE_FLOW_ITEM_TYPE_NVGRE,
.name = "NVGRE",
.prev_layer = SFC_FLOW_ITEM_L3,
.layer = SFC_FLOW_ITEM_START_LAYER,
.ctx_type = SFC_FLOW_PARSE_CTX_MAE,
@ -2293,7 +2306,7 @@ sfc_mae_rule_parse_pattern(struct sfc_adapter *sa,
if (rc != 0)
goto fail_encap_parse_init;
rc = sfc_flow_parse_pattern(sfc_flow_items, RTE_DIM(sfc_flow_items),
rc = sfc_flow_parse_pattern(sa, sfc_flow_items, RTE_DIM(sfc_flow_items),
pattern, &ctx, error);
if (rc != 0)
goto fail_parse_pattern;
@ -2926,6 +2939,22 @@ sfc_mae_rule_parse_action_port_id(struct sfc_adapter *sa,
return rc;
}
static const char * const action_names[] = {
[RTE_FLOW_ACTION_TYPE_VXLAN_DECAP] = "VXLAN_DECAP",
[RTE_FLOW_ACTION_TYPE_OF_POP_VLAN] = "OF_POP_VLAN",
[RTE_FLOW_ACTION_TYPE_OF_PUSH_VLAN] = "OF_PUSH_VLAN",
[RTE_FLOW_ACTION_TYPE_OF_SET_VLAN_VID] = "OF_SET_VLAN_VID",
[RTE_FLOW_ACTION_TYPE_OF_SET_VLAN_PCP] = "OF_SET_VLAN_PCP",
[RTE_FLOW_ACTION_TYPE_VXLAN_ENCAP] = "VXLAN_ENCAP",
[RTE_FLOW_ACTION_TYPE_FLAG] = "FLAG",
[RTE_FLOW_ACTION_TYPE_MARK] = "MARK",
[RTE_FLOW_ACTION_TYPE_PHY_PORT] = "PHY_PORT",
[RTE_FLOW_ACTION_TYPE_PF] = "PF",
[RTE_FLOW_ACTION_TYPE_VF] = "VF",
[RTE_FLOW_ACTION_TYPE_PORT_ID] = "PORT_ID",
[RTE_FLOW_ACTION_TYPE_DROP] = "DROP",
};
static int
sfc_mae_rule_parse_action(struct sfc_adapter *sa,
const struct rte_flow_action *action,
@ -3024,6 +3053,14 @@ sfc_mae_rule_parse_action(struct sfc_adapter *sa,
if (rc == 0) {
bundle->actions_mask |= (1ULL << action->type);
} else if (!custom_error) {
if (action->type < RTE_DIM(action_names)) {
const char *action_name = action_names[action->type];
if (action_name != NULL) {
sfc_err(sa, "action %s was rejected: %s",
action_name, strerror(rc));
}
}
rc = rte_flow_error_set(error, rc, RTE_FLOW_ERROR_TYPE_ACTION,
NULL, "Failed to request the action");
}