[PATCH] supplicant: Allow disabling reassoc based on scan results.

[Ben Greear]

On 03/23/2016 08:18 AM, Dan Williams wrote:
> On Wed, 2016-03-23 at 07:24 -0700, Ben Greear wrote:
>>
>> On 03/23/2016 06:41 AM, Dan Williams wrote:
>>>
>>> On Tue, 2016-03-22 at 19:12 -0400, greearb at candelatech.com wrote:
>>>>
>>>> From: Ben Greear <greearb at candelatech.com>
>>>>
>>>> This gives configurable control over whether to consider
>>>> roaming based on scan results.  I find this useful when
>>>> doing explicit roaming tests, where I do not want scan
>>>> requests to cause the roam automatically.
>>> Manually triggered ones, or *all* scan requests even background
>>> ones?
>> I believe my patch would be for all scan results.  In my particular
>> case, I really care only about manual scans and external scans,
>> so if that makes the patch more useful for others, I can see if there
>> is
>> a way to distinguish one scan from another.
>
> Because there's already the flag to prevent roam on manual (eg, control
> interface initiated) scan requests, I was asking to clarify if you
> explicitly wanted to allow it to be disabled for all scans.  The only
> downside I see here is that with this patch/option, the supplicant will
> never roam, and there's no official command to request roaming either
> (except the testing one, which is IIRC only supposed to be used for
> testing...).

Well, I am doing testing.

But, what flag are you talking about?  I can check the code to see if
it will meet my needs.

Either way, it may be useful (if rarely used) to be able to completely disable roaming
with a config change instead of having to re-compile.

The scan that was causing the roam for me was from "wpa_cli -i foo scan" command.

Thanks,
Ben

-- 
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc  http://www.candelatech.com