Re: About dynamically loading the acpiphp.ko

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 24, 2013 at 3:55 PM, Rafael J. Wysocki <rjw@xxxxxxx> wrote:
> On Wednesday, April 24, 2013 02:03:04 PM Bjorn Helgaas wrote:
>>
>> We install a notify handler on PCI0 in find_root_bridges(), and that
>> notify handler leads to _handle_hotplug_event_root(), which assumes
>> this is a host bridge hotplug event.  But I would expect the notify
>> for a host bridge hot-add would go to the *parent* of the host bridge,
>> not to the host bridge itself, so something seems wrong here.
>>
>> Maybe the ACPI & hotplug experts can comment on this.
>
> I think it may (ie. won't be against the spec) go both ways, so it may go
> to the parent or the host bridge object itself.  We should be handling both
> cases IMO.

Yes, we missed that path.

Attached patch should fix the problem.

Gavin,

Can you please check that on top of pci/next?

Thanks

Yinghai

Attachment: acpiphp_check_host_bridge.patch
Description: Binary data


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux