Re: [PATCH 1/3] station: handle ROAMING state in disconnect event

[Denis Kenzior <denkenz@gmail.com>]

Hi James,

On 9/27/22 18:15, James Prestwood wrote:
> This both adds proper handling to the new roaming logic and fixes
> a potential bug with firmware roams.
> 
> The new way roaming works doesn't use a connect callback. This
> means that any disconnect event or call to netdev_connect_failed
> will result in the event handler being called, where before the
> connect callback would. This means we need to handle the ROAMING
> state in the station disconnect event so IWD properly disassociates
> and station goes out of ROAMING.
> 
> With firmware roams netdev gets an event which transitions station
> into ROAMING. Then netdev issues GET_SCAN. During this time a
> disconnect event could come in which would end up in
> station_disconnect_event since there is no connect callback. This
> needs to be handled the same and let IWD transition out of the
> ROAMING state.
> ---
>   src/station.c | 1 +
>   1 file changed, 1 insertion(+)
> 

All applied, thanks.

Regards,
-Denis