[ovs-dev] [PATCH v2 1/1] doc: Remove "experimental" warning for userspace.
Kevin Traynor
ktraynor at redhat.com
Fri Feb 3 18:39:00 UTC 2017
On 02/03/2017 06:26 PM, Daniele Di Proietto wrote:
> 2017-02-03 9:38 GMT-08:00 Kevin Traynor <ktraynor at redhat.com>:
>> 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 <ian.stokes at intel.com>
>>>>> ---
>>>>> 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.
>
> Thanks for the patch, I'm in favor of the change for DPDK.
>
> Unless you have a strong reason do to so, I'd prefer to keep the
> experimental tag for the userspace datapath without DPDK for the
> following reasons:
>
> * I don't see a lot of valid use cases for it. I think it is
> important for testing.
> * There's at least a known problem with accessing linux device in
> userspace without DPDK for containers with offloads. See
> ddcf96d2dcc1("system-tests: Disable offloads in userspace tests.")
>
> What do you think?
>
That sounds good, ACK
thanks,
Kevin.
> Thanks,
>
> Daniele
>
>>
>>>>
>>>>
>>>>>
>>>>> 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
>> dev at openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
More information about the dev
mailing list