Re: [PATCH v8 10/13] PCI/acpiphp: do not use ACPI PCI subdriver mechanism

2013-04-11 Thread Yijing Wang
On 2013/4/12 1:29, Bjorn Helgaas wrote: > On Wed, Apr 10, 2013 at 7:50 PM, Yijing Wang wrote: Hi Bjorn, Thanks for review. > My goal is that a user should never have to specify a kernel boot > parameter or edit a modules.conf file, but the user did previously > h

Re: [PATCH v8 10/13] PCI/acpiphp: do not use ACPI PCI subdriver mechanism

2013-04-11 Thread Bjorn Helgaas
On Wed, Apr 10, 2013 at 7:50 PM, Yijing Wang wrote: >>> Hi Bjorn, >>> Thanks for review. >>> My goal is that a user should never have to specify a kernel boot parameter or edit a modules.conf file, but the user did previously have some way to influence whether we use pciehp

Re: [PATCH v8 10/13] PCI/acpiphp: do not use ACPI PCI subdriver mechanism

2013-04-10 Thread Yijing Wang
>> Hi Bjorn, >> Thanks for review. >> >>> My goal is that a user should never have to specify a kernel boot >>> parameter or edit a modules.conf file, but the user did previously >>> have some way to influence whether we use pciehp or acpiphp. I know >>> we still have some issues, particul

Re: [PATCH v8 10/13] PCI/acpiphp: do not use ACPI PCI subdriver mechanism

2013-04-10 Thread Bjorn Helgaas
On Wed, Apr 10, 2013 at 10:14 AM, Jiang Liu wrote: > On 04/10/2013 07:38 AM, Bjorn Helgaas wrote: >> On Tue, Feb 26, 2013 at 8:25 AM, Jiang Liu wrote: >>> Previously the acpiphp driver registers itself as an ACPI PCI subdriver, >>> so it's callbacks will be invoked when creating/destroying PCI ro

Re: [PATCH v8 10/13] PCI/acpiphp: do not use ACPI PCI subdriver mechanism

2013-04-10 Thread Jiang Liu
On 04/10/2013 07:38 AM, Bjorn Helgaas wrote: > On Tue, Feb 26, 2013 at 8:25 AM, Jiang Liu wrote: >> Previously the acpiphp driver registers itself as an ACPI PCI subdriver, >> so it's callbacks will be invoked when creating/destroying PCI root >> buses to manage ACPI based PCI hotplug slots. But i

Re: [PATCH v8 10/13] PCI/acpiphp: do not use ACPI PCI subdriver mechanism

2013-04-09 Thread Bjorn Helgaas
On Tue, Feb 26, 2013 at 8:25 AM, Jiang Liu wrote: > Previously the acpiphp driver registers itself as an ACPI PCI subdriver, > so it's callbacks will be invoked when creating/destroying PCI root > buses to manage ACPI based PCI hotplug slots. But it doesn't handle > P2P bridge hotplug events, so i