linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bruno Randolf <br1@einfach.org>
To: "Jaroslav Fojtik" <jafojtik@seznam.cz>
Cc: linux-wireless@vger.kernel.org
Subject: Re: ATH5k - Signal measurement reports wrong values - thanks for a fix
Date: Mon, 28 Jun 2010 10:21:47 +0900	[thread overview]
Message-ID: <201006281021.47132.br1@einfach.org> (raw)
In-Reply-To: <4C26FD91.10140.F4D6B@jafojtik.seznam.cz>

On Sun June 27 2010 16:28:17 Jaroslav Fojtik wrote:
> Dear Bruno,
> 
> > > I hope that this patch would not influence performance or any other
> > > functionality.
> > 
> > no it just fixes a regression i have introduced in an earlier change.
> > thanks for reporting and testing this! i will send the patch for
> > inclusion on monday, then it might take some time (few days to weeks)
> > until it apperars in compat- wireless.
> 
> Thank you for explanation.
> The patch surely improves signal strength measurement and I recomend it.
>
> Anyway, when you look at the magenta line here:
>   http://78.108.103.11:11080/cgi-bin/rodga_1day_big.cgi
> 
> There are big drops in measured signal. Did you fix all places?
> 
> Before this patch there were short peeks from -78dB to -71dB.
> After patch there are short measurement drops from -62dB to -70dB.
> 
> The week charting simply averages it:
>   http://78.108.103.11:11080/cgi-bin/rodga_1week_big.cgi

i guess this must be the same situation as before, no?

how many antennas do you have attached to your device? it might be interesting 
to check if you get more consistent signal measurements by disabling antenna 
diversity, setting it to a fixed antenna. you can do this by using a debugfs 
file "echo fixed-a > /sys/kernel/debug/ath5k/phy0/antenna" (or "fixed-b")...

btw: when the signal measurement was low, did you see any negative impact on 
the thruput as well?

bruno

  reply	other threads:[~2010-06-28  1:23 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-20 19:20 ATH5k - Signal measurement reports wrong values Jaroslav Fojtik
2010-06-21  1:26 ` Bruno Randolf
2010-06-21  7:23   ` Jaroslav Fojtik
2010-06-21  7:50     ` Bruno Randolf
2010-06-21 19:28       ` Jaroslav Fojtik
2010-06-22  4:01         ` Bruno Randolf
2010-06-23  9:03       ` ATH5k - Signal measurement reports wrong values - "calib" did not help Jaroslav Fojtik
2010-06-25  0:47         ` Bruno Randolf
2010-06-26 12:43           ` Jaroslav Fojtik
2010-06-26 16:07             ` Bruno Randolf
2010-06-26 17:54               ` ATH5k - Signal measurement reports wrong values - thanks for a fix Jaroslav Fojtik
2010-06-27  2:20                 ` Bruno Randolf
2010-06-27  7:28                   ` Jaroslav Fojtik
2010-06-28  1:21                     ` Bruno Randolf [this message]
2010-06-30  7:15                       ` Jaroslav Fojtik
2010-06-30  7:26                         ` Bruno Randolf
2010-07-01 20:11                           ` ATH5k - Signal measurement drops Jaroslav Fojtik

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=201006281021.47132.br1@einfach.org \
    --to=br1@einfach.org \
    --cc=jafojtik@seznam.cz \
    --cc=linux-wireless@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 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).