[PATCH] hostapd: take configured antenna set into consideration

Jouni Malinen j at w1.fi
Sun Dec 20 12:12:35 PST 2015


On Wed, Dec 09, 2015 at 12:20:33PM -0800, greearb at candelatech.com wrote:
> If user has configured TX antennas to be less than what
> hardware advertises, the MCS reported by hardware will
> be too large.  So, remove MCS sets accordingly.

The nl80211 related changes seem mostly reasonable, but I'm not sure I
understood what exactly the HT/VHT changes were supposed to do. Why are
they different? HT uses tx_ant & rx_ant and masks values from
cap->supported_mcs_set[] until the highest 1 bit is found. VHT masks
values separately for TX and RX and does this with |= 0x3 << i. Is that
really correct? Should i be "i * 2"? Why is there difference in the
configured bitmap checks (BIT(i) vs. BIT(i - 1) and different i > 0 vs.
i >= 0 condition)?

Is it correct to find the MSB 1 from the values instead of count the
number of 1 bits (configured antennas)?

I'm attaching the cleaned up patches from my work branch. Please provide
any possible updated on top of these.
 
-- 
Jouni Malinen                                            PGP id EFC895FA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-nl80211-Fetch-antenna-configuration.patch
Type: text/x-diff
Size: 3811 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/hostap/attachments/20151220/095f8ca0/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-hostapd-Take-configured-antenna-set-into-considerati.patch
Type: text/x-diff
Size: 3431 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/hostap/attachments/20151220/095f8ca0/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-tests-HT-VHT-and-antenna-configuration.patch
Type: text/x-diff
Size: 3587 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/hostap/attachments/20151220/095f8ca0/attachment-0002.bin>


More information about the Hostap mailing list