[PATCH] Reschedule scan from wpas_stop_pno if it was postponed

Dmitry Shmidt dimitrysh
Tue Mar 31 08:23:16 PDT 2015


On Tue, Mar 31, 2015 at 1:54 AM, Arik Nemtsov <arik at wizery.com> wrote:
> Patch looks good in general, but don't you also want to remove the
> below from wpas_stop_pno:
>
> if (wpa_s->wpa_state == WPA_SCANNING)
>     wpa_supplicant_req_scan(wpa_s, 1, 0);
>

In general - you are right, but if some kernel will not send SCHED_SCAN_STOPPED,
it will be additinal "insurance"...

> Arik
>
> On Mon, Mar 30, 2015 at 11:55 PM, Dmitry Shmidt <dimitrysh at google.com> wrote:
>> How it was:
>> 03-30 20:40:34.365 I/wpa_supplicant( 1239): wpas_stop_pno: Stop PNO
>> 03-30 20:40:34.366 I/wpa_supplicant( 1239): Already scanning -
>> Reschedule the incoming scan req
>> 03-30 20:40:34.366 I/wpa_supplicant( 1239): wpa_supplicant_event: Stopped event
>> 03-30 20:40:35.368 I/wpa_supplicant( 1239): wpa_supplicant_scan: Trigger scan
>>
>> With the patch:
>> 03-30 20:42:32.977 I/wpa_supplicant( 3472): wpas_stop_pno: Stop PNO
>> 03-30 20:42:32.977 I/wpa_supplicant( 3472): Already scanning -
>> Reschedule the incoming scan req
>> 03-30 20:42:32.978 I/wpa_supplicant( 3472): wpa_supplicant_event: Stopped event
>> 03-30 20:42:32.978 I/wpa_supplicant( 3472): wpa_supplicant_scan: Trigger scan
>>
>> _______________________________________________
>> HostAP mailing list
>> HostAP at lists.shmoo.com
>> http://lists.shmoo.com/mailman/listinfo/hostap
>>



More information about the Hostap mailing list