lkml.org 
[lkml]   [2016]   [Jul]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 1/4] mac80211: mesh: flush stations before beacons are stopped
Date
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.

Thanks,
Yaniv

> Changing behaviour to something a single driver desires isn't
> necessarily the best thing to do, since there always are multiple drivers.
>
> If you're able to demonstrate that it works with the other drivers I'm
> willing to take that - the change makes sense after all, and it seems
> drivers must support this ordering since peers are also removed
> dynamically... But still. Don't just make a change like that without
> even giving any indication why you think it's fine for other drivers!
>
> johannes


\
 
 \ /
  Last update: 2016-07-13 13:01    [W:0.070 / U:0.832 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site