[PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-09 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan Add support for new netlink attribute 'NL80211_STA_INFO_RX_DURATION' This flag will be set when drivers can fill rx_duration (aggregate PPDU duration(usecs) for all the frames from a peer) via 'drv_sta_statistics' callback Also make

Re: [PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-09 Thread Mohammed Shafi Shajakhan
On Wed, Mar 09, 2016 at 12:18:25PM +0100, Johannes Berg wrote: > On Wed, 2016-03-09 at 16:22 +0530, Mohammed Shafi Shajakhan wrote: > >  > > [shafi] ath10k implementation is based on PPDU duration calculation > > and it does not includes any IFS duration. > > > > So 'Energy' (time spent by the

Re: [PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-09 Thread Johannes Berg
On Wed, 2016-03-09 at 16:22 +0530, Mohammed Shafi Shajakhan wrote: >  > [shafi] ath10k implementation is based on PPDU duration calculation > and it does not includes any IFS duration. > > So 'Energy' (time spent by the radio on receiving frames from a > particular peer) makes more sense right ?

Re: [PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-09 Thread Mohammed Shafi Shajakhan
Hi Johannes, On Wed, Mar 09, 2016 at 10:58:08AM +0100, Johannes Berg wrote: > On Wed, 2016-03-09 at 15:08 +0530, Mohammed Shafi Shajakhan wrote: > > > [shafi] Please suggest if this is ok ? I will be thankful > > if you can suggest a better one if this is not ok > > > > * @rx_duration:

Re: [PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-09 Thread Johannes Berg
On Wed, 2016-03-09 at 15:08 +0530, Mohammed Shafi Shajakhan wrote: > [shafi] Please suggest if this is ok ? I will be thankful > if you can suggest a better one if this is not ok > > * @rx_duration: approximate total air time(usecs) for all the >  frames from a peer > That seems fine. We

Re: [PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-09 Thread Mohammed Shafi Shajakhan
On Wed, Mar 09, 2016 at 10:30:03AM +0100, Johannes Berg wrote: > On Wed, 2016-03-09 at 12:50 +0530, Mohammed Shafi Shajakhan wrote: > >  > > + * @rx_duration: approximate total air time(usecs) for all the > > frames from a > > + * connected client > > I don't see why this should be restricted to

Re: [PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-09 Thread Johannes Berg
On Wed, 2016-03-09 at 12:50 +0530, Mohammed Shafi Shajakhan wrote: >  > + * @rx_duration: approximate total air time(usecs) for all the > frames from a > + * connected client I don't see why this should be restricted to "connected clients". The same value could be used for a mesh peer. Or TDLS

[PATCH v2] cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION

2016-03-08 Thread Mohammed Shafi Shajakhan
From: Mohammed Shafi Shajakhan 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 all the frames from a connected client) via 'drv_sta_statistics' callback