On 02/02/2017 08:22 PM, Stokes, Ian wrote:
>> On 02/02/2017 04:44 PM, Ian Stokes wrote:
>>> Remove the experimental warning tag in documentation regarding OVS
>>> deployed via userspace.
>>>
>>> Signed-off-by: Ian Stokes <[email protected]>
>>> ---
>>>  Documentation/intro/install/dpdk.rst      |    3 ---
>>>  Documentation/intro/install/userspace.rst |    4 ----
>>>  NEWS                                      |    2 ++
>>>  README.rst                                |    6 +++---
>>>  4 files changed, 5 insertions(+), 10 deletions(-)
>>>
>>> diff --git a/Documentation/intro/install/dpdk.rst
>>> b/Documentation/intro/install/dpdk.rst
>>> index fff0a1a..3018590 100644
>>> --- a/Documentation/intro/install/dpdk.rst
>>> +++ b/Documentation/intro/install/dpdk.rst
>>> @@ -29,9 +29,6 @@ This document describes how to build and install
>>> Open vSwitch using a DPDK  datapath. Open vSwitch can use the DPDK
>>> library to operate entirely in  userspace.
>>>
>>> -.. warning::
>>> -  The DPDK support of Open vSwitch is considered 'experimental'.
>>> -
>>>  Build requirements
>>>  ------------------
>>>
>>> diff --git a/Documentation/intro/install/userspace.rst
>>> b/Documentation/intro/install/userspace.rst
>>> index 0368527 <sip:0368527>..ebd0c12 100644
>>> --- a/Documentation/intro/install/userspace.rst
>>> +++ b/Documentation/intro/install/userspace.rst
>>> @@ -34,10 +34,6 @@ This version of Open vSwitch should be built
>>> manually with ``configure`` and  been recently tested, and so Debian
>>> packages are not a recommended way to use  this version of Open vSwitch.
>>>
>>> -.. warning::
>>> -  The userspace-only mode of Open vSwitch is considered experimental.
>>> It has
>>> -  not been thoroughly tested.
>>> -
>>>  Building and Installing
>>>  -----------------------
>>>
>>> diff --git a/NEWS b/NEWS
>>> index 5efcce2..8600f0e 100644
>>> --- a/NEWS
>>> +++ b/NEWS
>>> @@ -3,6 +3,8 @@ Post-v2.7.0
>>>     - Tunnels:
>>>       * Added support to set packet mark for tunnel endpoint using
>>>         `egress_pkt_mark` OVSDB option.
>>> +   - Documentation:
>>> +     * OVS deployed in userspace mode no longer tagged as experimental.
>>
>> I think this would be a bit clearer. What do you think?
>>
>> --- a/NEWS
>> +++ b/NEWS
>> @@ -59,4 +59,5 @@ v2.7.0 - xx xxx xxxx
>>       * Removed support for IPsec tunnels.
>>     - DPDK:
>> +     * Removal of experimental tag.
> 
> We can put it under a DPDK header like you have here, I wonder though is the 
> removal of experimental only relevant to DPDK or do we need to consider OVS 
> userspace without DPDK also? I would think both are well tested at this stage 
> so experimental could be removed for both. Thoughts?
> 

Fine by me. I just want it to be clear that it's removal is for/covers
the DPDK datapath.

>>
>>
>>>
>>>  v2.7.0 - xx xxx xxxx
>>>  ---------------------
>>> diff --git a/README.rst b/README.rst
>>> index f5cdaa5..90050e3 100644
>>> --- a/README.rst
>>> +++ b/README.rst
>>> @@ -38,9 +38,9 @@ following features:
>>>
>>>  The included Linux kernel module supports Linux 3.10 and up.
>>>
>>> -Open vSwitch can also operate, at a cost in performance, entirely in
>>> userspace, -without assistance from a kernel module.  This userspace
>>> implementation should -be easier to port than the kernel-based switch.
>> It is considered experimental.
>>> +Open vSwitch can also operate entirely in userspace without
>>> +assistance from a kernel module.  This userspace implementation
>>> +should be easier to port than the kernel-based switch.
>>>
>>>  What's here?
>>>  ------------
>>>
> 

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

Reply via email to