lkml.org 
[lkml]   [2005]   [Mar]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] USB: usbnet uses netif_msg_*() ethtool filtering
Linux Kernel Mailing List wrote:
> ChangeSet 1.2181.4.72, 2005/03/24 15:31:29-08:00, david-b@pacbell.net
>
> [PATCH] USB: usbnet uses netif_msg_*() ethtool filtering
>
> This converts most of the usbnet code to actually use the ethtool
> message flags. The ASIX code is left untouched, since there are
> a bunch of patches pending there ... that's where the remaining
> handful of "sparse -Wbitwise" warnings come from.
>
> Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

It would be nice if people at least CC'd me on net driver patches.


> @@ -2886,18 +2885,21 @@
> defer_kevent (dev, EVENT_RX_MEMORY);
> break;
> case -ENODEV:
> - devdbg (dev, "device gone");
> + if (netif_msg_ifdown (dev))
> + devdbg (dev, "device gone");
> netif_device_detach (dev->net);
> break;
> default:
> - devdbg (dev, "rx submit, %d", retval);
> + if (netif_msg_rx_err (dev))
> + devdbg (dev, "rx submit, %d", retval);
> tasklet_schedule (&dev->bh);
> break;
> case 0:
> __skb_queue_tail (&dev->rxq, skb);
> }
> } else {
> - devdbg (dev, "rx: stopped");
> + if (netif_msg_ifdown (dev))
> + devdbg (dev, "rx: stopped");
> retval = -ENOLINK;
> }
> spin_unlock_irqrestore (&dev->rxq.lock, lockflags);

netfi_msg_ifdown() is only for __interface__ up/down events; as such,
there should be only one message of this type in dev->open(), and one
message of this type in dev->stop().


> @@ -2963,9 +2967,8 @@
> // software-driven interface shutdown
> case -ECONNRESET: // async unlink
> case -ESHUTDOWN: // hardware gone
> -#ifdef VERBOSE
> - devdbg (dev, "rx shutdown, code %d", urb_status);
> -#endif
> + if (netif_msg_ifdown (dev))
> + devdbg (dev, "rx shutdown, code %d", urb_status);
> goto block;
>
> // we get controller i/o faults during khubd disconnect() delays.

ditto


> @@ -3026,9 +3030,8 @@
> /* software-driven interface shutdown */
> case -ENOENT: // urb killed
> case -ESHUTDOWN: // hardware gone
> -#ifdef VERBOSE
> - devdbg (dev, "intr shutdown, code %d", status);
> -#endif
> + if (netif_msg_ifdown (dev))
> + devdbg (dev, "intr shutdown, code %d", status);
> return;
>
> /* NOTE: not throttling like RX/TX, since this endpoint

ditto


> @@ -3044,7 +3047,7 @@
>
> memset(urb->transfer_buffer, 0, urb->transfer_buffer_length);
> status = usb_submit_urb (urb, GFP_ATOMIC);
> - if (status != 0)
> + if (status != 0 && netif_msg_timer (dev))
> deverr(dev, "intr resubmit --> %d", status);
> }
>

this looks more like a debugging message?


> @@ -3094,7 +3097,7 @@
>
> netif_stop_queue (net);
>
> - if (dev->msg_level >= 2)
> + if (netif_msg_ifdown (dev))
> devinfo (dev, "stop stats: rx/tx %ld/%ld, errs %ld/%ld",
> dev->stats.rx_packets, dev->stats.tx_packets,
> dev->stats.rx_errors, dev->stats.tx_errors

see -- this one is actually OK


> @@ -3110,7 +3113,8 @@
> && skb_queue_len (&dev->txq)
> && skb_queue_len (&dev->done)) {
> msleep(UNLINK_TIMEOUT_MS);
> - devdbg (dev, "waited for %d urb completions", temp);
> + if (netif_msg_ifdown (dev))
> + devdbg (dev, "waited for %d urb completions", temp);
> }
> dev->wait = NULL;
> remove_wait_queue (&unlink_wakeup, &wait);

not OK


> @@ -3142,16 +3146,19 @@
>
> // put into "known safe" state
> if (info->reset && (retval = info->reset (dev)) < 0) {
> - devinfo (dev, "open reset fail (%d) usbnet usb-%s-%s, %s",
> - retval,
> - dev->udev->bus->bus_name, dev->udev->devpath,
> + if (netif_msg_ifup (dev))
> + devinfo (dev,
> + "open reset fail (%d) usbnet usb-%s-%s, %s",
> + retval,
> + dev->udev->bus->bus_name, dev->udev->devpath,
> info->description);
> goto done;
> }
>
> // insist peer be connected
> if (info->check_connect && (retval = info->check_connect (dev)) < 0) {
> - devdbg (dev, "can't open; %d", retval);
> + if (netif_msg_ifup (dev))
> + devdbg (dev, "can't open; %d", retval);
> goto done;
> }
>

not OK


> @@ -3159,13 +3166,14 @@
> if (dev->interrupt) {
> retval = usb_submit_urb (dev->interrupt, GFP_KERNEL);
> if (retval < 0) {
> - deverr (dev, "intr submit %d", retval);
> + if (netif_msg_ifup (dev))
> + deverr (dev, "intr submit %d", retval);
> goto done;
> }
> }
>
> netif_start_queue (net);
> - if (dev->msg_level >= 2) {
> + if (netif_msg_ifup (dev)) {
> char *framing;
>
> if (dev->driver_info->flags & FLAG_FRAMING_NC)

not OK


> @@ -3526,7 +3541,7 @@
> if (urb != NULL)
> rx_submit (dev, urb, GFP_ATOMIC);
> }
> - if (temp != dev->rxq.qlen)
> + if (temp != dev->rxq.qlen && netif_msg_link (dev))
> devdbg (dev, "rxqlen %d --> %d",
> temp, dev->rxq.qlen);
> if (dev->rxq.qlen < qlen)

highly questionable.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-04-06 13:31    [W:0.869 / U:24.556 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site