From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Berg Subject: Re: [PATCH 1/4] mac80211: mesh: flush stations before beacons are stopped Date: Mon, 01 Aug 2016 11:38:50 +0200 Message-ID: <1470044330.3389.19.camel@sipsolutions.net> References: <20160628111307.8784-1-yanivma@ti.com> <20160628111307.8784-2-yanivma@ti.com> (sfid-20160628_131058_738638_5F9A964D) <1467184459.2461.1.camel@sipsolutions.net> (sfid-20160713_121132_350432_32DE2AAC) Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit Cc: "Hahn, Maital" To: "Machani, Yaniv" , "linux-kernel@vger.kernel.org" , "David S . Miller" , "linux-wireless@vger.kernel.org" , "netdev@vger.kernel.org" Return-path: In-Reply-To: (sfid-20160713_121132_350432_32DE2AAC) Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org 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 > > > > > > 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