From: Pavel Roskin <proski@gnu.org>
To: jt@hpl.hp.com
Cc: NetDev <netdev@vger.kernel.org>
Subject: Re: dBm cutoff at -1dBm is too low
Date: Tue, 09 May 2006 00:54:55 -0400 [thread overview]
Message-ID: <1147150495.4453.65.camel@dv> (raw)
In-Reply-To: <20060508171711.GA10948@bougret.hpl.hp.com>
On Mon, 2006-05-08 at 10:17 -0700, Jean Tourrilhes wrote:
> > But shouldn't you trust the drivers using IW_QUAL_DBM, whether the value
> > is positive or negative?
>
> You can't remove the test, making the rest pointeless. Old
> style driver never used the flags, new style driver that don't report
> dBm will never use the flags, and there is not way to dinstinguish
> both, apart from the 'sign' of the value.
I used "new style driver" as synonymous to the driver using IW_QUAL_DBM
and "old style driver" as the one that doesn't use IW_QUAL_DBM.
I don't think any drivers need to specify both dBm and non-dBm data for
the same device.
Drivers that give non-dBm data are already well served by wireless
tools. They don't need to change.
Drivers that give dBm data had to limit the data to avoid its
misinterpretation as non-dBm data. Now IW_QUAL_DBM is supposed to free
drivers from such checks. But it doesn't deliver its promise, because
the data can still be misinterpreted, just is a different way. Namely,
a strong signal (0dBm) can be interpreted as an incredibly weak signal
(-256dBm). That's what I want to see fixed.
One tricky case would be when the driver sets the max signal e.g. to 30
and reports 35 (i.e. a positive value within the "reasonable" dBm
range). I would probably prefer to show it as 30/30 rather than 35dBm,
but the driver is nuts anyway, and I'm not really concerned about it to
write an extra line of code.
--
Regards,
Pavel Roskin
next prev parent reply other threads:[~2006-05-09 4:55 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-05-04 16:37 dBm cutoff at -1dBm is too low Pavel Roskin
2006-05-05 17:28 ` Jean Tourrilhes
2006-05-07 3:32 ` Pavel Roskin
2006-05-08 17:17 ` Jean Tourrilhes
2006-05-09 4:54 ` Pavel Roskin [this message]
2006-05-09 18:54 ` Jean Tourrilhes
2006-05-08 3:35 ` Pavel Roskin
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=1147150495.4453.65.camel@dv \
--to=proski@gnu.org \
--cc=jt@hpl.hp.com \
--cc=netdev@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.