[dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc

[dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc

[Stephen Hemminger]

Breakout three small changes to netvsc maintainers

Stephen Hemminger (3):
  MAINTAINERS remove experimental tag from vdev_netvsc
  MAINTAINERS - drop Kys and Haiyang
  MAINTAINERS - remove expermental tag of netvsd PMD

 MAINTAINERS | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

-- 
2.27.0

[dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc

[Stephen Hemminger]

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
-- 
2.27.0

[dpdk-dev] [PATCH 2/3] MAINTAINERS - drop Kys and Haiyang

[Stephen Hemminger]

These two maintainers deal with the Linux side of netvsc and
are not relevant for DPDK.

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

diff --git a/MAINTAINERS b/MAINTAINERS
index 32db0996181f..e3e70c25560d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -775,8 +775,6 @@ F: doc/guides/nics/vdev_netvsc.rst
 
 Microsoft Hyper-V netvsc - EXPERIMENTAL
 M: Stephen Hemminger <sthemmin@microsoft.com>
-M: K. Y. Srinivasan <kys@microsoft.com>
-M: Haiyang Zhang <haiyangz@microsoft.com>
 M: Long Li <longli@microsoft.com>
 F: drivers/net/netvsc/
 F: doc/guides/nics/netvsc.rst
-- 
2.27.0

[dpdk-dev] [PATCH 3/3] MAINTAINERS - remove expermental tag of netvsd PMD

[Stephen Hemminger]

With Long's help the driver is now stable enough for real usage.

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

diff --git a/MAINTAINERS b/MAINTAINERS
index e3e70c25560d..d9717c1997e6 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -773,7 +773,7 @@ M: Matan Azrad <matan@mellanox.com>
 F: drivers/net/vdev_netvsc/
 F: doc/guides/nics/vdev_netvsc.rst
 
-Microsoft Hyper-V netvsc - EXPERIMENTAL
+Microsoft Hyper-V netvsc
 M: Stephen Hemminger <sthemmin@microsoft.com>
 M: Long Li <longli@microsoft.com>
 F: drivers/net/netvsc/
-- 
2.27.0

Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc

[Ferruh Yigit]

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?

Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc

[Stephen Hemminger]

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.

Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc

[Thomas Monjalon]

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?

Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc

[Thomas Monjalon]

06/10/2020 00:41, Thomas Monjalon:
> 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?

No update...
I classify this patch as rejected.

Re: [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc

[Thomas Monjalon]

15/09/2020 04:03, Stephen Hemminger:
> Breakout three small changes to netvsc maintainers
> 
> Stephen Hemminger (3):
>   MAINTAINERS remove experimental tag from vdev_netvsc
>   MAINTAINERS - drop Kys and Haiyang
>   MAINTAINERS - remove expermental tag of netvsd PMD

Squashed patches 2 and 3, and rejected patch 1 because of no reply.

Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc

[Stephen Hemminger]

On Tue, 06 Oct 2020 00:41:55 +0200
Thomas Monjalon <thomas@monjalon.net> wrote:

> 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?

This needs more discussion. Netvsc PMD does not support rte_flow because
there is not a good/complete implementation of rte_flow library in pure
software form. It might be possible using the BPF stuff that tap supports.

I do not know of anybody who is using vdev_netvsc/failsafe with rte_flow.
Part of the problem is that the TAP implementation of rte_flow supports
a much smaller subset of features than the VF device (MLX).