diff options
| author | Haiyang Zhang | 2011-09-01 21:19:42 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2011-09-06 20:56:18 +0200 |
| commit | c4b6a2eaf1c14810a4803d658f68614365978738 (patch) | |
| tree | 7831df8c49478771b9b82ca371cec7f914b31b19 /drivers/staging/hv/netvsc_drv.c | |
| parent | staging: hv: convert dev_<loglevel> to netdev_<loglevel> in netvsc (diff) | |
| download | kernel-qcow2-linux-c4b6a2eaf1c14810a4803d658f68614365978738.tar.gz kernel-qcow2-linux-c4b6a2eaf1c14810a4803d658f68614365978738.tar.xz kernel-qcow2-linux-c4b6a2eaf1c14810a4803d658f68614365978738.zip | |
staging: hv: fix a kernel warning in netvsc_linkstatus_callback()
netif_notify_peers() caused a kernel warning in netvsc_linkstatus_callback(),
because netvsc_linkstatus_callback() is within IRQ context. So we move
the first call to netif_notify_peers() into queued work as well, but with
zero delay.
In addition to "staging-next", this should also be back-ported to stable
kernels 2.6.32 and later.
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/hv/netvsc_drv.c')
| -rw-r--r-- | drivers/staging/hv/netvsc_drv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv/netvsc_drv.c index 30a0cb2c0fea..e6513be00c4d 100644 --- a/drivers/staging/hv/netvsc_drv.c +++ b/drivers/staging/hv/netvsc_drv.c @@ -214,8 +214,8 @@ void netvsc_linkstatus_callback(struct hv_device *device_obj, if (status == 1) { netif_carrier_on(net); netif_wake_queue(net); - netif_notify_peers(net); ndev_ctx = netdev_priv(net); + schedule_delayed_work(&ndev_ctx->dwork, 0); schedule_delayed_work(&ndev_ctx->dwork, msecs_to_jiffies(20)); } else { netif_carrier_off(net); |