2012-10-08 16:03:55

by Mohammed Shafi Shajakhan

[permalink] [raw]
Subject: [PATCH] mac80211: Use appropriate debug wrapper

From: Mohammed Shafi Shajakhan <[email protected]>

ieee80211_sta_expire will be called by both IBSS and mesh
interfaces to account for inactive stations, so it would be more
appropriate to use sta_dbg instead of ibss_dbg.

Signed-off-by: Mohammed Shafi Shajakhan <[email protected]>
---
net/mac80211/sta_info.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c
index 06fa75c..163cf22 100644
--- a/net/mac80211/sta_info.c
+++ b/net/mac80211/sta_info.c
@@ -880,8 +880,8 @@ void ieee80211_sta_expire(struct ieee80211_sub_if_data *sdata,
continue;

if (time_after(jiffies, sta->last_rx + exp_time)) {
- ibss_dbg(sdata, "expiring inactive STA %pM\n",
- sta->sta.addr);
+ sta_dbg(sta->sdata, "expiring inactive STA %pM\n",
+ sta->sta.addr);
WARN_ON(__sta_info_destroy(sta));
}
}
--
1.7.0.4



2012-10-10 07:50:31

by Johannes Berg

[permalink] [raw]
Subject: Re: [PATCH] mac80211: Use appropriate debug wrapper

On Mon, 2012-10-08 at 21:33 +0530, Mohammed Shafi Shajakhan wrote:
> From: Mohammed Shafi Shajakhan <[email protected]>
>
> ieee80211_sta_expire will be called by both IBSS and mesh
> interfaces to account for inactive stations, so it would be more
> appropriate to use sta_dbg instead of ibss_dbg.

Applied, thanks.

johannes