| Message ID | 20200915020341.28742-2-stephen@networkplumber.org |
|---|---|
| State | New |
| Delegated to: | Thomas Monjalon |
| Headers | show |
| Series |
|
| Related | show |
| Context | Check | Description |
|---|---|---|
| ci/checkpatch | success | coding style OK |
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?
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.
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?
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
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(-)