Re: [PATCH 9/9] hs20-client: Add 1 second sleep after IP is detected to allow external config to complete.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 07, 2020 at 01:10:17AM -0800, greearb@xxxxxxxxxxxxxxx wrote:
> For instance, setting up VRFs and such might take a small bit of extra time.

What is VRF and why would a user space tool on a station device need to
know about that?

> diff --git a/hs20/client/oma_dm_client.c b/hs20/client/oma_dm_client.c
> @@ -1211,6 +1211,8 @@ int cmd_oma_dm_sim_prov(struct hs20_osu_client *ctx, const char *url)
>  	if (wait_ip_addr(ctx->ifname, 15) < 0) {
>  		wpa_printf(MSG_INFO, "Could not get IP address for WLAN - try connection anyway");
>  	}
> +	/* Give a bit more time in case external tools are still configuring things, like VRF. */
> +	os_sleep(1, 0);
>  	write_summary(ctx, "OMA-DM SIM provisioning");

Wouldn't it make more sense to do whatever is needed within
wait_ip_addr() instead of copy pasting this into every location where
wait_ip_addr() is called? That said, I'm not convinced waiting one
second is the thing that should happen here.

What exactly are the cases where you have needed to wait longer after
the IP address has been assigned? Maybe wait_ip_addr() (or
wpa_supplicant before indicating STATUS command ip_address field) should
wait for those exact things rather than adding a hardcoded wait for one
second.

-- 
Jouni Malinen                                            PGP id EFC895FA

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux