On Tue, Mar 22, 2016 at 3:53 AM, Patrik Flykt <Patrik.Flykt(a)linux.intel.com>
On Thu, 2016-03-17 at 17:33 -0700, naveen(a)nestlabs.com wrote:
> From: Naveen Singh <nasingh(a)google.com>
> In case network path is not NULL, network should only be removed
> if the new network (to be added) is different from what is sitting
> in wpa_supplicant. This would avoid two DBUS calls: AddNetwork and
This was also applied, thanks!
I think this patch standalone would break things. This patch needs to be
applied with the other patch "Do Not disable network on Disconnect".
What would happen now with just this patch is that on disconnect
notification, we will disable the network. This will cause wpa_supplicant
to stop trying to connect. But at this point network_path is still valid.
Any further connect attempt with the same network would not go through
because we do this additional check if it is same SSID. And since in this
case SSID is same, AddNetwork will be dropped. Now this is a complete
deadlock until we have a different SSID to attempt to.
It was my bad that I split this patch in two but when I did it, it was
supposed to answer two different issues and later on i realized that there
is dependency. Can you revert back this patch. I am really very sorry for
I can create another patch with other changes and you can review it.
Let me know what do you think