[dpdk-dev] doc: announce kni_vhost removal

Message ID 20161117132721.13106-1-ferruh.yigit@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Checks

Context Check Description
checkpatch/checkpatch success coding style OK

Commit Message

Ferruh Yigit Nov. 17, 2016, 1:27 p.m. UTC
  Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 doc/guides/prog_guide/kernel_nic_interface.rst | 2 ++
 doc/guides/rel_notes/deprecation.rst           | 6 ++++++
 2 files changed, 8 insertions(+)
  

Comments

John McNamara Nov. 29, 2016, 1:38 p.m. UTC | #1
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ferruh Yigit
> Sent: Thursday, November 17, 2016 1:27 PM
> To: dev@dpdk.org
> Cc: Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [dpdk-dev] [PATCH] doc: announce kni_vhost removal
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

Acked-by: John McNamara <john.mcnamara@intel.com>
  
Ferruh Yigit Feb. 7, 2017, 10:52 a.m. UTC | #2
On 11/17/2016 1:27 PM, Ferruh Yigit wrote:
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  doc/guides/prog_guide/kernel_nic_interface.rst | 2 ++
>  doc/guides/rel_notes/deprecation.rst           | 6 ++++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/doc/guides/prog_guide/kernel_nic_interface.rst b/doc/guides/prog_guide/kernel_nic_interface.rst
> index eb16e2e..4f25595 100644
> --- a/doc/guides/prog_guide/kernel_nic_interface.rst
> +++ b/doc/guides/prog_guide/kernel_nic_interface.rst
> @@ -169,6 +169,8 @@ This provides flexibility in multiprocess scenarios
>  (where the KNI is created in the primary process but the callbacks are handled in the secondary one).
>  The constraint is that a single process can register and handle the requests.
>  
> +.. _kni_vhost_backend-label:
> +
>  KNI Working as a Kernel vHost Backend
>  -------------------------------------
>  
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 2d17bc6..956473a 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -71,3 +71,9 @@ Deprecation Notices
>  * mempool: The functions for single/multi producer/consumer are deprecated
>    and will be removed in 17.02.
>    It is replaced by ``rte_mempool_generic_get/put`` functions.
> +
> +* kni: Remove :ref:`kni_vhost_backend-label` feature (KNI_VHOST) in 17.05 release.
> +  :doc:`Vhost Library </prog_guide/vhost_lib>` is currently preferred method for
> +  guest - host communication. Just for clarification, this is not to remove KNI
> +  or VHOST feature, but KNI_VHOST which is a KNI feature enabled via a compile
> +  time option, and disabled by default.
> 

Reminder for this patch.

Thanks,
ferruh
  
Thomas Monjalon Feb. 14, 2017, 10:48 a.m. UTC | #3
2016-11-17 13:27, Ferruh Yigit:
> +* kni: Remove :ref:`kni_vhost_backend-label` feature (KNI_VHOST) in 17.05 release.
> +  :doc:`Vhost Library </prog_guide/vhost_lib>` is currently preferred method for
> +  guest - host communication. Just for clarification, this is not to remove KNI
> +  or VHOST feature, but KNI_VHOST which is a KNI feature enabled via a compile
> +  time option, and disabled by default.

Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
  
Bruce Richardson Feb. 14, 2017, 11:42 a.m. UTC | #4
On Tue, Feb 14, 2017 at 11:48:29AM +0100, Thomas Monjalon wrote:
> 2016-11-17 13:27, Ferruh Yigit:
> > +* kni: Remove :ref:`kni_vhost_backend-label` feature (KNI_VHOST) in 17.05 release.
> > +  :doc:`Vhost Library </prog_guide/vhost_lib>` is currently preferred method for
> > +  guest - host communication. Just for clarification, this is not to remove KNI
> > +  or VHOST feature, but KNI_VHOST which is a KNI feature enabled via a compile
> > +  time option, and disabled by default.
> 
> Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Acked-by: Bruce Richardson <bruce.richardson@intel.com>
  
De Lara Guarch, Pablo Feb. 14, 2017, 11:46 a.m. UTC | #5
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Bruce Richardson
> Sent: Tuesday, February 14, 2017 11:42 AM
> To: Thomas Monjalon
> Cc: Yigit, Ferruh; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] doc: announce kni_vhost removal
> 
> On Tue, Feb 14, 2017 at 11:48:29AM +0100, Thomas Monjalon wrote:
> > 2016-11-17 13:27, Ferruh Yigit:
> > > +* kni: Remove :ref:`kni_vhost_backend-label` feature (KNI_VHOST) in
> 17.05 release.
> > > +  :doc:`Vhost Library </prog_guide/vhost_lib>` is currently preferred
> method for
> > > +  guest - host communication. Just for clarification, this is not to
> remove KNI
> > > +  or VHOST feature, but KNI_VHOST which is a KNI feature enabled via
> a compile
> > > +  time option, and disabled by default.
> >
> > Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
  
Thomas Monjalon Feb. 14, 2017, 8:31 p.m. UTC | #6
Applied
  

Patch

diff --git a/doc/guides/prog_guide/kernel_nic_interface.rst b/doc/guides/prog_guide/kernel_nic_interface.rst
index eb16e2e..4f25595 100644
--- a/doc/guides/prog_guide/kernel_nic_interface.rst
+++ b/doc/guides/prog_guide/kernel_nic_interface.rst
@@ -169,6 +169,8 @@  This provides flexibility in multiprocess scenarios
 (where the KNI is created in the primary process but the callbacks are handled in the secondary one).
 The constraint is that a single process can register and handle the requests.
 
+.. _kni_vhost_backend-label:
+
 KNI Working as a Kernel vHost Backend
 -------------------------------------
 
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 2d17bc6..956473a 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -71,3 +71,9 @@  Deprecation Notices
 * mempool: The functions for single/multi producer/consumer are deprecated
   and will be removed in 17.02.
   It is replaced by ``rte_mempool_generic_get/put`` functions.
+
+* kni: Remove :ref:`kni_vhost_backend-label` feature (KNI_VHOST) in 17.05 release.
+  :doc:`Vhost Library </prog_guide/vhost_lib>` is currently preferred method for
+  guest - host communication. Just for clarification, this is not to remove KNI
+  or VHOST feature, but KNI_VHOST which is a KNI feature enabled via a compile
+  time option, and disabled by default.