> 2017-02-03 9:38 GMT-08:00 Kevin Traynor <ktray...@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.sto...@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?
>
> Thanks,
>
> Daniele
>
Ok that sounds good, I'll modify the readme and userspace doc to specify that
ovs userspace without DPDK is experimental in the v3 (right now it’s a bit
ambiguous as regards what's experimental with userspace).
Thanks
Ian
> >
> >>>
> >>>
> >>>>
> >>>> 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
> > d...@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
