On 14/02/2017 01:53, Chandran, Sugesh wrote:


Regards
_Sugesh


-----Original Message-----
From: Roi Dayan [mailto:[email protected]]
Sent: Wednesday, February 8, 2017 3:29 PM
To: [email protected]
Cc: Paul Blakey <[email protected]>; Or Gerlitz
<[email protected]>; Hadar Hen Zion <[email protected]>; Shahar
Klein <[email protected]>; Mark Bloch <[email protected]>; Rony
Efraim <[email protected]>; Fastabend, John R
<[email protected]>; Joe Stringer <[email protected]>; Andy
Gospodarek <[email protected]>; Lance Richardson
<[email protected]>; Marcelo Ricardo Leitner <[email protected]>;
Simon Horman <[email protected]>; Jiri Pirko
<[email protected]>; Chandran, Sugesh <[email protected]>
Subject: [PATCH ovs V3 04/25] other-config: Add hw-offload switch to control
netdev flow offloading

From: Paul Blakey <[email protected]>

     }

diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml index
146a816..942e68f 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -170,6 +170,17 @@
         <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
[Sugesh] Hope its only the case of current implementation. Its possible to 
change the hardware offload status at run-time in the future. The system will 
re-populate the hardware ports accordingly when the status get changed.

We actually already changed that and the current implementation is same as for dpdk-init where we can change from false to true during run time but not from true back to false again. can be fixed later.

In case you do change this at run time we are missing to go over the netdevs and add the needed ingress qdisc so this will be needed to be done manually if working with this patchset.
We intend to fix this for V4.

I see the comment requiring restart also still exists for dpdk-init.
Maybe we should keep it currently as well until it's fully supported..

+        </p>
       </column>

       <column name="other_config" key="dpdk-init"
--
2.7.4

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

Reply via email to