usbnet: silence an unnecessary warning
authorOliver Neukum <oneukum@suse.com>
Wed, 17 Jan 2018 15:33:54 +0000 (16:33 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 22 Jan 2018 20:32:09 +0000 (15:32 -0500)
That a kevent could not be scheduled is not an error.
Such handlers must be able to deal with multiple events anyway.
As the successful scheduling of a work is a debug event, make
the failure debug priority, too.

V2: coding style

Signed-off-by: Oliver Neukum <oneukum@suse.com>
Reported-by: Cristian Caravena <caravena@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/usbnet.c

index d56fe32bf48dea8c617c011d5bd6ddc8d9d5270f..8a22ff67b0268a588428c61c6a6211e3c6c2a02a 100644 (file)
@@ -457,12 +457,10 @@ static enum skb_state defer_bh(struct usbnet *dev, struct sk_buff *skb,
 void usbnet_defer_kevent (struct usbnet *dev, int work)
 {
        set_bit (work, &dev->flags);
-       if (!schedule_work (&dev->kevent)) {
-               if (net_ratelimit())
-                       netdev_err(dev->net, "kevent %d may have been dropped\n", work);
-       } else {
+       if (!schedule_work (&dev->kevent))
+               netdev_dbg(dev->net, "kevent %d may have been dropped\n", work);
+       else
                netdev_dbg(dev->net, "kevent %d scheduled\n", work);
-       }
 }
 EXPORT_SYMBOL_GPL(usbnet_defer_kevent);