9984 with 10.4.3.5.3-0057 crashes

[Sebastian Gottschall]

Am 25.04.2018 um 17:23 schrieb Kalle Valo:
> Sebastian Gottschall <s.gottschall at dd-wrt.com> writes:
>
>> Am 24.04.2018 um 07:27 schrieb Kalle Valo:
>>> Sebastian Gottschall <s.gottschall at 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 at dd-wrt.com
Tel.: +496251-582650 / Fax: +496251-5826565