On Tue, 2019-01-08 at 22:52 +0530, vamsi krishna wrote: > Add support to transmit the frames at a rate specified by userspace > when needed in NL80211_CMD_FRAME command. This commit adds support to > specify tx rate, number of retries and tx power. > > Tx rate at which packet shall be transmitted can be specified in > %NL80211_ATTR_RATE_INFO, number of retries can be specified in > %NL80211_ATTR_WIPHY_RETRY_SHORT and %NL80211_ATTR_WIPHY_RETRY_LONG, > tx power can be specified in %NL80211_ATTR_WIPHY_TX_POWER_LEVEL. > > The drivers shall indicate the support to send frames at rate specified > by userspace by setting %NL80211_EXT_FEATURE_CMD_FRAME_TXRATE flag in > wiphy capabilities. The userspace can specify the rate within > %NL80211_ATTR_RATE_INFO attribute while sending %NL80211_CMD_FRAME. > NL80211_ATTR_RATE_INFO is a nested attribute and encapsulates the > attributes defined in &enum nl80211_rate_info. I don't usually strongly enforce that we'd like to see upstream users for upstream cfg80211 functionality, but in this case I feel that we don't even have the *ability* to implement this in most/any upstream driver(s), so there's no point. Perhaps also a mac80211 implementation would make sense to some extent. I also feel that to some extent this overlaps with the per-station TX power control? I'd like you to come back with this when there's more upstream support in general, and when you've fleshed out the exact use cases and whether or not this fine-grained control is actually needed/desired, or if something like what Ben had suggested earlier would make sense (IIRC he suggested limiting to a certain preamble type and similar - i.e. classes of rates rather than a single very precisely defined rate). Also, some nitpicks since I'm looking anyway: * don't support RATE_INFO_BITRATE - RATE_INFO_BITRATE32 is for new applications and by definition there can be no old application for a new feature * NL80211_ATTR_WIPHY_TX_POWER_LEVEL has a small indentation problem in the docs * would make sense to capitalize "TX" throughout, rather than writing "tx power" johannes
