Commit 53fa1a6f authored by Haiyang Zhang's avatar Haiyang Zhang Committed by David S. Miller

hv_netvsc: Fix the carrier state error when data path is off

When the VF NIC is opened, the synthetic NIC's carrier state is set to
off. This tells the host to transitions data path to the VF device. But
if startup script or user manipulates the admin state of the netvsc
device directly for example:
        # ifconfig eth0 down
	# ifconfig eth0 up
Then the carrier state of the synthetic NIC would be on, even though the
data path was still over the VF NIC. This patch sets the carrier state
of synthetic NIC with consideration of the related VF state.
Signed-off-by: default avatarHaiyang Zhang <haiyangz@microsoft.com>
Reviewed-by: default avatarStephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent dedb459e
...@@ -717,6 +717,8 @@ struct net_device_context { ...@@ -717,6 +717,8 @@ struct net_device_context {
u32 vf_alloc; u32 vf_alloc;
/* Serial number of the VF to team with */ /* Serial number of the VF to team with */
u32 vf_serial; u32 vf_serial;
bool datapath; /* 0 - synthetic, 1 - VF nic */
}; };
/* Per channel data */ /* Per channel data */
......
...@@ -57,6 +57,8 @@ void netvsc_switch_datapath(struct net_device *ndev, bool vf) ...@@ -57,6 +57,8 @@ void netvsc_switch_datapath(struct net_device *ndev, bool vf)
sizeof(struct nvsp_message), sizeof(struct nvsp_message),
(unsigned long)init_pkt, (unsigned long)init_pkt,
VM_PKT_DATA_INBAND, 0); VM_PKT_DATA_INBAND, 0);
net_device_ctx->datapath = vf;
} }
static struct netvsc_device *alloc_net_device(void) static struct netvsc_device *alloc_net_device(void)
......
...@@ -68,7 +68,8 @@ static void netvsc_set_multicast_list(struct net_device *net) ...@@ -68,7 +68,8 @@ static void netvsc_set_multicast_list(struct net_device *net)
static int netvsc_open(struct net_device *net) static int netvsc_open(struct net_device *net)
{ {
struct netvsc_device *nvdev = net_device_to_netvsc_device(net); struct net_device_context *ndev_ctx = netdev_priv(net);
struct netvsc_device *nvdev = ndev_ctx->nvdev;
struct rndis_device *rdev; struct rndis_device *rdev;
int ret = 0; int ret = 0;
...@@ -84,7 +85,7 @@ static int netvsc_open(struct net_device *net) ...@@ -84,7 +85,7 @@ static int netvsc_open(struct net_device *net)
netif_tx_wake_all_queues(net); netif_tx_wake_all_queues(net);
rdev = nvdev->extension; rdev = nvdev->extension;
if (!rdev->link_state) if (!rdev->link_state && !ndev_ctx->datapath)
netif_carrier_on(net); netif_carrier_on(net);
return ret; return ret;
...@@ -1284,6 +1285,7 @@ static void netvsc_link_change(struct work_struct *w) ...@@ -1284,6 +1285,7 @@ static void netvsc_link_change(struct work_struct *w)
case RNDIS_STATUS_MEDIA_CONNECT: case RNDIS_STATUS_MEDIA_CONNECT:
if (rdev->link_state) { if (rdev->link_state) {
rdev->link_state = false; rdev->link_state = false;
if (!ndev_ctx->datapath)
netif_carrier_on(net); netif_carrier_on(net);
netif_tx_wake_all_queues(net); netif_tx_wake_all_queues(net);
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment