From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.kernel.org ([198.145.29.136]:39645 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752073AbcAFWEg (ORCPT ); Wed, 6 Jan 2016 17:04:36 -0500 Date: Wed, 6 Jan 2016 16:04:30 -0600 From: Bjorn Helgaas To: Sanjeev Sharma Cc: Richard.Zhu@freescale.com, l.stach@pengutronix.de, bhelgaas@google.com, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, David Mueller Subject: Re: [PATCH] PCI: imx6:don't sleep in atomic context Message-ID: <20160106220430.GC16231@localhost> References: <1447066080-5859-1-git-send-email-sanjeev_sharma@mentor.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1447066080-5859-1-git-send-email-sanjeev_sharma@mentor.com> Sender: linux-pci-owner@vger.kernel.org List-ID: Hi Sanjeev, On Mon, Nov 09, 2015 at 04:18:00PM +0530, Sanjeev Sharma wrote: > If additional PCIe switch get connected between the > host and the NIC,the kernel crashes with "BUG: > scheduling while atomic". To handle this we need to > call mdelay() instead of usleep_range(). > > For more detail please refer bugzilla.kernel.org, Bug > 100031 > > Signed-off-by: Sanjeev Sharma > Signed-off-by: David Mueller I'm dropping this for now because we've been kicking around the same solution (with tweaks to the mdelay amount) since June, but no progress on the *real* issue, which is that imx6_pcie_link_up() should never wait; it should simply return the link status. I'm pretty sure the amount of time I've spent looking into this would have been enough to make some progress on that underlying issue. Bjorn > --- > drivers/pci/host/pci-imx6.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/host/pci-imx6.c b/drivers/pci/host/pci-imx6.c > index 233a196..9769b13 100644 > --- a/drivers/pci/host/pci-imx6.c > +++ b/drivers/pci/host/pci-imx6.c > @@ -499,7 +499,7 @@ static int imx6_pcie_link_up(struct pcie_port *pp) > * Wait a little bit, then re-check if the link finished > * the training. > */ > - usleep_range(1000, 2000); > + mdelay(1000); > } > /* > * From L0, initiate MAC entry to gen2 if EP/RC supports gen2. > -- > 1.7.11.7 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html