On Thu, Oct 4, 2012 at 2:53 PM, Rafael J. Wysocki <rjw@xxxxxxx> wrote: > On Thursday 04 of October 2012 14:31:46 Yinghai Lu wrote: >> On Thu, Oct 4, 2012 at 2:23 PM, Rafael J. Wysocki <rjw@xxxxxxx> wrote: >> >> >> >> At last we could remove all acpi_bus_start workaround. >> > >> > Do I understand correctly that you just want to prevent acpi_pci_root_driver >> > from binding to the host bridge's struct acpi_device created while we're >> > walking the ACPI namespace? >> >> yes, during hot add path. > > Why exactly do you want to prevent that from happening? > during adding pci root bus hotplug, Bjorn found some unsafe searching that caused by pci_bus_add_devices. pci devices are created during pci scan root, but until very late acpi_pci_root_start call pci_bus_add_devices. To fill the gap, we need to move pci_bus_add_devices to acpi_pci_root_add at first. but after we move that there, pci device will be added to device tree, and it will try to bind with acpi devices that should be under acpi pci root, but are not created yet. because device_add for acpi_device for acpi pci root is done yet. it still calling the .add in the acpi_driver aka acpi_pci_root_add. So I want to hold the driver attach for pci root acpi devices, and later attach it until pci devices created. booting path, all acpi devices get created, and attach driver for them one by one. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
