r8152: check RTL8152_UNPLUG and netif_running before autoresume
authorhayeswang <hayeswang@realtek.com>
Wed, 12 Nov 2014 02:05:05 +0000 (10:05 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 12 Nov 2014 19:49:08 +0000 (14:49 -0500)
If the device is unplugged or !netif_running(), the workqueue
doesn't need to wake the device, and could return directly.

Signed-off-by: Hayes Wang <hayeswang@realtek.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/r8152.c

index ad9dd7d175080e58a7312211b4e35d763ccd6862..0a30fd3f673d03458ced805de0e62f916bb525c1 100644 (file)
@@ -2857,15 +2857,18 @@ static void rtl_work_func_t(struct work_struct *work)
 {
        struct r8152 *tp = container_of(work, struct r8152, schedule.work);
 
+       /* If the device is unplugged or !netif_running(), the workqueue
+        * doesn't need to wake the device, and could return directly.
+        */
+       if (test_bit(RTL8152_UNPLUG, &tp->flags) || !netif_running(tp->netdev))
+               return;
+
        if (usb_autopm_get_interface(tp->intf) < 0)
                return;
 
        if (!test_bit(WORK_ENABLE, &tp->flags))
                goto out1;
 
-       if (test_bit(RTL8152_UNPLUG, &tp->flags))
-               goto out1;
-
        if (!mutex_trylock(&tp->control)) {
                schedule_delayed_work(&tp->schedule, 0);
                goto out1;