diff mbox series

[ovs-dev,v2] vswitchd: update tc hwol docs regarding vswitchd restarts

Message ID 008684e0fa3b6ca8a95e753647549d1e49a2e7b5.1612378956.git.marcelo.leitner@gmail.com
State Changes Requested
Headers show
Series [ovs-dev,v2] vswitchd: update tc hwol docs regarding vswitchd restarts | expand

Commit Message

Marcelo Ricardo Leitner Feb. 3, 2021, 7:05 p.m. UTC
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.

v2: include changes regarding the new understanding on hw-offload

Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
---
 vswitchd/vswitch.xml | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index 5b2a174ab813139d3f3253be47758d4bfb1beac3..de7aba00ae91357a561e60c1e367e702713ac108 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -228,8 +228,8 @@ 
           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>. Disabling this value requires
+          restarting the daemon.
         </p>
         <p>
           Currently Open vSwitch supports hardware offloading on
@@ -267,7 +267,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>