DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc
@ 2020-09-15  2:03 Stephen Hemminger
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc Stephen Hemminger
                   ` (3 more replies)
  0 siblings, 4 replies; 10+ messages in thread
From: Stephen Hemminger @ 2020-09-15  2:03 UTC (permalink / raw)
  To: dev; +Cc: 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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc
  2020-09-15  2:03 [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Stephen Hemminger
@ 2020-09-15  2:03 ` Stephen Hemminger
  2020-09-15  9:42   ` Ferruh Yigit
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 2/3] MAINTAINERS - drop Kys and Haiyang Stephen Hemminger
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 10+ messages in thread
From: Stephen Hemminger @ 2020-09-15  2:03 UTC (permalink / raw)
  To: dev; +Cc: 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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [dpdk-dev] [PATCH 2/3] MAINTAINERS - drop Kys and Haiyang
  2020-09-15  2:03 [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Stephen Hemminger
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc Stephen Hemminger
@ 2020-09-15  2:03 ` Stephen Hemminger
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 3/3] MAINTAINERS - remove expermental tag of netvsd PMD Stephen Hemminger
  2020-11-27 14:03 ` [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Thomas Monjalon
  3 siblings, 0 replies; 10+ messages in thread
From: Stephen Hemminger @ 2020-09-15  2:03 UTC (permalink / raw)
  To: dev; +Cc: 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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [dpdk-dev] [PATCH 3/3] MAINTAINERS - remove expermental tag of netvsd PMD
  2020-09-15  2:03 [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Stephen Hemminger
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc Stephen Hemminger
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 2/3] MAINTAINERS - drop Kys and Haiyang Stephen Hemminger
@ 2020-09-15  2:03 ` Stephen Hemminger
  2020-11-27 14:03 ` [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Thomas Monjalon
  3 siblings, 0 replies; 10+ messages in thread
From: Stephen Hemminger @ 2020-09-15  2:03 UTC (permalink / raw)
  To: dev; +Cc: 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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc Stephen Hemminger
@ 2020-09-15  9:42   ` Ferruh Yigit
  2020-09-15 14:47     ` Stephen Hemminger
  0 siblings, 1 reply; 10+ messages in thread
From: Ferruh Yigit @ 2020-09-15  9:42 UTC (permalink / raw)
  To: Stephen Hemminger, dev

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?

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc
  2020-09-15  9:42   ` Ferruh Yigit
@ 2020-09-15 14:47     ` Stephen Hemminger
  2020-10-05 22:41       ` Thomas Monjalon
  0 siblings, 1 reply; 10+ messages in thread
From: Stephen Hemminger @ 2020-09-15 14:47 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: dev

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.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc
  2020-09-15 14:47     ` Stephen Hemminger
@ 2020-10-05 22:41       ` Thomas Monjalon
  2020-11-27 14:01         ` Thomas Monjalon
  2020-11-28 17:40         ` Stephen Hemminger
  0 siblings, 2 replies; 10+ messages in thread
From: Thomas Monjalon @ 2020-10-05 22:41 UTC (permalink / raw)
  To: Stephen Hemminger
  Cc: Ferruh Yigit, dev, matan, asafp, shys, K. Y. Srinivasan,
	Madhan Sivakumar

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?



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc
  2020-10-05 22:41       ` Thomas Monjalon
@ 2020-11-27 14:01         ` Thomas Monjalon
  2020-11-28 17:40         ` Stephen Hemminger
  1 sibling, 0 replies; 10+ messages in thread
From: Thomas Monjalon @ 2020-11-27 14:01 UTC (permalink / raw)
  To: Stephen Hemminger, matan
  Cc: dev, Ferruh Yigit, dev, asafp, shys, K. Y. Srinivasan, Madhan Sivakumar

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.



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc
  2020-09-15  2:03 [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Stephen Hemminger
                   ` (2 preceding siblings ...)
  2020-09-15  2:03 ` [dpdk-dev] [PATCH 3/3] MAINTAINERS - remove expermental tag of netvsd PMD Stephen Hemminger
@ 2020-11-27 14:03 ` Thomas Monjalon
  3 siblings, 0 replies; 10+ messages in thread
From: Thomas Monjalon @ 2020-11-27 14:03 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: dev, matan, david.marchand

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.



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc
  2020-10-05 22:41       ` Thomas Monjalon
  2020-11-27 14:01         ` Thomas Monjalon
@ 2020-11-28 17:40         ` Stephen Hemminger
  1 sibling, 0 replies; 10+ messages in thread
From: Stephen Hemminger @ 2020-11-28 17:40 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Ferruh Yigit, dev, matan, asafp, shys, K. Y. Srinivasan,
	Madhan Sivakumar

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).


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2020-11-28 17:41 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-15  2:03 [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Stephen Hemminger
2020-09-15  2:03 ` [dpdk-dev] [PATCH 1/3] MAINTAINERS remove experimental tag from vdev_netvsc Stephen Hemminger
2020-09-15  9:42   ` Ferruh Yigit
2020-09-15 14:47     ` Stephen Hemminger
2020-10-05 22:41       ` Thomas Monjalon
2020-11-27 14:01         ` Thomas Monjalon
2020-11-28 17:40         ` Stephen Hemminger
2020-09-15  2:03 ` [dpdk-dev] [PATCH 2/3] MAINTAINERS - drop Kys and Haiyang Stephen Hemminger
2020-09-15  2:03 ` [dpdk-dev] [PATCH 3/3] MAINTAINERS - remove expermental tag of netvsd PMD Stephen Hemminger
2020-11-27 14:03 ` [dpdk-dev] [PATCH 0/3] MAINTAINERS changes related to netvsc Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).