All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sebastian Gottschall <s.gottschall@dd-wrt.com>
To: Michal Kazior <michal.kazior@tieto.com>,
	"ath10k@lists.infradead.org" <ath10k@lists.infradead.org>,
	Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Subject: Re: AP mode firmware crash on QCA9880-BR4A
Date: Tue, 14 Jul 2015 16:29:42 +0200	[thread overview]
Message-ID: <55A51CD6.6090808@dd-wrt.com> (raw)
In-Reply-To: <CA+BoTQkOeLarxTcOR=aeFoixP-41Jri72gM5MRqUQdNhbQoXOg@mail.gmail.com>

xo just revert that commit and test again. okay. i will do that that evening

Am 14.07.2015 um 12:50 schrieb Michal Kazior:
> + ath10k list back, martin
>
> Make sure to reply to everyone next time and not just me, please ;-)
>
> On 14 July 2015 at 12:18, Sebastian Gottschall <s.gottschall@dd-wrt.com> wrote:
>> Am 14.07.2015 um 07:45 schrieb Michal Kazior:
>>> On 14 July 2015 at 00:57, Sebastian Gottschall <s.gottschall@dd-wrt.com>
>>> wrote:
>>>> you can reproduce this issue with a standard compex wle 600v card. its a
>>>> on
>>>> board eeprom card. so calibration issues will not occur.
>>>> but this card will also crash with recent firmwares on any platform. but
>>>> (and now the big thing) it will only happen in ap mode.
>>>> if you initialize it in station mode, it will work
>>> That's interesting. Did the card ever work as AP with ath10k for you?
>>> By "any platform" you mean you tested on both little- and big- endian
>>> machines?
>> yes. with older firmwares :-)
> I assume it's the same as Martin.
>
> Do you guys know if 10.2-00082[1] crashes as well?
>
> [1]: https://github.com/kvalo/ath10k-firmware/blob/master/10.2/firmware-3.bin_10.2-00082-4-2
>
>
>> tested on AR71XX (MIPS BE) , Gateworks Ventana ( ARM Little endian), and
>> Gateworks Laguna ( ARM Little Endian)
>>
>> all 3 devices crash the firmware. , e
> Thanks for the details.
>
>
>>> My current guess is this could have something to do with AP beaconing
>>> - if it was to get an invalid dma pointer it could crash the device so
>>> hard as to effectively prevent host from accessing register dump
>>> memory.
>> but whats the difference? some cards work and some are not. so something in
>> the otp must be likelly responsible for crashing the firmware.
>> just a guess
> I can imagine firmware could execute some code based on board data or
> device specific bits which do extra preparations on beacon path and
> which presence or lack of could prompt a crash. This is merely a
> speculation on my part.
>
> The OTP most likely isn't the problem because - as proven by Martin -
> the crash occurs with caldata at apparently same spot as when
> board+otp. When caldata is used driver does not execute OTP program at
> all. Maybe caldata itself is misinterpreted by newer firmware?
>
> If someone is brave enough with spare time one could attempt to
> re-implement the old by-copy beaconing (which copies entire beacon
> frame into WMI command instead of passing just dma pointer; see [2]
> for reference) to see if it makes any difference. If that helps it
> would imply the crash is indeed related to dma beaconing. If it
> doesn't it would prove nothing though.
>
> [2]: https://github.com/kvalo/ath/commit/748afc4735361e21ad655c0dc021ea3aaeeb3efd
>
>
> Michał
>


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

  reply	other threads:[~2015-07-14 14:29 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-12 20:05 AP mode firmware crash on QCA9880-BR4A Martin Blumenstingl
2015-07-13  5:57 ` Michal Kazior
2015-07-13 19:32   ` Martin Blumenstingl
2015-07-13 22:57     ` Sebastian Gottschall
2015-07-14  5:45       ` Michal Kazior
     [not found]         ` <55A4E1FC.9000005@dd-wrt.com>
2015-07-14 10:50           ` Michal Kazior
2015-07-14 14:29             ` Sebastian Gottschall [this message]
2015-07-18 11:53             ` Martin Blumenstingl
2015-07-20 11:41               ` Michal Kazior
2015-07-20 12:06                 ` Martin Blumenstingl
2015-07-20 13:36                   ` Martin Blumenstingl
2015-07-22  7:47                     ` Michal Kazior
2015-07-22 14:53                       ` Martin Blumenstingl
2015-07-23  5:50                         ` Michal Kazior
2015-07-29  0:04                           ` Martin Blumenstingl
2015-07-29  7:38                             ` Michal Kazior
2015-07-29  9:18                               ` Martin Blumenstingl
2015-08-04  5:19                                 ` Michal Kazior
2015-08-08 21:06                                   ` Martin Blumenstingl
2015-08-17  5:46                                     ` Michal Kazior
2015-08-22 18:38                                       ` Martin Blumenstingl
2015-09-19 10:29                                         ` Martin Blumenstingl
2015-07-14  5:39     ` Michal Kazior
  -- strict thread matches above, loose matches on Subject: below --
2015-07-16 22:11 Mercer, Karl
2015-07-17  5:56 ` Michal Kazior
2015-07-17 10:46   ` Mercer, Karl

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=55A51CD6.6090808@dd-wrt.com \
    --to=s.gottschall@dd-wrt.com \
    --cc=ath10k@lists.infradead.org \
    --cc=martin.blumenstingl@googlemail.com \
    --cc=michal.kazior@tieto.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.