From patchwork Tue Jun 26 14:45:37 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stefan Agner X-Patchwork-Id: 10489249 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 3E09C6031B for ; Tue, 26 Jun 2018 14:46:22 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 2AF43219AC for ; Tue, 26 Jun 2018 14:46:22 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 1E9C22223E; Tue, 26 Jun 2018 14:46:22 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id D14D8219AC for ; Tue, 26 Jun 2018 14:46:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Message-ID:References:In-Reply-To: Subject:To:From:Date:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=I3mu90zs4CHPO7aa4q8mXobm/Y4j+0HaD1RFmLUE1ow=; b=B7nRycWQJPM/Pu 6x7DT7eE4L6qtJcQ79teJdZ+ud9lRfVhKJG/Q7BCaysK8AlMsLZta1ZEaL7ztB20vyqac7As0uO1E xp30Xo5CS8swfAa7cJCI0njF0h4TCtpEyX/9JoUwnAM+wGm+5Gy8J00auQari8nIOV/WlhM7Pwbde ZGKEzpbHbvfQVVb73cGGzdtAePmRVGb6Fh9tFa8yJcMHeDklvGJ51WvR8zAdMVp1gwndRVh4hRR2+ qaF3qOplzuVXa6qpkTUx3LykuT7PtaOwzJr8OWRKwdd3WsVYS/pDe3+uCA7L8yTwPmQqx36kgQHu8 2O417l5wgnjv9h7ynBQg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fXpET-0000EP-1U; Tue, 26 Jun 2018 14:46:13 +0000 Received: from mail.kmu-office.ch ([2a02:418:6a02::a2]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1fXpE8-0000A2-Hb for linux-arm-kernel@lists.infradead.org; Tue, 26 Jun 2018 14:46:10 +0000 Received: from webmail.kmu-office.ch (unknown [IPv6:2a02:418:6a02::a3]) by mail.kmu-office.ch (Postfix) with ESMTPSA id 1AC3F5C0E11; Tue, 26 Jun 2018 16:45:38 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=agner.ch; s=dkim; t=1530024338; 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=hyXPwCzfFDemBbz6eljmyp5qrgNChGkiX73eyjRguyc=; b=bv0S60JJLQBenpw0QSof7xXOpy1pmbxrsiwYvXl13/5kd+CstQpqI4b4Uf/Z/1El3A13wW TQH7068yim/7ZSaugDKGnHh0A1apSpcV/EFzdJErndqqipT9sy0nCaUy+y6qcssxxYbR+6 URZBdpPneNLnqewZD5ySd2YiR+mEm8Q= MIME-Version: 1.0 Date: Tue, 26 Jun 2018 16:45:37 +0200 From: Stefan Agner To: adrian.hunter@intel.com, aisheng.dong@nxp.com, fabio.estevam@nxp.com Subject: Re: Self-detected stall on CPU when using SD card In-Reply-To: <037cbd3560a68bad0cea92f453c1b819@agner.ch> References: <037cbd3560a68bad0cea92f453c1b819@agner.ch> Message-ID: <0f262b04fab61ec75b7e368a8f4968aa@agner.ch> X-Sender: stefan@agner.ch User-Agent: Roundcube Webmail/1.3.4 X-Spamd-Result: default: False [-3.09 / 15.00]; TO_MATCH_ENVRCPT_ALL(0.00)[]; MID_RHS_MATCH_FROM(0.00)[]; BAYES_HAM(-2.99)[99.96%]; RCPT_COUNT_FIVE(0.00)[6]; MIME_GOOD(-0.10)[text/plain]; FROM_HAS_DN(0.00)[]; FROM_EQ_ENVFROM(0.00)[]; DKIM_SIGNED(0.00)[]; RCVD_COUNT_ZERO(0.00)[0]; ASN(0.00)[asn:29691, ipnet:2a02:418::/29, country:CH]; TO_DN_NONE(0.00)[]; RCVD_TLS_ALL(0.00)[]; ARC_NA(0.00)[] X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20180626_074552_756563_C9383976 X-CRM114-Status: GOOD ( 15.06 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-mmc-owner@vger.kernel.org, linux-mmc@vger.kernel.org, linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP On 26.06.2018 12:53, Stefan Agner wrote: > Hi, > > On our Colibri iMX6 (arch/arm/boot/dts/imx6qdl-colibri.dtsi) we > experience the following stack trace when a SD card is plugged in: > [...] > [] (esdhc_pltfm_set_clock) from [] > (sdhci_set_ios+0xd8/0x584) > r10:ffffe000 r9:c1105900 r8:00004097 r7:d818f2e8 r6:d818f480 > r5:d818f2e8 > r4:d818f000 > [] (sdhci_set_ios) from [] > (sdhci_runtime_resume_host+0xa0/0x184) > r9:c1105900 r8:00004097 r7:d818f2e8 r6:d818f648 r5:d818f000 r4:d818f480 [...] > > It used to work in v4.9, so I started a git bisect. It pointed me to > this commit: > > Commit d1e4f74f911d ("mmc: sdhci: Do not use spin lock in set_ios > paths"). > > Reverting the commit on-top of v4.18-rc1 seems to fix the issue too. > > Any idea? I figured out that the same platform had a GPIO Key which triggered all the time. This seems to exacerbate the MMC issue such that it triggers on very boot, about rootfs mount time. This change seems to fix the issue as well, not sure though whether this is a proper fix: if (clock == 0) { @@ -724,6 +725,7 @@ static inline void esdhc_pltfm_set_clock(struct sdhci_host *host, pre_div = 2; } + spin_lock_irqsave(&host->lock, flags); temp = sdhci_readl(host, ESDHC_SYSTEM_CONTROL); temp &= ~(ESDHC_CLOCK_IPGEN | ESDHC_CLOCK_HCKEN | ESDHC_CLOCK_PEREN | ESDHC_CLOCK_MASK); @@ -754,6 +756,7 @@ static inline void esdhc_pltfm_set_clock(struct sdhci_host *host, writel(val | ESDHC_VENDOR_SPEC_FRC_SDCLK_ON, host->ioaddr + ESDHC_VENDOR_SPEC); } + spin_unlock_irqrestore(&host->lock, flags); mdelay(1); } --- Stefan --- a/drivers/mmc/host/sdhci-esdhc-imx.c +++ b/drivers/mmc/host/sdhci-esdhc-imx.c @@ -697,6 +697,7 @@ static inline void esdhc_pltfm_set_clock(struct sdhci_host *host, int ddr_pre_div = imx_data->is_ddr ? 2 : 1; int pre_div = 1; int div = 1; + unsigned long flags; u32 temp, val;