Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752323AbdFMFyv (ORCPT ); Tue, 13 Jun 2017 01:54:51 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:54544 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751955AbdFMFyt (ORCPT ); Tue, 13 Jun 2017 01:54:49 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 6EA32601D9 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=kvalo@codeaurora.org Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: brcmfmac: Fix kernel oops on resume when request firmware fails. From: Kalle Valo In-Reply-To: <20170523180733.26276-1-enric.balletbo@collabora.com> References: <20170523180733.26276-1-enric.balletbo@collabora.com> To: Enric Balletbo i Serra Cc: Arend van Spriel , linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Hante Meuleman , Christian Daudt User-Agent: pwcli/0.0.0-git (https://github.com/kvalo/pwcli/) Python/2.7.12 Message-Id: <20170613055448.C0CD36075B@smtp.codeaurora.org> Date: Tue, 13 Jun 2017 05:54:48 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 985 Lines: 27 Enric Balletbo i Serra wrote: > When request firmware fails, brcmf_ops_sdio_remove is being called and > brcmf_bus freed. In such circumstancies if you do a suspend/resume cycle > the kernel hangs on resume due a NULL pointer dereference in resume > function. > > Steps to reproduce the problem: > - modprobe brcmfmac without the firmware > brcmfmac mmc1:0001:1: Direct firmware load for brcm/brcmfmac4354-sdio.bin > failed with error -2 > - do a suspend/resume cycle (echo mem > /sys/power/state) > > Protect against the NULL pointer derefence by checking if dev_get_drvdata > returned a valid pointer. > > Signed-off-by: Enric Balletbo i Serra My understanding is that there's a new version of this patch which fixes the issue. If not, let me know. Patch set to Superseded. -- https://patchwork.kernel.org/patch/9743159/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches