Re: [ovs-dev] [PATCH 1/2] vswitch.xml: Fix key type and description style of tc-policy.

2018-08-30 Thread Simon Horman 
On Mon, Aug 27, 2018 at 01:56:53PM +0200, Simon Horman wrote:
> On Fri, Aug 24, 2018 at 06:13:19PM +0300, Ilya Maximets wrote:
> > The set of supported values specified.
> > Style fixed to look good in man page. Fixed indents.
> > 
> > CC: Paul Blakey 
> > Fixes: 691d20cbdcf3 ("other-config: Add tc-policy switch to
> >   control tc flower flag")
> > Signed-off-by: Ilya Maximets 
> 
> Thanks,
> 
> This looks fine to me but I will wait to see if there are other reviews
> before applying.

Thanks again,

applied to master, branch-2.10, branch-2.9 and branch-2.8.

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH 1/2] vswitch.xml: Fix key type and description style of tc-policy.

2018-08-27 Thread Simon Horman 
On Fri, Aug 24, 2018 at 06:13:19PM +0300, Ilya Maximets wrote:
> The set of supported values specified.
> Style fixed to look good in man page. Fixed indents.
> 
> CC: Paul Blakey 
> Fixes: 691d20cbdcf3 ("other-config: Add tc-policy switch to
>   control tc flower flag")
> Signed-off-by: Ilya Maximets 

Thanks,

This looks fine to me but I will wait to see if there are other reviews
before applying.

Acked-by: Simon Horman 
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH 1/2] vswitch.xml: Fix key type and description style of tc-policy.

2018-08-24 Thread Ilya Maximets 
The set of supported values specified.
Style fixed to look good in man page. Fixed indents.

CC: Paul Blakey 
Fixes: 691d20cbdcf3 ("other-config: Add tc-policy switch to
  control tc flower flag")
Signed-off-by: Ilya Maximets 
---
 vswitchd/vswitch.xml | 27 +--
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index a858e8ef4..570c1619f 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -201,16 +201,23 @@
   
 
   
-
-Specified the policy used with HW offloading.
-Options:
-none- Add software rule and offload rule to 
HW.
-skip_sw - Offload rule to HW only.
-skip_hw - Add software rule without offloading 
rule to HW.
-
-
-This is only relevant if HW offloading is enabled (hw-offload).
+  type='{"type": "string",
+ "enum": ["set", ["none", "skip_sw", "skip_hw"]]}'>
+
+  Specified the policy used with HW offloading.
+  Options:
+  
+none
+Add software rule and offload rule to HW.
+skip_sw
+Offload rule to HW only.
+skip_hw
+Add software rule without offloading rule to HW.
+  
+
+
+  This is only relevant if
+   is enabled.
 
 
   The default value is none.
-- 
2.17.1

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev