On Tue, Sep 18, 2012 at 12:24 AM, Taku Izumi <izumi.taku@xxxxxxxxxxxxxx> wrote: > > Use mutex to protect acpi_pci_roots list against PCI host bridge > hotplug operations. > > Signed-off-by: Taku Izumi <izumi.taku@xxxxxxxxxxxxxx> > --- > drivers/acpi/pci_root.c | 27 ++++++++++++++++++--------- > 1 file changed, 18 insertions(+), 9 deletions(-) > > Index: Bjorn-next-0903/drivers/acpi/pci_root.c > =================================================================== > --- Bjorn-next-0903.orig/drivers/acpi/pci_root.c > +++ Bjorn-next-0903/drivers/acpi/pci_root.c > @@ -112,12 +112,17 @@ EXPORT_SYMBOL(acpi_pci_unregister_driver > acpi_handle acpi_get_pci_rootbridge_handle(unsigned int seg, unsigned int bus) > { > struct acpi_pci_root *root; > + struct acpi_handle *handle = NULL; This should be "acpi_handle handle", not "struct acpi_handle *handle". > + mutex_lock(&acpi_pci_root_lock); > list_for_each_entry(root, &acpi_pci_roots, node) > if ((root->segment == (u16) seg) && > - (root->secondary.start == (u16) bus)) > - return root->device->handle; > - return NULL; > + (root->secondary.start == (u16) bus)) { > + handle = root->device->handle; > + break; > + } > + mutex_unlock(&acpi_pci_root_lock); > + return handle; > } -- 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