I have no experience with brcmfmac, but ... > > I was helping Keith out here and wanted to provide a bit more > information. I found there were a few differences between IWD and > wpa_supplicant related to scanning which may aid in figuring out why > brcmfmac is behaving this way: > > - IWD scans using the wdev ID where wpa_supplicant uses ifindex. Not > sure if this has anything to do with the difference behavior. This is not even visible to the driver, it's entirely resolved in nl80211, so no impact here. > - Passive scans (which IWD prefers) seem to exacerbate the behavior. > Simple testing using "wpa_cli scan" showed wpa_supplicant was only using > active scans. I also tested with iw and saw repeatable disconnects when > passive scanning. Disconnects while using active scans (using iw) were > much less frequent. This makes sense, especially if it's __ap rather than __p2p_go type, since it *has* to be off the channel for some time -- especially for passive scans it has to be off-channel for more than a typical interval to even do scanning correctly. > - Scanning with IWD, wpa_supplicant, or iw, passive or active, always > resulted in beacon loss for clients connected to the AP. This was 100% > guaranteed. The clients just could recover when active scans were used > over passive. But either way this does not seem like normal behavior, > the AP interface should still be beaconing on its active channel during > a scan right? That's "normal" in the sense that you have to be off the channel for scanning, and if you're off the channel you can't transmit beacons on it? For P2P GO rather than AP it should publish NoA descriptors in the beacon to let clients know there won't be a beacon. Now it's perhaps possible to time - especially active - scanning so you can still beacon somewhat and inbetween, but I suppose the firmware doesn't do that here. In fact even outside of the beaconing, APs aren't expected to be off- channel, clients can send data to them after all. Again P2P GO solves that with NoA, but the spec itself has no good way to solve this and I'm not even sure it would even want to. In any case, you could argue that starting AP and client and then scanning is pretty much _asking_ for trouble. > If this isn't possible or can't be done reliably then > should the interface combinations be changed to disallow concurrent sta > + AP mode interfaces? Maybe it could restrict it to P2P GO instead of AP? But then people will anyway just notice that they can use P2P GO and connect arbitrary clients to it (not just P2P client), then those clients won't honour the NoA because they're not P2P, and then you're back to the exact same situation... johannes
