Return-path: Received: from youngberry.canonical.com ([91.189.89.112]:44961 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751368AbdJQOKt (ORCPT ); Tue, 17 Oct 2017 10:10:49 -0400 Received: from mail-vk0-f50.google.com ([209.85.213.50]) by youngberry.canonical.com with esmtpsa (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.76) (envelope-from ) id 1e4Sa0-0007yM-L5 for linux-wireless@vger.kernel.org; Tue, 17 Oct 2017 14:10:48 +0000 Received: by mail-vk0-f50.google.com with SMTP id r82so1172484vke.6 for ; Tue, 17 Oct 2017 07:10:48 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1508233890.10607.70.camel@sipsolutions.net> <1508237298.10607.76.camel@sipsolutions.net> <1508246019.10607.77.camel@sipsolutions.net> From: Jesse Sung Date: Tue, 17 Oct 2017 22:10:47 +0800 Message-ID: (sfid-20171017_161053_181432_00B2D558) Subject: Re: Commit 0711d638 breaks mwifiex To: Johannes Berg Cc: Amitkumar Karwar , Nishant Sarmukadam , Ilan Peer , Anthony Wong , Jason Yen , Terry.Wey@dell.com, linux-wireless@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, Oct 17, 2017 at 10:08 PM, Jesse Sung wrote: > On Tue, Oct 17, 2017 at 9:13 PM, Johannes Berg > wrote: >> >> On Tue, 2017-10-17 at 21:07 +0800, Jesse Sung wrote: >> > >> > > Not really quite sure about it yet, but that should address the >> > > issue? >> > >> > A very rough test by issuing reassociate in wpa_cli: >> > mwifiex works after reassociate - looks good >> >> Ok. Discussing this with Ilan, we realized that this was buggy, and >> came up with this patch instead: >> >> https://p.sipsolutions.net/8f9d5917a5cbe4b3.txt >> >> Can you also give that a spin? > > Issued four reassociate, the network interface is still alive. Also with this > patch it stays with BSSID1 and I don't see any "Association request to > the driver failed" in the process. Correction: I can still see "Association request to the driver failed" and switching between BSSIDs with more reassociates but I think that shouldn't be an issue?