Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <0688b1a52258ad1bee88facf4cbe72f46430108c.camel@gmail.com>
Subject: Re: [REGRESSION] wifi: brcmfmac: add firmware vendor info in driver
 info
From:   Nimrod MacIomhair <nimrod4garoa@gmail.com>
To:     Kalle Valo <kvalo@kernel.org>, Hans de Goede <hdegoede@redhat.com>
Cc:     Arend van Spriel <aspriel@gmail.com>,
        Franky Lin <franky.lin@broadcom.com>,
        Hante Meuleman <hante.meuleman@broadcom.com>,
        regressions@lists.linux.dev,
        Arend van Spriel <arend.vanspriel@broadcom.com>,
        linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org,
        brcm80211-dev-list.pdl@broadcom.com,
        SHA-cyfmac-dev-list@infineon.com
Date:   Sat, 13 May 2023 19:35:49 +0200
In-Reply-To: <3c195b2430668788f993df7872b21e52f4450ef0.camel@gmail.com>
References: <4ef3f252ff530cbfa336f5a0d80710020fc5cb1e.camel@gmail.com>
         <c7aa9034-59f8-11aa-7170-86c0db012dda@redhat.com>
         <87bkiso0aw.fsf@kernel.org>
         <3c195b2430668788f993df7872b21e52f4450ef0.camel@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.48.1 
MIME-Version: 1.0
Precedence: bulk


On Wed, 2023-05-10 at 14:53 +0200, Felix wrote:
> On Wed, 2023-05-10 at 15:09 +0300, Kalle Valo wrote:
> > Hans de Goede <hdegoede@redhat.com> writes:
> >=20
> > > On 5/10/23 10:25, Felix wrote:
> > > > Changes made by the mentioned commit lead to Oops when waking up af=
ter
> > > > suspend to RAM.
> > > >=20
> > > > When waking up a Dell XPS 13 9350 with BCM4350 wifi card after susp=
end
> > > > to RAM,=C2=A0the kernel hangs with a NULL pointer dereference and O=
ops. The
> > > > issues was introduced by commit
> > > > da6d9c8ecd00e20218461007948f2b0a8e7fa242 and only happens when the
> > > > brcmfmac module is loaded. This issue is the reason we could not
> > > > upgrade to newer kernels than the 6.1 series on the affected machin=
e.
> > > >=20
> > > >=20
> > > > Details:
> > > >=20
> > > > On a Dell XPS 13 9350 laptop with Broadcom BCM4350 wifi card (accor=
ding
> > > > to lspci), every kernel starting with 6.2 hangs when waking up afte=
r
> > > > suspend to RAM. The issue persisted as of May 2nd (last tested comm=
it
> > > > was 348551ddaf311c76b01cdcbaf61b6fef06a49144). Passing the debug an=
d
> > > > no_console_suspend parameters to the kernel show that a NULL pointe=
r
> > > > dereference and an Oops happen at wakeup. Please find the kernel
> > > > messages readable on the screen at this point attached
> > > > (kernel_log_after_suspend.txt).
> > > >=20
> > > > A bisection was successful and produced the first bad commit
> > > > [da6d9c8ecd00e20218461007948f2b0a8e7fa242]=C2=A0wifi: brcmfmac: add=
 firmware
> > > > vendor info in driver info
> > > > I've attached the bisection log as well (bisect.txt).
> > > >=20
> > > > Suspend to RAM works on this machine on newer kernels once the brcm=
fmac
> > > > module is unloaded.
> > > >=20
> > > > Distribution: Arch Linux
> > > > Architecture: x86_64
> > > > Device: Dell XPS 13 9350 with BCM4350 wifi card
> > > > Kernel config used at compilation (created using the Arch Linux def=
ault
> > > > config and make olddefconfig):
> > > > https://gist.githubusercontent.com/maclomhair/e33fa7eece8f8c77e5a88=
c4eede2f523/raw/429dfb022498c026225865dbb3ab2f75d5030a54/config
> > > > Dmesg output after boot:
> > > > https://gist.githubusercontent.com/maclomhair/47c75fa759d93800e7fb5=
e937fabef3e/raw/10750996a1ea9dcb3fa14d9b94e11fedd3abf280/dmesg
> > > >=20
> > > > #regzbot introduced da6d9c8ecd00e20218461007948f2b0a8e7fa242
> > >=20
> > > Thank you for reporting this and thank you for bisecting it too!
> > >=20
> > > As a result of your excellent bug report I've been able to write
> > > a fix for this. I'm confident enough in the fix that I've submitted
> > > it upstream right away.
> > >=20
> > > But it would still be good if you can test the fix and let us know
> > > if it fixes this issue for you. You should have received a copy
> > > of the patch when I submitted it upstream.
>=20
> I have now tested the patch you sent and it indeed fixes the issue for
> me! Thank you for your quick response!

I have now also tested the second version of the patch, and it also
fixes the issue for me!

>=20
>=20
> >=20
> > For the archives here's the patch:
> >=20
> > https://patchwork.kernel.org/project/linux-wireless/patch/2023051010005=
0.27099-1-hdegoede@redhat.com/
> >=20
>=20