All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ben Greear <greearb@candelatech.com>
To: Michal Kazior <michal.kazior@tieto.com>,
	Sergey Naumov <sknaumov@gmail.com>
Cc: "ath10k@lists.infradead.org" <ath10k@lists.infradead.org>
Subject: Re: ath10k fw_stats rx_frame/rx_clear counters
Date: Fri, 12 Jun 2015 06:43:50 -0700	[thread overview]
Message-ID: <557AE216.5020105@candelatech.com> (raw)
In-Reply-To: <CA+BoTQkjfiO-hZe+ngvRe0NCK=GroTMCeXuNNn9eb+BnRq+MJg@mail.gmail.com>



On 06/12/2015 12:16 AM, Michal Kazior wrote:
> On 11 June 2015 at 23:49, Sergey Naumov <sknaumov@gmail.com> wrote:
>> Hi all.
>>
>> I'm using barrier breaker OpenWRT on TP-link Archer C7 v2.0 router and
>> looking at /sys/kernel/debug/ieee80211/phy0/ath10k/fw_stats content.
>> As far as I understand "RX frame" accounts just rx of our and all the
>> other APs on the same channel, while "RX clear" also accounts tx of
>> our AP and non-wifi interference.
>> At least it is true for ath9k with 2.4GHz chip on the same router,
>> where "channel busy time" from survey report is always greater than
>> "channel receive time".
>>
>> But for ath10k I see that with absense of the traffic to/from our AP,
>> "RX frame" register value is always increased a little bit more than a
>> value of "RX clear" register, and it is strange.
>>
>> Do you know what could be a reason?
>
> What is the interval you're polling the values at? There's a buggy 24
> second wraparound on these cycle count related stats. Maybe you're
> hitting that..

Here is a link to my previous email on the wrapping issue..it is not *just*
a matter of polling more often:

https://www.marc.info/?l=linux-wireless&m=143353920909147&w=2

After dealing with this, I see expected results when using 10.1.467 based
firmware (both CT and stock).

I am using WLE900VX NIC, my 4.0.4+ kernel.

Thanks,
Ben

>
> Another idea/guess is that firmware doesn't read these values
> atomically (I recall ath9k locks CC values via control register before
> reading them) and it ends up with inconsistent results.
>
>
>> P.S. I tried new Chaos Calmer RC1 OpenWRT release with updated ath10k
>> driver and binary firmware and there these low-level stats are
>> unavaiable (file /sys/kernel/debug/ieee80211/phy0/ath10k/fw_stats
>> exists but read returns error).
>
> The firmware stats interface has a very clunky and unstable ABI. It's
> broken often by firmware updates and remains so until someone
> notices..
>
> Can you provide more details, please:
>   - which revision of CC you're using,
>   - what ath10k firmware version is used,
>   - what compat package version ath10k is from.
>
>
> Michał
>
> _______________________________________________
> ath10k mailing list
> ath10k@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/ath10k
>

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

  reply	other threads:[~2015-06-12 13:44 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-11 21:49 ath10k fw_stats rx_frame/rx_clear counters Sergey Naumov
2015-06-12  7:16 ` Michal Kazior
2015-06-12 13:43   ` Ben Greear [this message]
2015-06-12 17:51     ` Sergey Naumov
2015-06-12 18:15       ` Adrian Chadd
2015-06-12 20:34       ` Ben Greear
2016-09-08 12:00         ` Sergey Naumov
2016-09-08 18:15           ` Ben Greear

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=557AE216.5020105@candelatech.com \
    --to=greearb@candelatech.com \
    --cc=ath10k@lists.infradead.org \
    --cc=michal.kazior@tieto.com \
    --cc=sknaumov@gmail.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 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.