Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1453106 > unrolled thread

Re: [PATCH 1/4] mac80211: mesh: flush stations before beacons are stopped

Started byJohannes Berg <johannes@sipsolutions.net>
First post2016-08-01 12:20 +0200
Last post2016-08-01 12:20 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 1/4] mac80211: mesh: flush stations before beacons are  stopped Johannes Berg <johannes@sipsolutions.net> - 2016-08-01 12:20 +0200

#1453106 — Re: [PATCH 1/4] mac80211: mesh: flush stations before beacons are stopped

FromJohannes Berg <johannes@sipsolutions.net>
Date2016-08-01 12:20 +0200
SubjectRe: [PATCH 1/4] mac80211: mesh: flush stations before beacons are stopped
Message-ID<s1iud-500-7@gated-at.bofh.it>
On Wed, 2016-07-13 at 10:11 +0000, Machani, Yaniv wrote:
> On Wed, Jun 29, 2016 at 10:14:19, Johannes Berg wrote:
> > Cc: Hahn, Maital
> > Subject: Re: [PATCH 1/4] mac80211: mesh: flush stations before
> > beaconsĀ 
> > are stopped
> > 
> > On Tue, 2016-06-28 at 14:13 +0300, Yaniv Machani wrote:
> > > From: Maital Hahn <maitalm@ti.com>
> > > 
> > > Some drivers (e.g. wl18xx) expect that the last stage in theĀ 
> > > de-initialization process will be stopping the beacons, similar
> > > to ap.
> > > Update ieee80211_stop_mesh() flow accordingly.
> > > 
> > How well have you tested that with other drivers?
> > 
> 
> Sorry for the delayed response (I've been out) and thanks for your
> comments,
> I have tested it with RT3572 as well, and didn't see any issue.
> I'll update the comment to reflect that.
> 

I'm actually reasonably sure that it *must* work, similiar to the AP
mode change, since it's always valid to remove stations while the the
mesh beacon is still active.

I hoped you'd actually figure out that line of reasoning and put it
into the commit message :)

johannes

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web