The patch prints reason code for deauth/dissoc frames to give users
more ideas what's happened for the disconnection.
Signed-off-by: Zhu Yi <[email protected]>
---
net/mac80211/mlme.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
index d5006b2..0752e09 100644
--- a/net/mac80211/mlme.c
+++ b/net/mac80211/mlme.c
@@ -1134,7 +1134,8 @@ static void ieee80211_rx_mgmt_deauth(struct ieee80211_sub_if_data *sdata,
reason_code = le16_to_cpu(mgmt->u.deauth.reason_code);
if (ifsta->flags & IEEE80211_STA_AUTHENTICATED)
- printk(KERN_DEBUG "%s: deauthenticated\n", sdata->dev->name);
+ printk(KERN_DEBUG "%s: deauthenticated (Reason: %u)\n",
+ sdata->dev->name, reason_code);
if (ifsta->state == IEEE80211_STA_MLME_AUTHENTICATE ||
ifsta->state == IEEE80211_STA_MLME_ASSOCIATE ||
@@ -1166,7 +1167,8 @@ static void ieee80211_rx_mgmt_disassoc(struct ieee80211_sub_if_data *sdata,
reason_code = le16_to_cpu(mgmt->u.disassoc.reason_code);
if (ifsta->flags & IEEE80211_STA_ASSOCIATED)
- printk(KERN_DEBUG "%s: disassociated\n", sdata->dev->name);
+ printk(KERN_DEBUG "%s: disassociated (Reason: %u)\n",
+ sdata->dev->name, reason_code);
if (ifsta->state == IEEE80211_STA_MLME_ASSOCIATED) {
ifsta->state = IEEE80211_STA_MLME_ASSOCIATE;
--
1.5.3.6
On Tue, Oct 28, 2008 at 01:31:13PM +0200, Tomas Winkler wrote:
> Do anyone has idea how to push it to the user space?
> We already use ieee80211_sta_send_apinfo (SIOCGIWAP) for notification.
> Can be reason code attached without breaking user space? The
> iwreq_data it self has enough space to add this and this can be queued
> at the end.
Taken into account how broken WEXT events are already on 64/32-bit
systems, it would probably be better to add this as a new nl80211 event
which we will hopefully get in the near future. I'm looking for that to
fix the association info reporting, too (and issue that I fixed with
WEXT, but the fix was reverted due to userspace interop issues..).
--
Jouni Malinen PGP id EFC895FA
On Tue, 2008-10-28 at 15:58 +0800, Zhu Yi wrote:
> The patch prints reason code for deauth/dissoc frames to give users
> more ideas what's happened for the disconnection.
Indeed, that seems useful.
> Signed-off-by: Zhu Yi <[email protected]>
Acked-by: Johannes Berg <[email protected]>
> ---
> net/mac80211/mlme.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
> index d5006b2..0752e09 100644
> --- a/net/mac80211/mlme.c
> +++ b/net/mac80211/mlme.c
> @@ -1134,7 +1134,8 @@ static void ieee80211_rx_mgmt_deauth(struct ieee80211_sub_if_data *sdata,
> reason_code = le16_to_cpu(mgmt->u.deauth.reason_code);
>
> if (ifsta->flags & IEEE80211_STA_AUTHENTICATED)
> - printk(KERN_DEBUG "%s: deauthenticated\n", sdata->dev->name);
> + printk(KERN_DEBUG "%s: deauthenticated (Reason: %u)\n",
> + sdata->dev->name, reason_code);
>
> if (ifsta->state == IEEE80211_STA_MLME_AUTHENTICATE ||
> ifsta->state == IEEE80211_STA_MLME_ASSOCIATE ||
> @@ -1166,7 +1167,8 @@ static void ieee80211_rx_mgmt_disassoc(struct ieee80211_sub_if_data *sdata,
> reason_code = le16_to_cpu(mgmt->u.disassoc.reason_code);
>
> if (ifsta->flags & IEEE80211_STA_ASSOCIATED)
> - printk(KERN_DEBUG "%s: disassociated\n", sdata->dev->name);
> + printk(KERN_DEBUG "%s: disassociated (Reason: %u)\n",
> + sdata->dev->name, reason_code);
>
> if (ifsta->state == IEEE80211_STA_MLME_ASSOCIATED) {
> ifsta->state = IEEE80211_STA_MLME_ASSOCIATE;
On Tue, Oct 28, 2008 at 2:00 PM, Jouni Malinen <[email protected]> wrote:
> On Tue, Oct 28, 2008 at 01:31:13PM +0200, Tomas Winkler wrote:
>
>> Do anyone has idea how to push it to the user space?
>> We already use ieee80211_sta_send_apinfo (SIOCGIWAP) for notification.
>> Can be reason code attached without breaking user space? The
>> iwreq_data it self has enough space to add this and this can be queued
>> at the end.
>
> Taken into account how broken WEXT events are already on 64/32-bit
> systems, it would probably be better to add this as a new nl80211 event
> which we will hopefully get in the near future. I'm looking for that to
> fix the association info reporting, too (and issue that I fixed with
> WEXT, but the fix was reverted due to userspace interop issues..).
Okay, thanks for the info. will look into nl direction.
Tomas
On Tue, Oct 28, 2008 at 9:58 AM, Zhu Yi <[email protected]> wrote:
> The patch prints reason code for deauth/dissoc frames to give users
> more ideas what's happened for the disconnection.
Do anyone has idea how to push it to the user space?
We already use ieee80211_sta_send_apinfo (SIOCGIWAP) for notification.
Can be reason code attached without breaking user space? The
iwreq_data it self has enough space to add this and this can be queued
at the end.
struct {
sockaddr ap_addr; /** 0 address means association failed (Xeros
address dilemma)
__u16 reason_code /** IEEE reason code */
__u16 status_code /** IEEE reason code */
__u32 internal_reason /* mac80211/driver crashed/it's raining/too etc */
};
Thanks
Tomas
>
> Signed-off-by: Zhu Yi <[email protected]>
> ---
> net/mac80211/mlme.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
> index d5006b2..0752e09 100644
> --- a/net/mac80211/mlme.c
> +++ b/net/mac80211/mlme.c
> @@ -1134,7 +1134,8 @@ static void ieee80211_rx_mgmt_deauth(struct ieee80211_sub_if_data *sdata,
> reason_code = le16_to_cpu(mgmt->u.deauth.reason_code);
>
> if (ifsta->flags & IEEE80211_STA_AUTHENTICATED)
> - printk(KERN_DEBUG "%s: deauthenticated\n", sdata->dev->name);
> + printk(KERN_DEBUG "%s: deauthenticated (Reason: %u)\n",
> + sdata->dev->name, reason_code);
>
> if (ifsta->state == IEEE80211_STA_MLME_AUTHENTICATE ||
> ifsta->state == IEEE80211_STA_MLME_ASSOCIATE ||
> @@ -1166,7 +1167,8 @@ static void ieee80211_rx_mgmt_disassoc(struct ieee80211_sub_if_data *sdata,
> reason_code = le16_to_cpu(mgmt->u.disassoc.reason_code);
>
> if (ifsta->flags & IEEE80211_STA_ASSOCIATED)
> - printk(KERN_DEBUG "%s: disassociated\n", sdata->dev->name);
> + printk(KERN_DEBUG "%s: disassociated (Reason: %u)\n",
> + sdata->dev->name, reason_code);
>
> if (ifsta->state == IEEE80211_STA_MLME_ASSOCIATED) {
> ifsta->state = IEEE80211_STA_MLME_ASSOCIATE;
> --
> 1.5.3.6
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>