* [PATCH] mac80211: Fix addition of mesh configuration element
@ 2016-12-26 16:17 Ilan Peer
2016-12-26 21:49 ` Masashi Honma
2017-01-02 10:59 ` Johannes Berg
0 siblings, 2 replies; 4+ messages in thread
From: Ilan Peer @ 2016-12-26 16:17 UTC (permalink / raw)
To: johannes; +Cc: linux-wireless, masashi.honma, Ilan Peer
The code was setting the capabilities byte to zero,
after it was already properly set previously. Fix it.
The bug was found while debugging hwsim mesh tests failures
that happened in commit 76f43b4 (mac80211: Remove invalid flag
operations in mesh TSF synchronization).
Signed-off-by: Ilan Peer <ilan.peer@intel.com>
---
net/mac80211/mesh.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/net/mac80211/mesh.c b/net/mac80211/mesh.c
index cc2a63b..9c23172 100644
--- a/net/mac80211/mesh.c
+++ b/net/mac80211/mesh.c
@@ -279,8 +279,6 @@ int mesh_add_meshconf_ie(struct ieee80211_sub_if_data *sdata,
/* Mesh PS mode. See IEEE802.11-2012 8.4.2.100.8 */
*pos |= ifmsh->ps_peers_deep_sleep ?
IEEE80211_MESHCONF_CAPAB_POWER_SAVE_LEVEL : 0x00;
- *pos++ = 0x00;
-
return 0;
}
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] mac80211: Fix addition of mesh configuration element
2016-12-26 16:17 [PATCH] mac80211: Fix addition of mesh configuration element Ilan Peer
@ 2016-12-26 21:49 ` Masashi Honma
2017-01-02 10:59 ` Johannes Berg
1 sibling, 0 replies; 4+ messages in thread
From: Masashi Honma @ 2016-12-26 21:49 UTC (permalink / raw)
To: Ilan Peer, johannes; +Cc: linux-wireless
On 2016年12月27日 01:17, Ilan Peer wrote:
> The code was setting the capabilities byte to zero,
> after it was already properly set previously. Fix it.
>
> The bug was found while debugging hwsim mesh tests failures
> that happened in commit 76f43b4 (mac80211: Remove invalid flag
> operations in mesh TSF synchronization).
Thanks!
Reviewed-by: Masashi Honma <masashi.honma@gmail.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mac80211: Fix addition of mesh configuration element
2016-12-26 16:17 [PATCH] mac80211: Fix addition of mesh configuration element Ilan Peer
2016-12-26 21:49 ` Masashi Honma
@ 2017-01-02 10:59 ` Johannes Berg
2017-01-02 11:28 ` Peer, Ilan
1 sibling, 1 reply; 4+ messages in thread
From: Johannes Berg @ 2017-01-02 10:59 UTC (permalink / raw)
To: Ilan Peer; +Cc: linux-wireless, masashi.honma
On Mon, 2016-12-26 at 18:17 +0200, Ilan Peer wrote:
> The code was setting the capabilities byte to zero,
> after it was already properly set previously. Fix it.
>
> The bug was found while debugging hwsim mesh tests failures
> that happened in commit 76f43b4 (mac80211: Remove invalid flag
> operations in mesh TSF synchronization).
>
Applied, thanks.
When you have the commit already, please add a Fixes tag, and also use
12 hex digits to abbreviate SHA1s please. (I've done that now)
johannes
^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH] mac80211: Fix addition of mesh configuration element
2017-01-02 10:59 ` Johannes Berg
@ 2017-01-02 11:28 ` Peer, Ilan
0 siblings, 0 replies; 4+ messages in thread
From: Peer, Ilan @ 2017-01-02 11:28 UTC (permalink / raw)
To: Johannes Berg; +Cc: linux-wireless@vger.kernel.org, masashi.honma@gmail.com
PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBKb2hhbm5lcyBCZXJnIFttYWls
dG86am9oYW5uZXNAc2lwc29sdXRpb25zLm5ldF0NCj4gU2VudDogTW9uZGF5LCBKYW51YXJ5IDAy
LCAyMDE3IDEzOjAwDQo+IFRvOiBQZWVyLCBJbGFuIDxpbGFuLnBlZXJAaW50ZWwuY29tPg0KPiBD
YzogbGludXgtd2lyZWxlc3NAdmdlci5rZXJuZWwub3JnOyBtYXNhc2hpLmhvbm1hQGdtYWlsLmNv
bQ0KPiBTdWJqZWN0OiBSZTogW1BBVENIXSBtYWM4MDIxMTogRml4IGFkZGl0aW9uIG9mIG1lc2gg
Y29uZmlndXJhdGlvbiBlbGVtZW50DQo+IA0KPiBPbiBNb24sIDIwMTYtMTItMjYgYXQgMTg6MTcg
KzAyMDAsIElsYW4gUGVlciB3cm90ZToNCj4gPiBUaGUgY29kZSB3YXMgc2V0dGluZyB0aGUgY2Fw
YWJpbGl0aWVzIGJ5dGUgdG8gemVybywgYWZ0ZXIgaXQgd2FzDQo+ID4gYWxyZWFkeSBwcm9wZXJs
eSBzZXQgcHJldmlvdXNseS4gRml4IGl0Lg0KPiA+DQo+ID4gVGhlIGJ1ZyB3YXMgZm91bmQgd2hp
bGUgZGVidWdnaW5nIGh3c2ltIG1lc2ggdGVzdHMgZmFpbHVyZXMgdGhhdA0KPiA+IGhhcHBlbmVk
IGluIGNvbW1pdCA3NmY0M2I0IChtYWM4MDIxMTogUmVtb3ZlIGludmFsaWQgZmxhZyBvcGVyYXRp
b25zDQo+ID4gaW4gbWVzaCBUU0Ygc3luY2hyb25pemF0aW9uKS4NCj4gPg0KPiBBcHBsaWVkLCB0
aGFua3MuDQo+IA0KPiBXaGVuIHlvdSBoYXZlIHRoZSBjb21taXQgYWxyZWFkeSwgcGxlYXNlIGFk
ZCBhIEZpeGVzIHRhZywgYW5kIGFsc28gdXNlDQo+IDEyIGhleCBkaWdpdHMgdG8gYWJicmV2aWF0
ZSBTSEExcyBwbGVhc2UuIChJJ3ZlIGRvbmUgdGhhdCBub3cpDQo+IA0KDQpTdXJlLiBUaGFua3Mu
DQoNCklsYW4uDQo=
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-01-02 11:28 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-26 16:17 [PATCH] mac80211: Fix addition of mesh configuration element Ilan Peer
2016-12-26 21:49 ` Masashi Honma
2017-01-02 10:59 ` Johannes Berg
2017-01-02 11:28 ` Peer, Ilan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).