hw-offload is can actually be enabled without a restart. Then, once
enabled, a restart is needed in order to make the new value effective.

tc-policy, just like hw-offload, is protected by ovsthread_once_start()
in netdev_set_flow_api_enabled() and is only parsed when hw-offload is
enabled. So lets document that if hw-offload was already enabled
changing it requires a restart in order for it to have effect.

Signed-off-by: Marcelo Ricardo Leitner <[email protected]>
---

Notes:
    v2: include changes regarding the new understanding on hw-offload
    v3: fixed warnings
    v4: fix wording

 vswitchd/vswitch.xml | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index 
5b2a174ab813139d3f3253be47758d4bfb1beac3..cde791d4e19f69f49ab7f77ec3abfc6b8f4f30f9
 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -228,8 +228,9 @@
           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
+          The default value is <code>false</code>. Changing this value from
+          <code>true</code> to <code>false</code> requires restarting the
+          daemon.
         </p>
         <p>
           Currently Open vSwitch supports hardware offloading on
@@ -267,7 +268,9 @@
           <ref column="other_config" key="hw-offload"/> is enabled.
         </p>
         <p>
-          The default value is <code>none</code>.
+          The default value is <code>none</code>. Changing this value requires
+          restarting the daemon if
+          <ref column="other_config" key="hw-offload"/> was already enabled.
         </p>
       </column>
 
-- 
2.29.2

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to