From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bob Copeland Subject: Re: [PATCH 1/4] mac80211: mesh: flush stations before beacons are stopped Date: Wed, 13 Jul 2016 09:33:38 -0400 Message-ID: <20160713133338.GD7819@localhost> References: <20160628111307.8784-1-yanivma@ti.com> <20160628111307.8784-2-yanivma@ti.com> <1467184459.2461.1.camel@sipsolutions.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Johannes Berg , "linux-kernel@vger.kernel.org" , "David S . Miller" , "linux-wireless@vger.kernel.org" , "netdev@vger.kernel.org" , "Hahn, Maital" To: "Machani, Yaniv" Return-path: Received: from mail-it0-f50.google.com ([209.85.214.50]:37529 "EHLO mail-it0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751305AbcGMNe7 (ORCPT ); Wed, 13 Jul 2016 09:34:59 -0400 Received: by mail-it0-f50.google.com with SMTP id f6so43137699ith.0 for ; Wed, 13 Jul 2016 06:34:17 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Wed, Jul 13, 2016 at 10:11:25AM +0000, Machani, Yaniv wrote: > > > 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'll give this a test on ath10k and wcn36xx as they are the ones most likely to care about ordering. -- Bob Copeland %% http://bobcopeland.com/