Re: [PATCH] cfg80211: Be able to set bss expire time at config stage.

From: Johannes Berg
Date: Mon May 22 2017 - 12:24:30 EST



> Couldn't userspace just look at NL80211_BSS_SEEN_MS_AGO to filter and
> create its own list?ÂÂGiven that the kernel provides the information
> userspace needs to figure out the age of a particular BSS, it doesn't
> seem like there needs to be a kernel tunable for this.ÂÂUserspace can
> already avoid stale results.

Yeah, I agree. It can also ask for a flush, so that old results are
gone by the time the next scan returns. We don't have a flush operation
without requesting a new scan, but I guess that could be added.

> Also, different runtime situations might want different result ages,
> which wouldn't be possible if the kernel had a hardcoded maximum.Â
> Furthermore, different userspace apps might be reading the same scan
> list, and they might have different ideas about staleness.
>
> Or perhaps I misunderstand the problem, which could well be the case.

No, I think this is perfectly right - userspace should be able to deal
with this given the tools we gave it, or if not, we should probably
just give it more tools instead of hardcoding the kernel configuration.

This value really just kinda needed to be an upper bound so that we
don't start expiring entries while we're still scanning.

johannes