diff mbox series

[ovs-dev] ovs-actions.xml: Better document the "bundle" and "bundle_load" actions.

Message ID 20190502014445.2430-1-blp@ovn.org
State Superseded
Headers show
Series [ovs-dev] ovs-actions.xml: Better document the "bundle" and "bundle_load" actions. | expand

Commit Message

Ben Pfaff May 2, 2019, 1:44 a.m. UTC
Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 lib/ovs-actions.xml | 43 +++++++++++++++++++++++++++++++++++++++----
 1 file changed, 39 insertions(+), 4 deletions(-)

Comments

0-day Robot May 2, 2019, 2:51 a.m. UTC | #1
Bleep bloop.  Greetings Ben Pfaff, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
WARNING: Line is 188 characters long (recommended limit is 79)
#21 FILE: lib/ovs-actions.xml:792:
      <syntax><code>bundle(</code><var>fields</var><code>, </code><var>basis</var><code>, </code><var>algorithm</var><code>, ofport, slaves:</code><var>port</var>...<code>)</code></syntax>

WARNING: Line is 222 characters long (recommended limit is 79)
#22 FILE: lib/ovs-actions.xml:793:
      <syntax><code>bundle_load(</code><var>fields</var><code>, </code><var>basis</var><code>, </code><var>algorithm</var><code>, ofport, </code><var>dst</var><code>, slaves:</code><var>port</var>...<code>)</code></syntax>

WARNING: Line is 154 characters long (recommended limit is 79)
#59 FILE: lib/ovs-actions.xml:870:
<var>slave</var> = { <var>i</var> such that <var>weights</var>[<var>i</var>] &gt;= <var>weights</var>[<var>j</var>] for all <var>j</var> != <var>i</var> }

Lines checked: 79, Warnings: 3, Errors: 0


Please check this out.  If you feel there has been an error, please email aconole@bytheb.org

Thanks,
0-day Robot
diff mbox series

Patch

diff --git a/lib/ovs-actions.xml b/lib/ovs-actions.xml
index cfd9b81be604..da9d941ac7e0 100644
--- a/lib/ovs-actions.xml
+++ b/lib/ovs-actions.xml
@@ -789,15 +789,15 @@  $ ovs-ofctl -O OpenFlow10 add-flow br0 actions=mod_nw_src:1.2.3.4
 
     <action name="BUNDLE,BUNDLE_LOAD">
       <h2>The <code>bundle</code> and <code>bundle_load</code> actions</h2>
-      <syntax><code>bundle(</code><var>fields</var><code>, </code><var>basis</var><code>, </code><var>algorithm</var><code>, </code>ofport<code>, slaves:</code><var>port</var>...<code>)</code></syntax>
-      <syntax><code>bundle_load(</code><var>fields</var><code>, </code><var>basis</var><code>, </code><var>algorithm</var><code>, </code>ofport<code>, </code><var>dst</var><code>, slaves:</code><var>port</var>...<code>)</code></syntax>
+      <syntax><code>bundle(</code><var>fields</var><code>, </code><var>basis</var><code>, </code><var>algorithm</var><code>, ofport, slaves:</code><var>port</var>...<code>)</code></syntax>
+      <syntax><code>bundle_load(</code><var>fields</var><code>, </code><var>basis</var><code>, </code><var>algorithm</var><code>, ofport, </code><var>dst</var><code>, slaves:</code><var>port</var>...<code>)</code></syntax>
 
       <p>
         These actions choose a port (``slave'') from a comma-separated OpenFlow
         <var>port</var> list.  After selecting the port, <code>bundle</code>
         outputs to it, whereas <code>bundle_load</code> writes its port number
-        to <var>dst</var>, which must be a field or subfield in the syntax
-        described under ``Field Specifications'' above.
+        to <var>dst</var>, which must be a 16-bit or wider field or subfield in
+        the syntax described under ``Field Specifications'' above.
       </p>
 
       <p>
@@ -847,6 +847,41 @@  $ ovs-ofctl -O OpenFlow10 add-flow br0 actions=mod_nw_src:1.2.3.4
         </dd>
       </dl>
 
+      <p>
+        <var>algorithm</var> must be one of the following:
+      </p>
+
+      <dl>
+        <dt><code>active_backup</code></dt>
+        <dd>
+          Chooses the first live port listed in <var>slaves</var>.
+        </dd>
+
+        <dt><code>hrw</code> (Highest Random Weight)</dt>
+        <dd>
+          <p>
+            Computes the following, considering only the live ports in
+            <var>slaves</var>:
+          </p>
+
+          <pre>
+for <var>i</var> in [1,<var>n_slaves</var>]:
+    <var>weights</var>[<var>i</var>] = hash(<var>flow</var>, <var>i</var>)
+<var>slave</var> = { <var>i</var> such that <var>weights</var>[<var>i</var>] &gt;= <var>weights</var>[<var>j</var>] for all <var>j</var> != <var>i</var> }
+          </pre>
+
+          <p>
+            This algorithm is specified by RFC 2992.
+          </p>
+        </dd>
+      </dl>
+
+      <p>
+        Example: <code>bundle(eth_src,0,hrw,ofport,slaves:4,8)</code> uses an
+        Ethernet source hash with basis 0, to select between OpenFlow ports 4
+        and 8 using the Highest Random Weight algorithm.
+      </p>
+
       <conformance>
         Open vSwitch 1.2 introduced the <code>bundle</code> and
         <code>bundle_load</code> OpenFlow extension actions.