mbox series

[net-next,v2,0/2] hv_netvsc: fix a hang on channel/mtu changes

Message ID 20171102103531.5514-1-vkuznets@redhat.com
Headers show
Series hv_netvsc: fix a hang on channel/mtu changes | expand

Message

Vitaly Kuznetsov Nov. 2, 2017, 10:35 a.m. UTC 
It was found that netvsc driver doesn't survive e.g.

# while true; do ethtool -L eth0 combined 4; ethtool -L eth0 combined 8; done"

test. I was able to identify a hang in guest/host communication, it is
fixed by PATCH1 of this series. PATCH2 is a cosmetic change masking
unneeded messages.

Changes since v1:
- Throw away patches 2 and 3 of the original series as one is unneeded and
  the other is not justified [Eric Dumazet, Stephen Hemminger] so I'm only
  fixing the hang now, the crash doesn't reproduce. Will keep an eye on it.

Vitaly Kuznetsov (2):
  hv_netvsc: netvsc_teardown_gpadl() split
  hv_netvsc: hide warnings about uninitialized/missing rndis device

 drivers/net/hyperv/netvsc.c       | 69 ++++++++++++++++++++-------------------
 drivers/net/hyperv/rndis_filter.c |  4 +--
 2 files changed, 38 insertions(+), 35 deletions(-)

Comments

David Miller Nov. 8, 2017, 1:31 a.m. UTC | #1 
From: Vitaly Kuznetsov <vkuznets@redhat.com>
Date: Thu,  2 Nov 2017 11:35:29 +0100

> It was found that netvsc driver doesn't survive e.g.
> 
> # while true; do ethtool -L eth0 combined 4; ethtool -L eth0 combined 8; done"
> 
> test. I was able to identify a hang in guest/host communication, it is
> fixed by PATCH1 of this series. PATCH2 is a cosmetic change masking
> unneeded messages.
> 
> Changes since v1:
> - Throw away patches 2 and 3 of the original series as one is unneeded and
>   the other is not justified [Eric Dumazet, Stephen Hemminger] so I'm only
>   fixing the hang now, the crash doesn't reproduce. Will keep an eye on it.

Series applied to net-next, thank you.