From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ogre.sisk.pl ([217.79.144.158]:56145 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753324Ab1LMVGD (ORCPT ); Tue, 13 Dec 2011 16:06:03 -0500 From: "Rafael J. Wysocki" To: Jesse Barnes Subject: Re: [PATCH][Regression fix] PCI / Hotplug: Always allow acpiphp to handle non-PCIe bridges Date: Tue, 13 Dec 2011 22:09:04 +0100 Cc: LKML , Linus Torvalds , linux-pci@vger.kernel.org, Kenji Kaneshige References: <201112130002.28907.rjw@sisk.pl> <20111213104220.5cfb78b2@jbarnes-desktop> In-Reply-To: <20111213104220.5cfb78b2@jbarnes-desktop> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-2" Message-Id: <201112132209.04185.rjw@sisk.pl> Sender: linux-pci-owner@vger.kernel.org List-ID: On Tuesday, December 13, 2011, Jesse Barnes wrote: > On Tue, 13 Dec 2011 00:02:28 +0100 > "Rafael J. Wysocki" wrote: > > > From: Rafael J. Wysocki > > > > Commit 0d52f54e2ef64c189dedc332e680b2eb4a34590a (PCI / ACPI: Make > > acpiphp ignore root bridges using PCIe native hotplug) added code > > that made the acpiphp driver completely ignore PCIe root complexes > > for which the kernel had been granted control of the native PCIe > > hotplug feature by the BIOS through _OSC. Unfortunately, however, > > this was a mistake, because on some systems there were PCI bridges > > supporting PCI (non-PCIe) hotplug under such root complexes and > > those bridges should have been handled by acpiphp. > > > > For this reason, revert the changes made by the commit mentioned > > above and make register_slot() in drivers/pci/hotplug/acpiphp_glue.c > > avoid registering hotplug slots for PCIe ports that belong to > > root complexes with native PCIe hotplug enabled (which means that > > the BIOS has granted the kernel control of this feature for the > > given root complex). This is reported to address the original > > issue fixed by commit 0d52f54e2ef64c189dedc332e680b2eb4a34590a and > > to work on the system where that commit broke things. > > > > Signed-off-by: Rafael J. Wysocki > > --- > > Applied to my for-linus branch. I'll probably send to Linus on Thu. or > so. > > I also rebased the breakage out of linux-next. Thanks a lot! Rafael