Hi Johannes, thanks for taking time to review this. On Thu, Mar 03, 2016 at 04:46:23PM +0100, Johannes Berg wrote: > On Wed, 2016-03-02 at 19:38 +0530, Mohammed Shafi Shajakhan wrote: > > From: Mohammed Shafi Shajakhan <mohammed@xxxxxxxxxxxxxxxx> > > > > Add support for new netlink attribute 'NL80211_STA_INFO_RX_DURATION' > > This flag will be set when drivers can fill rx_duration ( approximate > > total air time(usecs) for unicast data/management frames from the > > connected client) via 'drv_sta_statistics' callback > > So, what's that useful for? [shafi] This is part of the Per Station statistics requirement, the information from 'iw dev wlan#N station dump' which has rx_duration field will be used by application assesing the statistics /performance of various clients connected to our AP. We will plot a graph based on this information (assesing the rx time spent by AP for various clients). > > Are you really sure you mean "unicast"? What if the station is the AP? > Why wouldn't multicast frames it transmitted be accounted for? [shafi] This is based on the implementation by the driver/firmware. We are focused on the AP side so unicast frames, if you are ok with this change, i can change the documentation to be specific (generic for all the drivers) something like implementation specific (please advise) > > > Also make sta_info flags 'filled' as 64 bit to accomodate for new > > typo: accommodate [shafi] will fix this regards, shafi -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
