DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type:content-transfer-encoding;
        b=jL2jyUkykMY7f42c2gywcSLf0dodZfEf5XQz5nUM+z84kmSPxmzSw1i8L8f6TV0fbc
         Ypnvafr3JAYAYlVW66Dw4CIze+7dLrOIneGlfQNGwQ8IaK/CVYdwGYD7etSB5Ow9sEAZ
         bImBj4A8K/8MLOau6T6eGiPir1jY9mPSf4uAk=
MIME-Version: 1.0
In-Reply-To: <alpine.LFD.2.00.0906251548300.2760@troy-laptop>
References: <alpine.LFD.2.00.0906251548300.2760@troy-laptop>
Date: Fri, 26 Jun 2009 00:02:07 +0800
Message-ID: <b6a2187b0906250902xede3041p599e538099340a04@mail.gmail.com>
Subject: Re: [2.6.31-rc1] oops in acpi_get_pci_dev
From: Jeff Chua <jeff.chua.linux@gmail.com>
To: Troy Moure <twmoure@szypr.net>
Cc: Alessandro Suardi <alessandro.suardi@gmail.com>,
       Alexander Chiang <achiang@hp.com>, Bjorn Helgaas <bjorn.helgaas@hp.com>,
       linux-kernel@vger.kernel.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1064
Lines: 29

On Thu, Jun 25, 2009 at 11:13 PM, Troy Moure<twmoure@szypr.net> wrote:
> (I don't know if it's the "right" fix or not, not being familiar with the
> system. ?If it is, I can send it in as a proper patch.)
>
> diff --git a/drivers/acpi/pci_root.c b/drivers/acpi/pci_root.c
> index 8a5bf3b..55b5b90 100644
> --- a/drivers/acpi/pci_root.c
> +++ b/drivers/acpi/pci_root.c
> @@ -395,7 +395,7 @@ struct pci_dev *acpi_get_pci_dev(acpi_handle handle)
> ? ? ? ? ? ? ? ?fn ?= adr & 0xffff;
>
> ? ? ? ? ? ? ? ?pdev = pci_get_slot(pbus, PCI_DEVFN(dev, fn));
> - ? ? ? ? ? ? ? if (hnd == handle)
> + ? ? ? ? ? ? ? if (!pdev || hnd == handle)
> ? ? ? ? ? ? ? ? ? ? ? ?break;
>
> ? ? ? ? ? ? ? ?pbus = pdev->subordinate;

I've the same problem with booting hangs, and your patch fixed the
problem  on my ThinkPad X61.


Thanks,
Jeff.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/