On Tue, 2020-08-04 at 18:18 -0700, Aloka Dixit wrote:
> FILS discovery attribute, NL80211_ATTR_FILS_DISCOVERY, is nested which
> supports following parameters as given in IEEE Std 802.11ai-2016,
> Annex C.3 MIB detail:
> (1) NL80211_FILS_DISCOVERY_INT_MIN - Minimum packet interval
> (2) NL80211_FILS_DISCOVERY_INT_MAX - Maximum packet interval
> (3) NL0211_FILS_DISCOVERY_TMPL - Template data
Please keep _ATTR_ in these names. I've sent you some cleaned up version
that you can start working from. I was going to do all the cleanups, but
then realized that there are some questions I cannot answer.
> + * @support_fils_discovery: can HW support FILS discovery transmission
> + *
> * @pmsr_capa: peer measurement capabilities
> *
> * @tid_config_support: describes the per-TID config support that the
> @@ -4880,6 +4901,8 @@ struct wiphy {
> u8 support_mbssid:1,
> support_only_he_mbssid:1;
>
> + u8 support_fils_discovery:1;
Notably, you never use this.
Actually, you do later, in the mac80211 patch, but that makes no sense.
Somehow it should be used in cfg80211 if the information lives there.
I suspect this might actually want to be an extended feature flag to
userspace also knows, and also that cfg80211 should reject the operation
if not supported?
The rest of my minor comments are about policy use, using
NLA_POLICY_RANGE() with NLA_BINARY now, etc. which I've all cleaned up
in the versions I just sent you.
johannes
