Re: [PATCH 1/2] {nl,cfg}80211: support high bitrates

[Vladimir Kondratiev <vkondrat_qca@qca.qualcomm.com>]

On Tuesday, July 03, 2012 09:04:38 AM Luis R. Rodriguez wrote:
> On Tue, Jul 3, 2012 at 1:37 AM, Vladimir Kondratiev
> 
> <qca_vkondrat@qca.qualcomm.com> wrote:
> > User space tools should switch to new bitrate. Old one is deprecated and
> > going to be removed in the future.
> 
> Removed?? No way. You can't make old userspace break.
> 
> > @@ -1638,12 +1638,16 @@ struct nl80211_sta_flag_update {
> > 
> >   *
> >   * These attribute types are used with %NL80211_STA_INFO_TXRATE
> >   * when getting information about the bitrate of a station.
> > 
> > + * Legacy bitrate represented with 16-bit value, while new
> > + * "high throughput" bitrate uses 32-bit value. User space tools
> > + * should use new bitrate.
> 
> Something a bit more clearer would say that we send both, and that
> userspace should use the new one given that new technologies required
> a higher data type. 

Agree - I'll rephrase

> Also explaining what gets sent to userspace for
> those high data bit rates on the old data type would help -- from what
> I read in your patch we set those high bit rates to 0 on the old data
> type.

Not exactly. If value is 0, it is just not send:
(bitrate_compat > 0 &&
	     nla_put_u16(msg, NL80211_RATE_INFO_BITRATE, bitrate_compat)

> 
> >   *
> >   * @__NL80211_RATE_INFO_INVALID: attribute number 0 is reserved
> >   * @NL80211_RATE_INFO_BITRATE: total bitrate (u16, 100kbit/s)
> >   * @NL80211_RATE_INFO_MCS: mcs index for 802.11n (u8)
> >   * @NL80211_RATE_INFO_40_MHZ_WIDTH: 40 Mhz dualchannel bitrate
> >   * @NL80211_RATE_INFO_SHORT_GI: 400ns guard interval
> > 
> > + * @NL80211_RATE_INFO_BITRATE_HT: total bitrate (u32, 100kbit/s)
> 
> NL80211_RATE_INFO_BITRATE_HT seems misleading as likely we will also
> use this for VHT, and whatever other fun acronym the industry comes up
> with for bitrates for 802.11.

In case "HT" for "High Throuput" is misleading, I am open to any
other proposals. What name can you suggest?

Thanks, Vladimir