mbox series

[0/5] netvsc changes for 18.11

Message ID 20180830223512.21297-1-stephen@networkplumber.org (mailing list archive)
Headers
Series netvsc changes for 18.11 |

Message

Stephen Hemminger Aug. 30, 2018, 10:35 p.m. UTC 
  These patches are against dpdk-net-next/master because
that is where previous patches have already been merged.

Stephen Hemminger (5):
  bus/vmbus: add devargs support
  net/netvsc: allow tuning latency with devargs
  net/netvsc: exhausting transmit descriptors is not an error
  net/netvsc: implement link state change callback
  net/netvsc: integrated VF support

 devtools/checkpatches.sh               |   2 +-
 doc/guides/nics/netvsc.rst             |  31 +-
 doc/guides/rel_notes/release_18_11.rst |  14 +-
 drivers/bus/vmbus/linux/vmbus_bus.c    |   2 +
 drivers/bus/vmbus/private.h            |   3 +
 drivers/bus/vmbus/vmbus_common.c       |  22 +-
 drivers/net/netvsc/Makefile            |   1 +
 drivers/net/netvsc/hn_ethdev.c         | 188 +++++++--
 drivers/net/netvsc/hn_nvs.c            |  11 +-
 drivers/net/netvsc/hn_nvs.h            |   9 +
 drivers/net/netvsc/hn_rndis.c          |  42 +-
 drivers/net/netvsc/hn_rndis.h          |   3 +-
 drivers/net/netvsc/hn_rxtx.c           | 144 ++++---
 drivers/net/netvsc/hn_var.h            |  48 ++-
 drivers/net/netvsc/hn_vf.c             | 512 +++++++++++++++++++++++++
 drivers/net/netvsc/meson.build         |   2 +-
 16 files changed, 924 insertions(+), 110 deletions(-)
 create mode 100644 drivers/net/netvsc/hn_vf.c

Comments

Ferruh Yigit Aug. 31, 2018, 12:04 p.m. UTC | #1 
On 8/30/2018 11:35 PM, Stephen Hemminger wrote:
> These patches are against dpdk-net-next/master because
> that is where previous patches have already been merged.
> 
> Stephen Hemminger (5):
>   bus/vmbus: add devargs support
>   net/netvsc: allow tuning latency with devargs
>   net/netvsc: exhausting transmit descriptors is not an error
>   net/netvsc: implement link state change callback
>   net/netvsc: integrated VF support

A highlevel question, since netvsc PMD is upstreamed, can we remove vdev_netvsc one?
Stephen Hemminger Aug. 31, 2018, 3:15 p.m. UTC | #2 
On Fri, 31 Aug 2018 13:04:18 +0100
Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> On 8/30/2018 11:35 PM, Stephen Hemminger wrote:
> > These patches are against dpdk-net-next/master because
> > that is where previous patches have already been merged.
> > 
> > Stephen Hemminger (5):
> >   bus/vmbus: add devargs support
> >   net/netvsc: allow tuning latency with devargs
> >   net/netvsc: exhausting transmit descriptors is not an error
> >   net/netvsc: implement link state change callback
> >   net/netvsc: integrated VF support  
> 
> A highlevel question, since netvsc PMD is upstreamed, can we remove vdev_netvsc one?

No.

There are some use cases that still need bifurcated model. And also netvsc PMD
is still in experimental state and vdev_netvsc/failsafe/tap are more stable.

I hope maybe by 19.11?
Thomas Monjalon Aug. 31, 2018, 4:56 p.m. UTC | #3 
31/08/2018 17:15, Stephen Hemminger:
> On Fri, 31 Aug 2018 13:04:18 +0100
> Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> 
> > On 8/30/2018 11:35 PM, Stephen Hemminger wrote:
> > > These patches are against dpdk-net-next/master because
> > > that is where previous patches have already been merged.
> > > 
> > > Stephen Hemminger (5):
> > >   bus/vmbus: add devargs support
> > >   net/netvsc: allow tuning latency with devargs
> > >   net/netvsc: exhausting transmit descriptors is not an error
> > >   net/netvsc: implement link state change callback
> > >   net/netvsc: integrated VF support  
> > 
> > A highlevel question, since netvsc PMD is upstreamed, can we remove vdev_netvsc one?
> 
> No.
> 
> There are some use cases that still need bifurcated model. And also netvsc PMD
> is still in experimental state and vdev_netvsc/failsafe/tap are more stable.
> 
> I hope maybe by 19.11?

19.11 would be reasonnable target to deprecate vdev_netvsc.
Ferruh Yigit Sept. 14, 2018, 1:25 p.m. UTC | #4 
On 8/30/2018 11:35 PM, Stephen Hemminger wrote:
> These patches are against dpdk-net-next/master because
> that is where previous patches have already been merged.
> 
> Stephen Hemminger (5):
>   bus/vmbus: add devargs support
>   net/netvsc: allow tuning latency with devargs
>   net/netvsc: exhausting transmit descriptors is not an error
>   net/netvsc: implement link state change callback
>   net/netvsc: integrated VF support

Series applied to dpdk-next-net/master, thanks.

(checkpatches.sh update dropped from patch, devarg syntax in function comment
updated.)