* [PATCH] Print text for disassociation reason.
@ 2017-02-06 13:53 Arkadiusz Miskiewicz
2017-02-08 8:18 ` Johannes Berg
0 siblings, 1 reply; 2+ messages in thread
From: Arkadiusz Miskiewicz @ 2017-02-06 13:53 UTC (permalink / raw)
To: Johannes Berg; +Cc: linux-wireless
Hi.
Don't know why it wasn't printed there with ieee80211_get_reason_code_strin=
g in first
place. Works for me:
kernel: wlan0: disassociated from 04:b0:20:33:ff:1f (Reason: 34=3DDISASSOC_=
LOW_ACK)
ps. can't send patch in normal way due to postmaster@vger weirdness, so ins=
erted
below
=46rom c9b55bb44fe0b902f376a41fa930c9a67a438511 Mon Sep 17 00:00:00 2001
=46rom: =3D?UTF-8?q?Arkadiusz=3D20Mi=3DC5=3D9Bkiewicz?=3D <arekm@maven.pl>
Date: Mon, 6 Feb 2017 14:45:15 +0100
Subject: [PATCH] Print text for disassociation reason.
MIME-Version: 1.0
Content-Type: text/plain; charset=3DUTF-8
Content-Transfer-Encoding: 8bit
When disassociation happens only numeric reason is printed
in ieee80211_rx_mgmt_disassoc(). Add text variant, too.
Signed-off-by: Arkadiusz Mi=C5=9Bkiewicz <arekm@maven.pl>
=2D--
net/mac80211/mlme.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
index 098ce9b179ee..fcf8d0aa66ec 100644
=2D-- a/net/mac80211/mlme.c
+++ b/net/mac80211/mlme.c
@@ -2801,8 +2801,9 @@ static void ieee80211_rx_mgmt_disassoc(struct ieee802=
11_sub_if_data *sdata,
=20
reason_code =3D le16_to_cpu(mgmt->u.disassoc.reason_code);
=20
=2D sdata_info(sdata, "disassociated from %pM (Reason: %u)\n",
=2D mgmt->sa, reason_code);
+ sdata_info(sdata, "disassociated from %pM (Reason: %u=3D%s)\n",
+ mgmt->sa, reason_code,
+ ieee80211_get_reason_code_string(reason_code));
=20
ieee80211_set_disassoc(sdata, 0, 0, false, NULL);
=20
=2D-=20
2.11.0
=2D-=20
Arkadiusz Mi=C5=9Bkiewicz, arekm / ( maven.pl | pld-linux.org )
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Print text for disassociation reason.
2017-02-06 13:53 [PATCH] Print text for disassociation reason Arkadiusz Miskiewicz
@ 2017-02-08 8:18 ` Johannes Berg
0 siblings, 0 replies; 2+ messages in thread
From: Johannes Berg @ 2017-02-08 8:18 UTC (permalink / raw)
To: arekm; +Cc: linux-wireless
> Don't know why it wasn't printed there with
> ieee80211_get_reason_code_string in first
> place. Works for me:
>
> kernel: wlan0: disassociated from 04:b0:20:33:ff:1f (Reason:
> 34=DISASSOC_LOW_ACK)
This patch needs a "mac80211: " prefix, and I'd prefer no . at the end
(but that's not a hard rule)
> ps. can't send patch in normal way due to postmaster@vger weirdness,
> so inserted
> below
>
> From c9b55bb44fe0b902f376a41fa930c9a67a438511 Mon Sep 17 00:00:00
> 2001
> From: =?UTF-8?q?Arkadiusz=20Mi=C5=9Bkiewicz?= <arekm@maven.pl>
> Date: Mon, 6 Feb 2017 14:45:15 +0100
> Subject: [PATCH] Print text for disassociation reason.
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> When disassociation happens only numeric reason is printed
> in ieee80211_rx_mgmt_disassoc(). Add text variant, too.
>
> Signed-off-by: Arkadiusz Miśkiewicz <arekm@maven.pl>
This is useless to me, please resubmit without all the fluff. You can
insert a "From: " line into the first line of the email *body* to get
the author correct, if you can't actually send it from the correct
email address. git send-email will even do that for you.
johannes
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-02-08 8:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-06 13:53 [PATCH] Print text for disassociation reason Arkadiusz Miskiewicz
2017-02-08 8:18 ` Johannes Berg
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.