hv_netvsc: rework link status change handling
authorVitaly Kuznetsov <vkuznets@redhat.com>
Fri, 27 Nov 2015 10:39:55 +0000 (11:39 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 Dec 2015 19:58:07 +0000 (14:58 -0500)
commit27a70af3f4cf633a1b86c0ac7b426e2fe16ad2e5
treee4dc499878da293844034dc90d2894189ae2bda6
parent77b75f4d8cf105b599beef38724f8171e557919d
hv_netvsc: rework link status change handling

There are several issues in hv_netvsc driver with regards to link status
change handling:
- RNDIS_STATUS_NETWORK_CHANGE results in calling userspace helper doing
  '/etc/init.d/network restart' and this is inappropriate and broken for
  many reasons.
- link_watch infrastructure only sends one notification per second and
  in case of e.g. paired disconnect/connect events we get only one
  notification with last status. This makes it impossible to handle such
  situations in userspace.

Redo link status changes handling in the following way:
- Create a list of reconfig events in network device context.
- On a reconfig event add it to the list of events and schedule
  netvsc_link_change().
- In netvsc_link_change() ensure 2-second delay between link status
  changes.
- Handle RNDIS_STATUS_NETWORK_CHANGE as a paired disconnect/connect event.

Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/hyperv/hyperv_net.h
drivers/net/hyperv/netvsc_drv.c