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

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

 




在 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.  :(


> 

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux