Re: [PATCH 6/6] bss coloring: handle the collision and CAC events coming from the kernel

John Crispin <[email protected]> Wed, 17 Jun 2020 17:39:42 +0200
Newsgroups gmane.linux.drivers.hostap
Message-ID <[email protected]>
Hi,
> Few design questions:
> 1. Why should hostapd be involved in adding the cca IE ?
>     Instead __ieee80211_beacon_get() can add it in mac80211/tx.c and update hostapd with the new color after CAC.

I basically followed the flow of how CSA does it.

> 2. Currently, the initial BSS Color is taken from the hostapd.conf file.
>     Don't you think it will be better to set it dynamically upon AP startup rather than take it from the conf file ?

I actually set a random value in hostapd.conf, however one can set a 
static one if desired. having the kernel autogenerate this would not 
allow explicit values to be set.

> 3. Are you planning to add the "collision" and "color in use" event reports as well ?

these 2 exist inside the patchset, EVENT_BSS_COLOR_COLLISION and 
EVENT_CCA_NOTIFY, latter being called along the lines of how CSA does it.

     John


>
> ________________________________
> The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential and/or privileged material. Any retransmission, dissemination, copying or other use of, or taking of any action in reliance upon this information is prohibited. If you received this in error, please contact the sender and delete the material from any computer. Nothing contained herein shall be deemed as a representation, warranty or a commitment by Celeno. No warranties are expressed or implied, including, but not limited to, any implied warranties of non-infringement, merchantability and fitness for a particular purpose.
> ________________________________
>
>
> _______________________________________________
> Hostap mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/hostap

_______________________________________________
Hostap mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/hostap