lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <1767472.DguSxT8x55@vostro.rjw.lan> Date: Sun, 13 Jan 2013 00:43:53 +0100 From: "Rafael J. Wysocki" <rjw@...k.pl> To: Yinghai Lu <yinghai@...nel.org> Cc: Bjorn Helgaas <bhelgaas@...gle.com>, Len Brown <lenb@...nel.org>, Taku Izumi <izumi.taku@...fujitsu.com>, Jiang Liu <jiang.liu@...wei.com>, linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org Subject: Re: [PATCH v8 16/22] PCI, acpiphp: Don't bailout even no slots found yet. On Friday, January 11, 2013 02:40:43 PM Yinghai Lu wrote: > Could have root bus hot addde later and there may be slots that need acpiphp. s/addde/added/ The changes look reasonable to me, but I'm not very familiar with the code in question. Thanks, Rafael > Signed-off-by: Yinghai Lu <yinghai@...nel.org> > --- > drivers/pci/hotplug/acpiphp.h | 1 - > drivers/pci/hotplug/acpiphp_core.c | 23 ++--------------------- > drivers/pci/hotplug/acpiphp_glue.c | 22 ---------------------- > 3 files changed, 2 insertions(+), 44 deletions(-) > > diff --git a/drivers/pci/hotplug/acpiphp.h b/drivers/pci/hotplug/acpiphp.h > index a1afb5b..b3ead7a 100644 > --- a/drivers/pci/hotplug/acpiphp.h > +++ b/drivers/pci/hotplug/acpiphp.h > @@ -193,7 +193,6 @@ extern void acpiphp_unregister_hotplug_slot(struct acpiphp_slot *slot); > /* acpiphp_glue.c */ > extern int acpiphp_glue_init (void); > extern void acpiphp_glue_exit (void); > -extern int acpiphp_get_num_slots (void); > typedef int (*acpiphp_callback)(struct acpiphp_slot *slot, void *data); > > extern int acpiphp_enable_slot (struct acpiphp_slot *slot); > diff --git a/drivers/pci/hotplug/acpiphp_core.c b/drivers/pci/hotplug/acpiphp_core.c > index 96316b7..c2fd309 100644 > --- a/drivers/pci/hotplug/acpiphp_core.c > +++ b/drivers/pci/hotplug/acpiphp_core.c > @@ -50,7 +50,6 @@ > bool acpiphp_debug; > > /* local variables */ > -static int num_slots; > static struct acpiphp_attention_info *attention_info; > > #define DRIVER_VERSION "0.5" > @@ -272,25 +271,6 @@ static int get_adapter_status(struct hotplug_slot *hotplug_slot, u8 *value) > return 0; > } > > -static int __init init_acpi(void) > -{ > - int retval; > - > - /* initialize internal data structure etc. */ > - retval = acpiphp_glue_init(); > - > - /* read initial number of slots */ > - if (!retval) { > - num_slots = acpiphp_get_num_slots(); > - if (num_slots == 0) { > - acpiphp_glue_exit(); > - retval = -ENODEV; > - } > - } > - > - return retval; > -} > - > /** > * release_slot - free up the memory used by a slot > * @hotplug_slot: slot to free > @@ -379,7 +359,8 @@ static int __init acpiphp_init(void) > return 0; > > /* read all the ACPI info from the system */ > - return init_acpi(); > + /* initialize internal data structure etc. */ > + return acpiphp_glue_init(); > } > > > diff --git a/drivers/pci/hotplug/acpiphp_glue.c b/drivers/pci/hotplug/acpiphp_glue.c > index 02c41ab..30467ec 100644 > --- a/drivers/pci/hotplug/acpiphp_glue.c > +++ b/drivers/pci/hotplug/acpiphp_glue.c > @@ -1446,28 +1446,6 @@ void acpiphp_glue_exit(void) > acpi_pci_unregister_driver(&acpi_pci_hp_driver); > } > > - > -/** > - * acpiphp_get_num_slots - count number of slots in a system > - */ > -int __init acpiphp_get_num_slots(void) > -{ > - struct acpiphp_bridge *bridge; > - int num_slots = 0; > - > - list_for_each_entry(bridge, &bridge_list, list) { > - dbg("Bus %04x:%02x has %d slot%s\n", > - pci_domain_nr(bridge->pci_bus), > - bridge->pci_bus->number, bridge->nr_slots, > - bridge->nr_slots == 1 ? "" : "s"); > - num_slots += bridge->nr_slots; > - } > - > - dbg("Total %d slots\n", num_slots); > - return num_slots; > -} > - > - > /** > * acpiphp_enable_slot - power on slot > * @slot: ACPI PHP slot > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists