Messages in this thread | | | Date | Wed, 17 Oct 2012 09:28:09 -0700 | Subject | Re: [PATCH 1/1] Update acpi_root_bridge_list in container hotplug path. | From | Yinghai Lu <> |
| |
On Wed, Oct 17, 2012 at 12:39 AM, Tang Chen <tangchen@cn.fujitsu.com> wrote: > On 10/17/2012 01:18 PM, Yinghai Lu wrote: > > And also, I have another 2 questions, maybe you can help me. > 1) Do we need to put PNP0A08 into acpi_pci_roots ?
looks like we need to unify those two ids.
> 2) In container_notify_cb(), when it got a ACPI_NOTIFY_EJECT_REQUEST > event, it doesn't do the hot-remove things. > I use your sci emulator patch to test it. I did the following thing: > echo echo "\_SB_.LSB1" > /sys/kernel/debug/acpi/sci_notify > where \_SB_.LSB1 is a container, it just did nothing. > Do we need to support this operation ?
yes, looks like need to add container_device_remove and call it under container_notify_cb/ACPI_NOTIFY_EJECT_REQUEST
and should look like handle_root_bridge_removal to call acpi_bus_trim two times.
Thanks
Yinghai
| |