[1/3] MAINTAINERS remove experimental tag from vdev_netvsc
diff mbox series

Message ID 20200915020341.28742-2-stephen@networkplumber.org
State New
Delegated to: Thomas Monjalon
Headers show
Series
  • MAINTAINERS changes related to netvsc
Related show

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Stephen Hemminger Sept. 15, 2020, 2:03 a.m. UTC
Vdev_netvsc has been around for several years. It no longer needs
to be marked experimental.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ferruh Yigit Sept. 15, 2020, 9:42 a.m. UTC | #1
On 9/15/2020 3:03 AM, Stephen Hemminger wrote:
> Vdev_netvsc has been around for several years. It no longer needs
> to be marked experimental.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  MAINTAINERS | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3b16d7a4b8c8..32db0996181f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -768,7 +768,7 @@ F: buildtools/options-ibverbs-static.sh
>  F: doc/guides/nics/mlx5.rst
>  F: doc/guides/nics/features/mlx5.ini
>  
> -Microsoft vdev_netvsc - EXPERIMENTAL
> +Microsoft vdev_netvsc
>  M: Matan Azrad <matan@mellanox.com>
>  F: drivers/net/vdev_netvsc/
>  F: doc/guides/nics/vdev_netvsc.rst
> 

As far as I remember 'vdev_netvsc' was interim solution until 'netvsc'
was ready. In this patchset 'netvsc' is also becoming mature.

Wouldn't be easier to keep 'vdev_netvsc' experimental to be able to
remove it soon?
Stephen Hemminger Sept. 15, 2020, 2:47 p.m. UTC | #2
On Tue, 15 Sep 2020 10:42:07 +0100
Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 9/15/2020 3:03 AM, Stephen Hemminger wrote:
> > Vdev_netvsc has been around for several years. It no longer needs
> > to be marked experimental.
> > 
> > Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> > ---
> >  MAINTAINERS | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 3b16d7a4b8c8..32db0996181f 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -768,7 +768,7 @@ F: buildtools/options-ibverbs-static.sh
> >  F: doc/guides/nics/mlx5.rst
> >  F: doc/guides/nics/features/mlx5.ini
> >  
> > -Microsoft vdev_netvsc - EXPERIMENTAL
> > +Microsoft vdev_netvsc
> >  M: Matan Azrad <matan@mellanox.com>
> >  F: drivers/net/vdev_netvsc/
> >  F: doc/guides/nics/vdev_netvsc.rst
> >   
> 
> As far as I remember 'vdev_netvsc' was interim solution until 'netvsc'
> was ready. In this patchset 'netvsc' is also becoming mature.
> 
> Wouldn't be easier to keep 'vdev_netvsc' experimental to be able to
> remove it soon?

Let me discuss with Long Li and management.
Maybe replace EXPERIMENTAL with DEPRECATED in 20.11.
Thomas Monjalon Oct. 5, 2020, 10:41 p.m. UTC | #3
15/09/2020 16:47, Stephen Hemminger:
> Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> > On 9/15/2020 3:03 AM, Stephen Hemminger wrote:
> > > Vdev_netvsc has been around for several years. It no longer needs
> > > to be marked experimental.
> > > 
> > > Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> > > ---
> > > -Microsoft vdev_netvsc - EXPERIMENTAL
> > > +Microsoft vdev_netvsc
> > >  M: Matan Azrad <matan@mellanox.com>
> > >  F: drivers/net/vdev_netvsc/
> > >  F: doc/guides/nics/vdev_netvsc.rst
> > 
> > As far as I remember 'vdev_netvsc' was interim solution until 'netvsc'
> > was ready. In this patchset 'netvsc' is also becoming mature.
> > 
> > Wouldn't be easier to keep 'vdev_netvsc' experimental to be able to
> > remove it soon?
> 
> Let me discuss with Long Li and management.
> Maybe replace EXPERIMENTAL with DEPRECATED in 20.11.

Would be strange to switch from experimental to deprecated :)

+Cc Matan

I think you still need this platform driver (with failsafe and tap)
in case you need rte_flow. Or is it well supported with netvsc PMD?

Patch
diff mbox series

diff --git a/MAINTAINERS b/MAINTAINERS
index 3b16d7a4b8c8..32db0996181f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -768,7 +768,7 @@  F: buildtools/options-ibverbs-static.sh
 F: doc/guides/nics/mlx5.rst
 F: doc/guides/nics/features/mlx5.ini
 
-Microsoft vdev_netvsc - EXPERIMENTAL
+Microsoft vdev_netvsc
 M: Matan Azrad <matan@mellanox.com>
 F: drivers/net/vdev_netvsc/
 F: doc/guides/nics/vdev_netvsc.rst