All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sebastian Gottschall <s.gottschall@dd-wrt.com>
To: Kalle Valo <kvalo@codeaurora.org>
Cc: ath10k@lists.infradead.org
Subject: Re: 9984 with 10.4.3.5.3-0057 crashes
Date: Thu, 26 Apr 2018 08:23:01 +0200	[thread overview]
Message-ID: <e1ed94a4-e9db-3cc3-81db-e21290bfcd97@dd-wrt.com> (raw)
In-Reply-To: <87a7tr8g3z.fsf@kamboji.qca.qualcomm.com>

Am 25.04.2018 um 17:23 schrieb Kalle Valo:
> Sebastian Gottschall <s.gottschall@dd-wrt.com> writes:
>
>> Am 24.04.2018 um 07:27 schrieb Kalle Valo:
>>> Sebastian Gottschall <s.gottschall@dd-wrt.com> writes:
>>>
>>>> just for your notice. 10.4.3.5.3-0057 on 9984 which was just released
>>>> crashes in vht160 operation mode immediatly after first station
>>>> associates
>>>> last known working stable fw so far is 10.4-3.4-00104. the whole
>>>> 10.4.3.5.3 series seem to be seriously broken or the api has been
>>>> changed in a way
>>>> which is unsupported by ath10k (which i think is the cause of the
>>>> problem)
>>>> it would be good to know what has been changed.
>>> At least I was not informed any changes in the firmware interface. What
>>> version of ath10k are you using? Do you have any custom patches on
>>> ath10k?
>> i'm using latest git version for testing here but there is one
>> relevant patch required for more recent firmwares like 3.4 series (see
>> below) i posted already a long time ago, that vht160 will not work
>> anymore without that patch in newer firmwares since qca changed the api
>> for channel mapping in vht160.
> I don't remember what happened to that patch, but please resubmit it
> with a proper commit log and S-o-b line.
will do asap.
>
> I was told that the firmware crash is an assert triggered by host not
> populating the parameter 'peer_bw_rxnss_override' in
> WMI_PEER_ASSOC_CMDID which is needed for 160/80+80 operation.
peer_bw_rxnss_override is calculated by taking peers rx_max_rate
if its 1560 peer_bw_rxnss_override will be 2
if its 780 peer_bw_rxnss_override will be 1

so for me this all looks wrong. die parameter for vht160 looks guessed 
based on the rx rate and may result in wrong values.
since the peer is also supplying the bandwidth itself, we dont need to 
guess vht160 operation.
i will try to fix this and will post a fix once its solved

Sebastian
>

-- 
Mit freundlichen Grüssen / Regards

Sebastian Gottschall / CTO

NewMedia-NET GmbH - DD-WRT
Firmensitz:  Stubenwaldallee 21a, 64625 Bensheim
Registergericht: Amtsgericht Darmstadt, HRB 25473
Geschäftsführer: Peter Steinhäuser, Christian Scheele
http://www.dd-wrt.com
email: s.gottschall@dd-wrt.com
Tel.: +496251-582650 / Fax: +496251-5826565


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

  reply	other threads:[~2018-04-26  6:23 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-17 12:06 [PATCH v2 0/4] Support for STA idle mode power save(IMPS) pillair
2018-04-17 12:06 ` pillair
2018-04-17 12:06 ` [PATCH v2 1/4] ath10k: Add hw params for shadow register support pillair
2018-04-17 12:06   ` pillair
2018-04-24  6:05   ` [v2,1/4] " Kalle Valo
2018-04-24  6:05   ` Kalle Valo
2018-04-17 12:06 ` [PATCH v2 2/4] ath10k: Add support for shadow register for WNC3990 pillair
2018-04-17 12:06   ` pillair
2018-04-17 12:07 ` [PATCH v2 3/4] ath10k: Enable SRRI/DRRI support on ddr for WCN3990 pillair
2018-04-17 12:07   ` pillair
2018-04-17 12:07 ` [PATCH v2 4/4] ath10k: Enable sta idle power save pillair
2018-04-17 12:07   ` pillair
2018-04-18  7:06   ` Sebastian Gottschall
2018-04-18  7:06     ` Sebastian Gottschall
2018-04-18 13:07     ` govinds
2018-04-18 13:07       ` govinds
2018-04-18 13:16       ` Kalle Valo
2018-04-18 13:16         ` Kalle Valo
2018-04-18 14:44         ` Arend van Spriel
2018-04-18 14:44           ` Arend van Spriel
2018-04-19  4:32         ` govinds
2018-04-19  4:32           ` govinds
2018-04-19 16:36           ` Kalle Valo
2018-04-19 16:36             ` Kalle Valo
2018-04-19 17:00             ` Adrian Chadd
2018-04-19 17:00               ` Adrian Chadd
2018-04-20  7:21             ` Sebastian Gottschall
2018-04-20  7:21               ` Sebastian Gottschall
2018-04-20  8:16               ` Arend van Spriel
2018-04-20  8:16                 ` Arend van Spriel
2018-04-20 12:43                 ` govinds
2018-04-20 12:43                   ` govinds
2018-04-23 15:50                 ` Kalle Valo
2018-04-23 15:50                   ` Kalle Valo
2018-04-23 16:17                   ` 9984 with 10.4.3.5.3-0057 crashes Sebastian Gottschall
2018-04-24  5:27                     ` Kalle Valo
2018-04-24  6:08                       ` Sebastian Gottschall
2018-04-25 15:23                         ` Kalle Valo
2018-04-26  6:23                           ` Sebastian Gottschall [this message]
2018-04-26  7:36                           ` Sebastian Gottschall
2018-04-23 16:41                   ` [PATCH v2 4/4] ath10k: Enable sta idle power save Adrian Chadd
2018-04-23 16:41                     ` Adrian Chadd
2018-04-24  5:35                     ` Kalle Valo
2018-04-24  5:35                       ` Kalle Valo

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=e1ed94a4-e9db-3cc3-81db-e21290bfcd97@dd-wrt.com \
    --to=s.gottschall@dd-wrt.com \
    --cc=ath10k@lists.infradead.org \
    --cc=kvalo@codeaurora.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.