On Thu, 2008-02-21 at 12:32 -0500, John W. Linville wrote: > On Thu, Feb 21, 2008 at 12:16:02PM -0500, Pavel Roskin wrote: > > On Thu, 2008-02-21 at 11:08 +0100, Johannes Berg wrote: > > > - if (CONFIG_MAC80211_IBSS_DEBUG || net_ratelimit()) > > > +#ifndef CONFIG_MAC80211_IBSS_DEBUG > > > + if (net_ratelimit()) > > > +#endif > > > > It looks like you did a better job at guessing the intention of the code > > than me. > > > > Please apply this patch, not mine. > > Hmmm...maybe you are right. Unfortunately I already apply your > patch instead. I don't really care. It's just a KERN_DEBUG message after all. I see no problems with using net_ratelimit() in all cases. And I don't see any strong need to suppress this message. I don't expect it will happen often. But I'm not the one who will be looking at the logs :) -- Regards, Pavel Roskin - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
