@@ -370,19 +370,28 @@ The flow hardware offload is disabled by default and can be enabled by::
$ ovs-vsctl set Open_vSwitch . other_config:hw-offload=true
-So far only partial flow offload is implemented. Moreover, it only works
-with PMD drivers have the rte_flow action "MARK + RSS" support.
+Matches and actions are programmed into HW to achieve full offload of
+the flow. If not all actions are supported, fallback to partial flow
+offload (offloading matches only). Moreover, it only works with PMD
+drivers that support the configured rte_flow actions.
+Partial flow offload requires support of "MARK + RSS" actions. Full
+hardware offload requires support of the actions listed below.
The validated NICs are:
- Mellanox (ConnectX-4, ConnectX-4 Lx, ConnectX-5)
- Napatech (NT200B01)
-Supported protocols for hardware offload are:
+Supported protocols for hardware offload matches are:
+
- L2: Ethernet, VLAN
-- L3: IPv4, IPv6
+- L3: IPv4
- L4: TCP, UDP, SCTP, ICMP
+Supported actions for hardware offload are:
+
+- Output.
+
Further Reading
---------------
@@ -34,6 +34,7 @@ Post-v2.12.0
interval is increased to 60 seconds for the connection to the
replication server. This value is configurable with the unixctl
command - ovsdb-server/set-active-ovsdb-server-probe-interval.
+ * Add hardware offload support for output actions (experimental).
- 'ovs-appctl dpctl/dump-flows' can now show offloaded=partial for
partially offloaded flows, dp:dpdk for fully offloaded by dpdk, and
type filter supports new filters: "dpdk" and "partially-offloaded".
@@ -347,6 +347,26 @@ dump_flow_action(struct ds *s, const struct rte_flow_action *actions)
} else {
ds_put_cstr(s, " RSS = null\n");
}
+ } else if (actions->type == RTE_FLOW_ACTION_TYPE_COUNT) {
+ const struct rte_flow_action_count *count = actions->conf;
+
+ ds_put_cstr(s, "rte flow count action:\n");
+ if (count) {
+ ds_put_format(s, " Count: shared=%d, id=%d\n", count->shared,
+ count->id);
+ } else {
+ ds_put_cstr(s, " Count = null\n");
+ }
+ } else if (actions->type == RTE_FLOW_ACTION_TYPE_PORT_ID) {
+ const struct rte_flow_action_port_id *port_id = actions->conf;
+
+ ds_put_cstr(s, "rte flow port-id action:\n");
+ if (port_id) {
+ ds_put_format(s, " Port-id: original=%d, id=%d\n",
+ port_id->original, port_id->id);
+ } else {
+ ds_put_cstr(s, " Port-id = null\n");
+ }
} else {
ds_put_format(s, "unknown rte flow action (%d)\n", actions->type);
}
@@ -724,19 +744,77 @@ netdev_offload_dpdk_mark_rss(struct flow_patterns *patterns,
return flow;
}
+static void
+add_count_action(struct flow_actions *actions)
+{
+ struct rte_flow_action_count *count = xzalloc(sizeof *count);
+
+ add_flow_action(actions, RTE_FLOW_ACTION_TYPE_COUNT, count);
+}
+
static int
-parse_flow_actions(struct netdev *netdev OVS_UNUSED,
+add_port_id_action(struct flow_actions *actions,
+ struct netdev *outdev)
+{
+ struct rte_flow_action_port_id *port_id;
+ int outdev_id;
+
+ outdev_id = netdev_dpdk_get_port_id(outdev);
+ if (outdev_id < 0) {
+ return -1;
+ }
+ port_id = xzalloc(sizeof *port_id);
+ port_id->id = outdev_id;
+ add_flow_action(actions, RTE_FLOW_ACTION_TYPE_PORT_ID, port_id);
+ return 0;
+}
+
+static int
+add_output_action(struct netdev *netdev,
+ struct flow_actions *actions,
+ const struct nlattr *nla,
+ struct offload_info *info)
+{
+ struct netdev *outdev;
+ odp_port_t port;
+ int ret = 0;
+
+ port = nl_attr_get_odp_port(nla);
+ outdev = netdev_ports_get(port, info->dpif_class);
+ if (outdev == NULL) {
+ VLOG_DBG_RL(&rl, "Cannot find netdev for odp port %"PRIu32, port);
+ return -1;
+ }
+ if (!netdev_flow_api_equals(netdev, outdev) ||
+ add_port_id_action(actions, outdev)) {
+ VLOG_DBG_RL(&rl, "%s: Output to port \'%s\' cannot be offloaded.",
+ netdev_get_name(netdev), netdev_get_name(outdev));
+ ret = -1;
+ }
+ netdev_close(outdev);
+ return ret;
+}
+
+static int
+parse_flow_actions(struct netdev *netdev,
struct flow_actions *actions,
struct nlattr *nl_actions,
size_t nl_actions_len,
- struct offload_info *info OVS_UNUSED)
+ struct offload_info *info)
{
struct nlattr *nla;
size_t left;
+ add_count_action(actions);
NL_ATTR_FOR_EACH_UNSAFE (nla, left, nl_actions, nl_actions_len) {
- VLOG_DBG_RL(&rl, "Unsupported action type %d", nl_attr_type(nla));
- return -1;
+ if (nl_attr_type(nla) == OVS_ACTION_ATTR_OUTPUT) {
+ if (add_output_action(netdev, actions, nla, info)) {
+ return -1;
+ }
+ } else {
+ VLOG_DBG_RL(&rl, "Unsupported action type %d", nl_attr_type(nla));
+ return -1;
+ }
}
if (nl_actions_len == 0) {