From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A8B0FC7EE22 for ; Wed, 10 May 2023 10:05:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236096AbjEJKF0 (ORCPT ); Wed, 10 May 2023 06:05:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59142 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229755AbjEJKFX (ORCPT ); Wed, 10 May 2023 06:05:23 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3A992659D for ; Wed, 10 May 2023 03:04:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1683713076; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=mxi0tssM/MIAC/Us8XUva2O6e9fov44nkm6fQlyWJnA=; b=B9/ZWLkKgz8obRLJxxxZ0rp8JeLEK257gW5Ud205VBuCK5jqHgB4sTfLB/TDfhchJ3rJ4H XvNeWJLfHA/0ksUFerQYo/fE4viEm40FEOECzBhjsj+ExAPAGW1cSg2Fbv8VD63Y+7r07J q+q04s0oJAgs7sUlctfaaycQaDc+mFI= Received: from mail-ej1-f69.google.com (mail-ej1-f69.google.com [209.85.218.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-231-eBu-xCx_Mn2ftR4odSHhQQ-1; Wed, 10 May 2023 06:04:35 -0400 X-MC-Unique: eBu-xCx_Mn2ftR4odSHhQQ-1 Received: by mail-ej1-f69.google.com with SMTP id a640c23a62f3a-965f0c4ae32so548027466b.1 for ; Wed, 10 May 2023 03:04:35 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683713074; x=1686305074; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=mxi0tssM/MIAC/Us8XUva2O6e9fov44nkm6fQlyWJnA=; b=X8KtbxWiOW8h/BBfUinPEgBroVBjyeI58ZdD3TZZUWYffFIuQb3TD92QHMRIfhJysT HKh4cE64JKB/stU/HYv5PmK7Ez8ngSNioBrPFPBzditcDGNjQq/725nKMS58CcqLOYvG uWzAaUTphSF8AijvWNh2GNH0dbNAfAr60Tjeo65KKbF3hG+A8vBOXxkWPvuE9LVyxSwF yeSLO3oXASHVkPblMerfpqguNE9kEl+9WEScjuP2s+1Axbp49YdharUD+jXRyQSKAMIj E7JV4ZYluraOxme3do9DUYX/NMV3tIkmMBi+0I+yvhHIsCkMWp6DmUFTU8QU00TD+fA9 ixbg== X-Gm-Message-State: AC+VfDxCX9TdSDJFg7NpwFEDS39A5kSa6jgTM0Oyarx+il8a4dk3jY+F 62R3FsAlDFbyNQyxJ6e/16e3NeZFWgICX6iNteAi4Y40+fRYbgPhy0hOYfEKtRVP5RgKqcJiZVk 58PqDXT6tUhFU19J8wwXbsPo7 X-Received: by 2002:a17:906:dc93:b0:94f:a292:20cc with SMTP id cs19-20020a170906dc9300b0094fa29220ccmr15643198ejc.41.1683713074310; Wed, 10 May 2023 03:04:34 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6g3VLlchZ4NQ4dVKKV7VFeDXbAdsULvGoTQhQBYH7wgUiWDxOc+E6cOLAsZqVz3BCBbS6/Sw== X-Received: by 2002:a17:906:dc93:b0:94f:a292:20cc with SMTP id cs19-20020a170906dc9300b0094fa29220ccmr15643186ejc.41.1683713074026; Wed, 10 May 2023 03:04:34 -0700 (PDT) Received: from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec? (2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl. [2001:1c00:c32:7800:5bfa:a036:83f0:f9ec]) by smtp.gmail.com with ESMTPSA id de25-20020a1709069bd900b0096664376ec9sm2486973ejc.55.2023.05.10.03.04.33 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 10 May 2023 03:04:33 -0700 (PDT) Message-ID: Date: Wed, 10 May 2023 12:04:32 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [REGRESSION] wifi: brcmfmac: add firmware vendor info in driver info Content-Language: en-US, nl To: Felix , Arend van Spriel , Franky Lin , Hante Meuleman Cc: regressions@lists.linux.dev, Arend van Spriel , Kalle Valo , linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com, SHA-cyfmac-dev-list@infineon.com References: <4ef3f252ff530cbfa336f5a0d80710020fc5cb1e.camel@gmail.com> From: Hans de Goede In-Reply-To: <4ef3f252ff530cbfa336f5a0d80710020fc5cb1e.camel@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Felix, On 5/10/23 10:25, Felix wrote: > Changes made by the mentioned commit lead to Oops when waking up after > suspend to RAM. > > When waking up a Dell XPS 13 9350 with BCM4350 wifi card after suspend > to RAM, the kernel hangs with a NULL pointer dereference and Oops. 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 machine. > > > Details: > > On a Dell XPS 13 9350 laptop with Broadcom BCM4350 wifi card (according > to lspci), every kernel starting with 6.2 hangs when waking up after > suspend to RAM. The issue persisted as of May 2nd (last tested commit > was 348551ddaf311c76b01cdcbaf61b6fef06a49144). Passing the debug and > no_console_suspend parameters to the kernel show that a NULL pointer > 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). > > A bisection was successful and produced the first bad commit > [da6d9c8ecd00e20218461007948f2b0a8e7fa242] wifi: brcmfmac: add firmware > vendor info in driver info > I've attached the bisection log as well (bisect.txt). > > Suspend to RAM works on this machine on newer kernels once the brcmfmac > module is unloaded. > > 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 default > config and make olddefconfig): > https://gist.githubusercontent.com/maclomhair/e33fa7eece8f8c77e5a88c4eede2f523/raw/429dfb022498c026225865dbb3ab2f75d5030a54/config > Dmesg output after boot: > https://gist.githubusercontent.com/maclomhair/47c75fa759d93800e7fb5e937fabef3e/raw/10750996a1ea9dcb3fa14d9b94e11fedd3abf280/dmesg > > #regzbot introduced da6d9c8ecd00e20218461007948f2b0a8e7fa242 Thank you for reporting this and thank you for bisecting it too! 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. 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. Regards, Hans