lkml.org 
[lkml]   [2011]   [Jun]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH 06/11] drivers/net: use printk_ratelimited() instead of printk_ratelimit()
From
Date
On Thu, 2011-06-16 at 14:12 +0200, Manuel Zerpies wrote:
> Since printk_ratelimit() shouldn't be used anymore (see comment in
> include/linux/printk.h), replace it with printk_ratelimited()
[]
> diff --git a/drivers/net/sc92031.c b/drivers/net/sc92031.c
[]
> @@ -536,8 +537,7 @@ static bool _sc92031_check_media(struct net_device *dev)
>
> netif_carrier_on(dev);
>
> - if (printk_ratelimit())
> - printk(KERN_INFO "%s: link up, %sMbps, %s-duplex\n",
> + printk_ratelimited(KERN_INFO "%s: link up, %sMbps, %s-duplex\n",
> dev->name,
> speed_100 ? "100" : "10",
> duplex_full ? "full" : "half");

All these might be better using net_ratelimit and netdev_<level>

if (net_ratelimit())
netdev_info(dev, "link up, %uMbps, %s-duplex\n",
speed_100 ? 100 : 10,
duplex_full ? "full" : "half");

> @@ -549,8 +549,7 @@ static bool _sc92031_check_media(struct net_device *dev)
>
> _sc92031_disable_tx_rx(dev);
>
> - if (printk_ratelimit())
> - printk(KERN_INFO "%s: link down\n", dev->name);
> + printk_ratelimited(KERN_INFO "%s: link down\n", dev->name);

etc...

netdev_<level>_ratelimited (and netdev_<level>_once)
could instead be added and used I suppose.




\
 
 \ /
  Last update: 2011-06-16 18:47    [W:0.054 / U:0.136 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site