Re: [ovs-dev] [PATCH] vswitch.xml: Fix bond-hash-basis documentation placement.

2011-11-30 Thread Ben Pfaff
Great, add a period to the final sentence and ship it. Thanks. On Wed, Nov 30, 2011 at 07:41:34PM -0800, Ethan Jackson wrote: > Alright, I've changed the wording to: > > +An integer hashed along with flows when choosing output slaves in > load > +balanced bonds. When changed, a

Re: [ovs-dev] [PATCH] vswitch.xml: Fix bond-hash-basis documentation placement.

2011-11-30 Thread Ethan Jackson
Alright, I've changed the wording to: +An integer hashed along with flows when choosing output slaves in load +balanced bonds. When changed, all flows will be assigned different +hash values possibly causing slave selection decisions to change. Does +not affect bo

Re: [ovs-dev] [PATCH] vswitch.xml: Fix bond-hash-basis documentation placement.

2011-11-30 Thread Ben Pfaff
On Wed, Nov 30, 2011 at 06:09:39PM -0800, Ethan Jackson wrote: > The bond-hash-basis other_config column was documented under "LACP > Configuration". Instead it should be documented as general bonding > configuration. Looks good but it might be worth adding a note that it does not affect the acti

[ovs-dev] [PATCH] vswitch.xml: Fix bond-hash-basis documentation placement.

2011-11-30 Thread Ethan Jackson
The bond-hash-basis other_config column was documented under "LACP Configuration". Instead it should be documented as general bonding configuration. --- vswitchd/vswitch.xml | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/vswitchd/vswitch.xml b/vswitchd/vswit