From: Paul Blakey <pa...@mellanox.com>
Add a new configuration option - hw-offload that enables netdev
flow api. Enabling this option will allow offloading flows
using netdev implementation instead of the kernel datapath.
This configuration option defaults to false - disabled.
Signed-off-by: Paul Blakey <pa...@mellanox.com>
Reviewed-by: Roi Dayan <r...@mellanox.com>
Reviewed-by: Simon Horman <simon.hor...@netronome.com>
---
lib/netdev.c | 35 +++++++++++++++++++++++++++++++++++
lib/netdev.h | 2 ++
vswitchd/bridge.c | 1 +
vswitchd/vswitch.xml | 15 +++++++++++++++
4 files changed, 53 insertions(+)
diff --git a/lib/netdev.c b/lib/netdev.c
index 9eb37ec..2807a9d 100644
--- a/lib/netdev.c
+++ b/lib/netdev.c
@@ -92,6 +92,8 @@ struct netdev_registered_class {
struct ovs_refcount refcnt;
};
+static bool netdev_flow_api_enabled = false;
+
/* This is set pretty low because we probably won't learn anything from the
* additional log messages. */
static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20);
@@ -2088,7 +2090,40 @@ netdev_init_flow_api(struct netdev *netdev)
{
const struct netdev_class *class = netdev->netdev_class;
+ if (!netdev_is_flow_api_enabled()) {
+ return EOPNOTSUPP;
+ }
+
return (class->init_flow_api
? class->init_flow_api(netdev)
: EOPNOTSUPP);
}
+
+bool
+netdev_is_flow_api_enabled(void)
+{
+ return netdev_flow_api_enabled;
+}
+
+#ifdef __linux__
+void
+netdev_set_flow_api_enabled(const struct smap *ovs_other_config)
+{
+ if (smap_get_bool(ovs_other_config, "hw-offload", false)) {
+ static struct ovsthread_once once = OVSTHREAD_ONCE_INITIALIZER;
+
+ if (ovsthread_once_start(&once)) {
+ netdev_flow_api_enabled = true;
+
+ VLOG_INFO("netdev: Flow API Enabled");
+
+ ovsthread_once_done(&once);
+ }
+ }
+}
+#else
+void
+netdev_set_flow_api_enabled(const struct smap *ovs_other_config OVS_UNUSED)
+{
+}
+#endif
diff --git a/lib/netdev.h b/lib/netdev.h
index 17890e6..696f0f4 100644
--- a/lib/netdev.h
+++ b/lib/netdev.h
@@ -178,6 +178,8 @@ int netdev_flow_get(struct netdev *, struct match *, struct
nlattr **actions,
int netdev_flow_del(struct netdev *, const ovs_u128 *,
struct dpif_flow_stats *);
int netdev_init_flow_api(struct netdev *);
+bool netdev_is_flow_api_enabled(void);
+void netdev_set_flow_api_enabled(const struct smap *ovs_other_config);
/* native tunnel APIs */
/* Structure to pass parameters required to build a tunnel header. */
diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index ebb6249..d1e130f 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -2954,6 +2954,7 @@ bridge_run(void)
cfg = ovsrec_open_vswitch_first(idl);
if (cfg) {
+ netdev_set_flow_api_enabled(&cfg->other_config);
dpdk_init(&cfg->other_config);
}
diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index bb66cb5..2caeb3b 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -178,6 +178,21 @@
<p>
The default is 10000.
</p>
+ </column>
+
+ <column name="other_config" key="hw-offload"
+ type='{"type": "boolean"}'>
+ <p>
+ Set this value to <code>true</code> to enable netdev flow offload.
+ </p>
+ <p>
+ The default value is <code>false</code>. Changing this value requires
+ restarting the daemon
+ </p>
+ <p>
+ Currently Open vSwitch supports hardware offloading on
+ Linux systems. On other systems, this value is ignored.
+ </p>
</column>
<column name="other_config" key="dpdk-init"
--
2.7.4
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
