Re: [PATCH v3 3/4] mac80211-hwsim: add rate-limited debugging for rx-netlink
From: Ben Greear <hidden>
Date: 2017-03-29 15:39:29
On 03/29/2017 01:46 AM, Johannes Berg wrote:
On Thu, 2017-03-23 at 16:26 -0700, greearb@candelatech.com wrote:quoted
From: Ben Greear <redacted> This makes it easier to understand why wmediumd (or similar) is getting errors when sending frames to the kernel. Signed-off-by: Ben Greear <redacted> --- drivers/net/wireless/mac80211_hwsim.c | 55 +++++++++++++++++++++++++++-------- 1 file changed, 43 insertions(+), 12 deletions(-)diff --git a/drivers/net/wireless/mac80211_hwsim.cb/drivers/net/wireless/mac80211_hwsim.c index 234df8c..84dcddf 100644--- a/drivers/net/wireless/mac80211_hwsim.c +++ b/drivers/net/wireless/mac80211_hwsim.c@@ -137,6 +137,12 @@ static int regtest = HWSIM_REGTEST_DISABLED; module_param(regtest, int, 0444); MODULE_PARM_DESC(regtest, "The type of regulatory test we want torun"); +DEFINE_RATELIMIT_STATE(hwsim_ratelimit_state, 5 * HZ, 10); +int hwsim_ratelimit(void) +{ + return __ratelimit(&hwsim_ratelimit_state); +} + static const char *hwsim_alpha2s[] = { "FI", "AL",@@ -1649,7 +1655,7 @@ static int mac80211_hwsim_config(structieee80211_hw *hw, u32 changed) if (conf->chandef.chan) wiphy_debug(hw->wiphy, - "%s (freq=%d(%d - %d)/%s idle=%d ps=%d smps=%s)\n", + "%s (chandef-chan freq=%d(%d - %d)/%s idle=%d ps=%d smps=%s)\n", __func__, conf->chandef.chan->center_freq, conf->chandef.center_freq1,@@ -1660,7 +1666,7 @@ static int mac80211_hwsim_config(structieee80211_hw *hw, u32 changed) smps_modes[conf->smps_mode]); else wiphy_debug(hw->wiphy, - "%s (freq=0 idle=%d ps=%d smps=%s)\n", + "%s (no-chandef-chan freq=0 idle=%d ps=%d smps=%s)\n",This seems unrelated?
It can be cut out of the patch. Want me to re-send? Thanks, Ben -- Ben Greear [off-list ref] Candela Technologies Inc http://www.candelatech.com