Re: [PATCH] PCI: Fix undefined reference to 'pci_fixup_final_inited'

[Bjorn Helgaas]

On Fri, Jul 13, 2012 at 4:53 PM, Randy Dunlap <rdunlap@xxxxxxxxxxxx> wrote:
> On 07/13/2012 01:29 PM, Myron Stowe wrote:
>
>> My "PCI: Integrate 'pci_fixup_final' quirks into hot-plug paths" patch
>> introduced an undefined reference to 'pci_fixup_final_inited' when
>> CONFIG_PCI_QUIRKS is not enabled (on x86_64):
>>   drivers/built-in.o: In function `pci_bus_add_device':
>>   (.text+0x4f62): undefined reference to `pci_fixup_final_inited'
>>
>> This patch removes the external reference ending up with a result closer
>> to what we ultimately want when the boot path issues described in the
>> original patch are resolved.
>>
>> References:
>>   https://lkml.org/lkml/2012/7/9/542    Original, offending, patch
>>   https://lkml.org/lkml/2012/7/12/338 Randy's catch
>>
>> Reported-by: rdunlap@xxxxxxxxxxxx
>> Signed-off-by: Myron Stowe <myron.stowe@xxxxxxxxxx>
>
>
> Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
>
> Thanks.

I added this to my "next" branch, thanks!

>>
>>  drivers/pci/bus.c    |    4 +---
>>  drivers/pci/quirks.c |   20 ++++----------------
>>  2 files changed, 5 insertions(+), 19 deletions(-)
>>
>> diff --git a/drivers/pci/bus.c b/drivers/pci/bus.c
>> index b511bd4..4b0970b 100644
>> --- a/drivers/pci/bus.c
>> +++ b/drivers/pci/bus.c
>> @@ -164,10 +164,8 @@ pci_bus_alloc_resource(struct pci_bus *bus, struct resource *res,
>>  int pci_bus_add_device(struct pci_dev *dev)
>>  {
>>       int retval;
>> -     extern bool pci_fixup_final_inited;
>>
>> -     if (pci_fixup_final_inited)
>> -             pci_fixup_device(pci_fixup_final, dev);
>> +     pci_fixup_device(pci_fixup_final, dev);
>>       retval = device_add(&dev->dev);
>>       if (retval)
>>               return retval;
>> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
>> index 52f44b5..003f356 100644
>> --- a/drivers/pci/quirks.c
>> +++ b/drivers/pci/quirks.c
>> @@ -2956,6 +2956,7 @@ extern struct pci_fixup __end_pci_fixups_resume_early[];
>>  extern struct pci_fixup __start_pci_fixups_suspend[];
>>  extern struct pci_fixup __end_pci_fixups_suspend[];
>>
>> +static bool pci_apply_fixup_final_quirks;
>>
>>  void pci_fixup_device(enum pci_fixup_pass pass, struct pci_dev *dev)
>>  {
>> @@ -2973,6 +2974,8 @@ void pci_fixup_device(enum pci_fixup_pass pass, struct pci_dev *dev)
>>               break;
>>
>>       case pci_fixup_final:
>> +             if (!pci_apply_fixup_final_quirks)
>> +                     return;
>>               start = __start_pci_fixups_final;
>>               end = __end_pci_fixups_final;
>>               break;
>> @@ -3006,21 +3009,6 @@ void pci_fixup_device(enum pci_fixup_pass pass, struct pci_dev *dev)
>>  EXPORT_SYMBOL(pci_fixup_device);
>>
>>
>> -/*
>> - * The global variable 'pci_fixup_final_inited' is being used as a interim
>> - * solution for calling the final quirks only during hot-plug events (not
>> - * during boot processing).
>> - *
>> - * When the boot path's PCI device setup sequencing is addressed, we can
>> - * remove the instance, and usages of, 'pci_fixup_final_inited' along with
>> - * removing 'fs_initcall_sync(pci_apply_final_quirks);' and end up with a
>> - * single, uniform, solution that satisfies both the boot path and the
>> - * various hot-plug event paths.
>> - *
>> - * ToDo: Remove 'pci_fixup_final_inited'
>> - */
>> -bool pci_fixup_final_inited;
>> -
>>  static int __init pci_apply_final_quirks(void)
>>  {
>>       struct pci_dev *dev = NULL;
>> @@ -3031,6 +3019,7 @@ static int __init pci_apply_final_quirks(void)
>>               printk(KERN_DEBUG "PCI: CLS %u bytes\n",
>>                      pci_cache_line_size << 2);
>>
>> +     pci_apply_fixup_final_quirks = true;
>>       for_each_pci_dev(dev) {
>>               pci_fixup_device(pci_fixup_final, dev);
>>               /*
>> @@ -3051,7 +3040,6 @@ static int __init pci_apply_final_quirks(void)
>>                       pci_cache_line_size = pci_dfl_cache_line_size;
>>               }
>>       }
>> -     pci_fixup_final_inited = 1;
>>
>>       if (!pci_cache_line_size) {
>>               printk(KERN_DEBUG "PCI: CLS %u bytes, default %u\n",
>>
>
>
>
> --
> ~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/