On 02/13/2014 11:56 AM, Kalle Valo wrote:
> greearb@xxxxxxxxxxxxxxx writes:
>
>> These are too important to miss, and if we ever do have some
>> that are overly verbose
>
> The thing is that in some platforms excessive amount log messages will
> cause watchdog to forcefully reboot the device. So I'm hesitant to
> remove this.
>
> First of all, the goal is that ath10k will never issue even a single
> warning message. So even if you see just one, it means something is
> wrong. And just seeing the first few warnings should give you a pretty
> good hint what's going on.
I see lots of EAGAIN (-11) warnings sometime, and they come in bunches
in my scenario (with lots of vifs).
> Secondly, it's not like net_ratelimit() will hide all the warning
> messages. It will still show some of the messages even if it starts to
> throttle them.
In my case, it hid exactly the messages I needed to see in a hard
to reproduce bug, so I was a bit peeved :P
>> could ratelimit the call to ath10k_warn instead of doing the ratelimit
>> inside of ath10k_warn.
>
> Not sure if I got this idea. Only to ratelimit some of the warnings or
> what do you mean by ratelimiting it outside ath10k_warn()?
If there are warnings that are likely to spam continuously,
then ratelimit only those, so something like:
if (netratelimit()) {
ath10k_warn(...)
}
But, this is an easy patch for me to carry, so no worries if it
doesn't go upstream.
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
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
