I'll drop some of the people since this is a sub-thread of the original,
I'll keep the lists for access to this using lore.kernel.org.
On 2021-05-15 18:53, Pali Rohár wrote:
> On Saturday 15 May 2021 18:32:30 Dave Olsthoorn wrote:
>> On 2021-05-15 17:40, Pali Rohár wrote:
>> > On Saturday 15 May 2021 17:10:31 Dave Olsthoorn wrote:
>> > > The firmware still seems to crash quicker than previously, but
>> > > that's a
>> > > unrelated problem.
>> > Hello! Do you have some more details (or links) about mentioned firmware
>> > crash?
>> Sure, firmware crashes have always been a problem on the Surface
> What wifi chip you have on these devices? Because very similar firmware
> crashes I see on 88W8997 chip (also with mwifiex) when wifi card is
> configured in SDIO mode (not PCIe).
The Surface Pro 2017 has an 88W8897.
> I know that there are new version of firmwares for these 88W8xxx chips,
> but they are available only under NXP NDA and only for NXP customers.
> So it looks like that end users with NXP wifi chips are out of luck.
>> They seem to be related, at least for some of the crashes, to power
>> management. For this reason I disabled powersaving in NetworkManager
>> used to make it at least stable enough for me, in 5.13 this trick does
>> seem to work.
>> The dmesg log attached shows a firmware crash happening, the card does
>> work even after a reset or remove & rescan on the pci(e) bus.
> Similar issue, card start working again only after whole system
> So this is something which can be resolved only in NXP.
After a conversation with the author of the patches, the problem is not
the power management itself (for most hardware revisions ) but a race
where pci commands are being written while the device is being put to
sleep. A fix for this problem is included in the patches which make all
pci commands synchronous instead of asynchronous .
After that a the wakeup patch seems relevant .
>> There are patches  which have not been submitted yet and where
>> as part of the linux-surface effort . From my experience these
>> resolve most if not all of the firmware crashes.
> Is somebody going to cleanup these patches and send them for inclusion
> into mainline kernel? I see that most of them are PCIe related, but due
> to seeing same issues also on SDIO bus, I guess adding similar hooks
> also for SDIO could make also SDIO more stable...
The author plans to upstream them, he just hasn't gotten around to it.