Re: [PATCH] PCI: release pci_host_bridge resource after remove root bus

[wangyijing <wangyijing@huawei.com>]

在 2016/8/25 5:48, Sinan Kaya 写道:
> On 8/24/2016 5:39 PM, Bjorn Helgaas wrote:
>>> Yes, may some pci drivers make a mistake, the refcount control the device object
>>>> release is fine, but I think move the mem resource release out is better.
>> If this is caused by driver bugs, I think we need to fix the driver
>> bugs.
>>
>> So far all I see here is "it works when I do this."  What we need is
>> an argument for "it's correct to do this."  It's certainly possible
>> that you're already making that argument and I'm just not
>> understanding it.
>>
> 
> I also want to point out that the sysfs is not the only device removal
> path. As Bjorn pointed out, you were moving resource free code into the sysfs
> path and breaking other path (ACPI hotplug).

No, this changes is also in ACPI hotplug path, but I found antoher problem,
now host bridge resource allocated pci_acpi_root_add_resources(), if we move the
free into pci_remove_root_bus(), it's not symmetrical as Bjorn pointed out.
This may introudce some problem, E.g. if we created host bridge, but create root bus
fail, in this case ,we have no way to free host bridge resource again.  :(


>