mbox series

[net-next,0/6] hv_netvsc: minor changes

Message ID 20171213004840.17507-1-sthemmin@microsoft.com
Headers show
Series hv_netvsc: minor changes | expand

Message

Stephen Hemminger Dec. 13, 2017, 12:48 a.m. UTC 
This includes minor cleanup of code in send and receive path and
also a new statistic to check for allocation failures. This also
eliminates some of the extra RCU when not needed.

There is a theoritical bug where buffered data could be blocked
for longer than necessary if the ring buffer got full. This
has not been seen in the wild, found by inspection.

The reference count between net device and internal RNDIS
is not needed.

Stephen Hemminger (6):
  hv_netvsc: copy_to_send buf can be void
  hv_netvsc: track memory allocation failures in ethtool stats
  hv_netvsc: simplify function args in receive status path
  hv_netvsc: pass netvsc_device to receive callback
  hv_netvsc: remove open_cnt reference count
  hv_netvsc: empty current transmit aggregation if flow blocked

 drivers/net/hyperv/hyperv_net.h   |  9 ++++----
 drivers/net/hyperv/netvsc.c       | 44 ++++++++++++++++++++-------------------
 drivers/net/hyperv/netvsc_drv.c   | 33 +++++++----------------------
 drivers/net/hyperv/rndis_filter.c | 29 +++++++++++---------------
 4 files changed, 47 insertions(+), 68 deletions(-)

Comments

David Miller Dec. 13, 2017, 8:57 p.m. UTC | #1 
From: Stephen Hemminger <stephen@networkplumber.org>
Date: Tue, 12 Dec 2017 16:48:34 -0800

> This includes minor cleanup of code in send and receive path and
> also a new statistic to check for allocation failures. This also
> eliminates some of the extra RCU when not needed.
> 
> There is a theoritical bug where buffered data could be blocked
> for longer than necessary if the ring buffer got full. This
> has not been seen in the wild, found by inspection.
> 
> The reference count between net device and internal RNDIS
> is not needed.

Series applied, thanks Stephen.