diff mbox

[ovs-dev,ovs,V8,18/26] netdev-linux: Disallow setting policing when configured with hw offload

Message ID 1493824097-47495-19-git-send-email-roid@mellanox.com
State Changes Requested
Headers show

Commit Message

Roi Dayan May 3, 2017, 3:08 p.m. UTC 
From: Paul Blakey <paulb@mellanox.com>

Notify as not supported. Otherwise the ingress qdisc is being removed and
offload rules will be removed.

Signed-off-by: Paul Blakey <paulb@mellanox.com>
Reviewed-by: Roi Dayan <roid@mellanox.com>
Reviewed-by: Simon Horman <simon.horman@netronome.com>
---
 lib/netdev-linux.c | 8 ++++++++
 1 file changed, 8 insertions(+)
diff mbox

Patch

diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
index f7941fd..189dbd6 100644
--- a/lib/netdev-linux.c
+++ b/lib/netdev-linux.c
@@ -2074,6 +2074,14 @@  netdev_linux_set_policing(struct netdev *netdev_,
     int error;
     int ifindex;
 
+    if (netdev_flow_api_enabled) {
+        if (kbits_rate) {
+            VLOG_WARN_RL(&rl, "%s: policing with offload isn't supported",
+                         netdev_name);
+        }
+        return EOPNOTSUPP;
+    }
+
     kbits_burst = (!kbits_rate ? 0       /* Force to 0 if no rate specified. */
                    : !kbits_burst ? 8000 /* Default to 8000 kbits if 0. */
                    : kbits_burst);       /* Stick with user-specified value. */