linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Valdis.Kletnieks@vt.edu
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Zhu Yi <yi.zhu@intel.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: 31-rc3-mmotm0716 - wonky wireless statistics..
Date: Mon, 03 Aug 2009 23:40:20 -0400	[thread overview]
Message-ID: <30110.1249357220@turing-police.cc.vt.edu> (raw)
In-Reply-To: Your message of "Wed, 29 Jul 2009 09:19:12 +0200." <1248851953.13742.12.camel@johannes.local>

[-- Attachment #1: Type: text/plain, Size: 1622 bytes --]

On Wed, 29 Jul 2009 09:19:12 +0200, Johannes Berg said:

> > > # cat /proc/net/wireless
> > > Inter-| sta-|   Quality        |   Discarded packets               | Missed | WE
> > >  face | tus | link level noise |  nwid  crypt   frag  retry   misc | beacon | 22
> > >  wlan0: 0000   63.  -47.  -256        0      0      0      0      0
> > >  wlan1: 0000    0     0     0        0      0      0      0      0
> > >  wlan2: 0000    0     0     0        0      0      0      0      0

> and since we have dBm for the signal strength (and dBm is "all or
> nothing") you get -256 since we don't fill noise right now.
> 
> However, it goes on like this:
> 	stats->qual.updated & IW_QUAL_NOISE_UPDATED ? '.' : ' '
> 
> so you're right -- the tool shouldn't be using the value unless followed
> by a dot (".").

OK, went back and checked a -mmotm0702 kernel, and there I see:

 cat /proc/net/wireless
Inter-| sta-|   Quality        |   Discarded packets               | Missed | WE
 face | tus | link level noise |  nwid  crypt   frag  retry   misc | beacon | 22
 wlan0: 0000   64.  -46.  -127.       0      0      0      0      0        0
 wlan1: 0000    0     0     0        0      0      0      0      0        0
 wlan2: 0000    0     0     0        0      0      0      0      0        0

so yes, it appears that the userspace program (gkrellm-wifi) is failing to
notice the lack of a trailing '.'.  Looks like I need to write a patch and push
it upstream, make it do something sensible with the S/N ratio when N is
missing. Too bad probably no easy way to compute an estimate based on the level
and link-quality values....


[-- Attachment #2: Type: application/pgp-signature, Size: 226 bytes --]

      reply	other threads:[~2009-08-04  3:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-28 21:01 31-rc3-mmotm0716 - wonky wireless statistics Valdis.Kletnieks
2009-07-29  1:31 ` Zhu Yi
2009-07-29  7:19   ` Johannes Berg
2009-08-04  3:40     ` Valdis.Kletnieks [this message]

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=30110.1249357220@turing-police.cc.vt.edu \
    --to=valdis.kletnieks@vt.edu \
    --cc=akpm@linux-foundation.org \
    --cc=johannes@sipsolutions.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=yi.zhu@intel.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).