linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bruno Randolf <br1@einfach.org>
To: Helmut Schaa <helmut.schaa@googlemail.com>
Cc: linville@tuxdriver.com, nbd@openwrt.org,
	ath5k-devel@lists.ath5k.org, linux-wireless@vger.kernel.org,
	adrian@freebsd.org, vasanth@atheros.com,
	"Luis R. Rodriguez" <lrodriguez@atheros.com>,
	Johannes Berg <johannes@sipsolutions.net>,
	Jonathan Guerin <jonathan@guerin.id.au>
Subject: Re: [PATCH 4/5] nl80211/mac80211: Add channel utilization to survey
Date: Thu, 7 Oct 2010 16:06:36 +0900	[thread overview]
Message-ID: <201010071606.37134.br1@einfach.org> (raw)
In-Reply-To: <AANLkTimf=f1DhP68=aHZ7WKVCN9RPBq7sh+46ExnYRVa@mail.gmail.com>

On Thu October 7 2010 15:51:49 Helmut Schaa wrote:
> On Thu, Oct 7, 2010 at 3:03 AM, Bruno Randolf <br1@einfach.org> wrote:
> > On Wed October 6 2010 18:54:37 Helmut Schaa wrote:
> >> On Tue, Oct 5, 2010 at 11:55 AM, Bruno Randolf <br1@einfach.org> wrote:
> >> > This adds three new values to the survey results:
> >> > 
> >> >  * BUSY - percentage of time the channel was busy
> >> >  * BUSY_TX - percentage of time spent transmitting frames
> >> >  * BUSY_RX - percentage of time spent receiving frames
> >> 
> >> Does BUSY include BUSY_RX and BUSY_TX already? At least on
> >> rt2800 devices we only have access to BUSY which already includes
> >> BUSY_RX and TX.
> > 
> > yes, BUSY includes TX and RX.
> > 
> >> And what about BUSY on the secondary channel in case of 40Mhz?
> > 
> > i have no information about that. luis?
> 
> At least we've got a register for that in rt2800 ;)
> That's why I'm asking.

which units do you have for the busy time? what about other drivers?

in ath[59]k we have it as cycles (which is about 1/44000 of a second, 
depending on the mode). so it counts up rather fast.

i have been thinking more about it and i think my current approach is flawed. 
the idea was that we probably need to use a percentage as the least common 
denominator between all drivers, and i wanted to leave the time period up to 
userspace. so if userspace wants to have the percentage in the last second, it 
would poll every second, if it wants it every 100ms it would poll every 100ms.
that would work, if there is only one userspace process querying this in 
regular intervals.

but what if there is a second process who does the same, or a user issues "iw 
wlan0 survey dump", that would reduce the timespan for the other process... so 
that is obviously flawed...

some absolute cumulative value would be better, so several userspace processes 
can poll and do their own averaging or percentage calculations. 

but which units to use? what would be a good way of solving that?
any ideas???

bruno

  reply	other threads:[~2010-10-07  7:06 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-05  9:55 [PATCH 1/5] ath: Add common function for reading cycle counters Bruno Randolf
2010-10-05  9:55 ` [PATCH 2/5] ath5k: Use common cycle counters for ANI Bruno Randolf
2010-10-05  9:55 ` [PATCH 3/5] ath9k: Use common cycle counters Bruno Randolf
2010-10-05 20:14   ` [ath5k-devel] " Luis R. Rodriguez
2010-10-06  1:10     ` Bruno Randolf
2010-10-06  1:13       ` Luis R. Rodriguez
2010-10-06  1:18   ` Luis R. Rodriguez
2010-10-06  2:00     ` Bruno Randolf
2010-10-06  2:15       ` [ath5k-devel] " Luis R. Rodriguez
2010-10-05  9:55 ` [PATCH 4/5] nl80211/mac80211: Add channel utilization to survey Bruno Randolf
2010-10-05 22:36   ` Luis R. Rodriguez
2010-10-06  2:35     ` Bruno Randolf
2010-10-06  9:54   ` Helmut Schaa
2010-10-07  1:03     ` Bruno Randolf
2010-10-07  3:02       ` [ath5k-devel] " Luis R. Rodriguez
2010-10-07  6:51       ` Helmut Schaa
2010-10-07  7:06         ` Bruno Randolf [this message]
2010-10-07  7:35           ` Jonathan Guerin
2010-10-07  7:44           ` Helmut Schaa
2010-10-07  7:52             ` Johannes Berg
2010-10-08 17:41               ` Helmut Schaa
2010-10-05  9:55 ` [PATCH 5/5] ath5k: Add busy ratios to survey data Bruno Randolf
2010-10-05 22:38   ` Luis R. Rodriguez
2010-10-06  2:25     ` Bruno Randolf
2010-10-06  2:45       ` Felix Fietkau
2010-10-06  2:50         ` Bruno Randolf
2010-10-06 14:13           ` Luis R. Rodriguez

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201010071606.37134.br1@einfach.org \
    --to=br1@einfach.org \
    --cc=adrian@freebsd.org \
    --cc=ath5k-devel@lists.ath5k.org \
    --cc=helmut.schaa@googlemail.com \
    --cc=johannes@sipsolutions.net \
    --cc=jonathan@guerin.id.au \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=lrodriguez@atheros.com \
    --cc=nbd@openwrt.org \
    --cc=vasanth@atheros.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).