On Mon, Jun 11, 2012 at 2:01 PM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > On Mon, Jun 11, 2012 at 10:43 AM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote: >> On Fri, Jun 8, 2012 at 12:01 AM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: >>> On Mon, May 28, 2012 at 12:33 AM, Jiang Liu <jiang.liu@xxxxxxxxxx> wrote: >>>> Changeset "9080648 PCI: acpiphp: remove all functions in slot, >>>> even without ACPI _EJx" introduces pci_bus_sem into the acpiphp >>>> driver, but pci_bus_sem hasn't been exported to modules yet. >>>> So export symbol pci_bus_sem to modules, otherwise the acpiphp >>>> driver can't be built as module any more. >>>> >>>> Signed-off-by: Yijing Wang <wangyijing@xxxxxxxxxx> >>>> Signed-off-by: Jiang Liu <liuj97@xxxxxxxxx> >>>> --- >>>> drivers/pci/search.c | 2 ++ >>>> 1 files changed, 2 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/drivers/pci/search.c b/drivers/pci/search.c >>>> index 9d75dc8..e0c7a22 100644 >>>> --- a/drivers/pci/search.c >>>> +++ b/drivers/pci/search.c >>>> @@ -15,6 +15,8 @@ >>>> #include "pci.h" >>>> >>>> DECLARE_RWSEM(pci_bus_sem); >>>> +EXPORT_SYMBOL(pci_bus_sem); >>>> + >>>> /* >>>> * find the upstream PCIe-to-PCI bridge of a PCI device >>>> * if the device is PCIE, return NULL >>> >>> Bjorn, >>> >>> can you please fold this one into >>> >>> 9080648: >>> PCI: acpiphp: remove all functions in slot, even without ACPI _EJx >>> >>> that commit now is in pci/for-3.6 >> >> Yes, I'll fix this. If you don't object, I'll make it >> EXPORT_SYMBOL_GPL(). I think we'll have more changes in the bus >> scan/remove paths, and it's possible we'll be able to un-export this >> in the future. > > yes, it should be EXPORT_SYMBOL_GPL. I applied this to my "next" branch, with EXPORT_SYMBOL_GPL. Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
