[PATCH 3/3] wpa_supplicant: cancel scan work when starting a connection

[Peer, Ilan]

> -----Original Message-----
> From: hostap-bounces at lists.shmoo.com [mailto:hostap-
> bounces at lists.shmoo.com] On Behalf Of Jouni Malinen
> Sent: Sunday, February 01, 2015 17:17
> To: hostap at lists.shmoo.com
> Subject: Re: [PATCH 3/3] wpa_supplicant: cancel scan work when starting a
> connection
> 
> On Sun, Mar 30, 2014 at 06:09:12AM +0000, Peer, Ilan wrote:
> > > On Thu, Mar 27, 2014 at 08:58:34AM +0200, Ilan Peer wrote:
> > > > Cancel a pending scan work if a connection is started, as
> > > > otherwise the connection flow would be blocked until the scan is
> complete.
> > >
> > > This looks a bit problematic since this seems to be able to drop a

> > > special scan request that was issued through the control interface. Why is
> this needed?
> > > Isn't it enough for sme_authenticate() to add a high priority
> > > sme-connect radio work that ends up getting added to the beginning
> > > of the queue, i.e., to be processed before the pending scan radio work if
> one was there?
> >
> > Agree ... missed this point.
> >
> > Do I need to do something special to abandon it?
> 
> I had left this pending to try to figure out what the proposed change was
> trying to fix (that blocking part). I still don't see what the issue here would be.
> If you think this is still needed, please provide a wpa_supplicant debug log
> showing a blocking case where this change would have helped.
> 

I do not fully remember what was the original motivation, but as far as the connection establishment is concerned, I think that this change is not required. However, it would be beneficial to cancel the scan work to ease post connection actions such dhcp and other higher layer protocol activities. 

Regard,

Ilan.