ldmvsw: better use of link up and down on ldom vswitch
authorShannon Nelson <shannon.nelson@oracle.com>
Tue, 14 Mar 2017 17:24:39 +0000 (10:24 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 17 Mar 2017 03:29:54 +0000 (20:29 -0700)
commit867fa150f8f7ee6e9e5a9ab768e2d0dc675a968b
treef0890a4d8268bad125dd68199f9a6e24352bdffa
parent19ddde1eeca1ee81f4add5e04da66055e09281ac
ldmvsw: better use of link up and down on ldom vswitch

When an ldom VM is bound, the network vswitch infrastructure is set up for
it, but was being forced 'UP' by the userland switch configuration script.
When 'UP' but not actually connected to a running VM, the ipv6 neighbor
probes fail (not a horrible thing) and start cluttering up the kernel logs.
Funny thing: these are debug messages that never actually show up, but
we do see the net_ratelimited messages that say N callbacks were
suppressed.

This patch defers the netif_carrier_on() until an actual link has been
established with the VM, as indicated by receiving an LDC_EVENT_UP from
the underlying LDC protocol.  Similarly, we take the link down when we
see the LDC_EVENT_RESET.  Now when we see the ndo_open(), we reset the
link to get things talking again.

Orabug: 25525312

Signed-off-by: Shannon Nelson <shannon.nelson@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sun/ldmvsw.c
drivers/net/ethernet/sun/sunvnet_common.c
drivers/net/ethernet/sun/sunvnet_common.h