On Fri, Jun 24, 2016 at 12:34 PM, Aaron Durbin <adurbin@xxxxxxxxxx> wrote:
> On Tue, Jun 14, 2016 at 12:04 PM, Aaron Durbin <adurbin@xxxxxxxxxx> wrote:
>> On Wed, May 18, 2016 at 3:32 PM, Aaron Durbin <adurbin@xxxxxxxxxx> wrote:
>>> On Wed, May 18, 2016 at 4:25 PM, Rafael J. Wysocki <rafael@xxxxxxxxxx> wrote:
>>>> On Wed, May 18, 2016 at 5:54 PM, Aaron Durbin <adurbin@xxxxxxxxxx> wrote:
>>>>> Hi,
>>>>>
>>>>> We're currently running into a problem of resource conflicts with a
>>>>> PCI device and ACPI devices.
>>>>>
>>>>> [ 0.243534] pci 0000:00:0d.0: can't claim BAR 0 [mem
>>>>> 0xd0000000-0xd0ffffff 64bit]: address conflict with INT3452:03 [mem
>>>>> 0xd0c00000-0xd0c03fff]
>>>>>
>>>>> The PCI BAR covers a large amount mmio resources, however, there are
>>>>> ACPI devices with their own HID (for probing) which uses resources
>>>>> that are a subset of the PCI BAR.
>>>>>
>>>>> Short of re-structuring the linux driver is there anything that can be
>>>>> done with ASL to properly have the ACPI device use a sub-resource of
>>>>> the PCI device during the ACPI/PCI probing?
>>>>
>>>> Do you have an ACPI device object corresponding to the PCI device?
>>>
>>> I've been debugging this by proxy, and I did request that test. The
>>> following is the overall structure:
>>>
>>> scope (\_SB.PCI0) {
>>>
>>> Device (P2S)
>>> {
>>> Name (_ADR, 0x000D0000)
>>> Device (GPO0)
>>> {
>>> Name (_ADR, 0)
>>> Name (_HID, "INT3452")
>>> Name (_CID, "INT3452")
>>> }
>>> }
>>> }
>>>
>>> There are _STA methods in both Devices. The GP0 device has a _CRS
>>> method which just returns a ResourceTemplate which is filled in with
>>> static values. The PCI bar is at a fixed address from the firmware
>>> which allows the fixed calculations. However there is no specific
>>> reference to the P2S device's resources proper -- only the parent
>>> child relationship within the ASL. I'm not sure how to directly say "I
>>> want this sub-region of this other device's resource for my resource."
>>> That seems like the right thing, but it's not clear if that's implied
>>> by hierarchy of the devices.
>>>
>>> Lastly, if it helps, the kernel being used is based on v4.4 (no core
>>> patches on top).
>>>
>>
>> Hi Rafael,
>>
>> I haven't tried a newer kernel yet, but are you of the opinion that
>> having the Devices as parent-child within the ASL should work? I'm
>> wondering if there's already a patch in newer kernels that doesn't
>> report the conflict and works as expected once there are child Devices
>> under the P2S device.
>>
>
> I've been looking at this more closely. A child ACPI device under a
> ACPI PCI device doesn't change the resource conflict even when a _CRS
> method is added to the ACPI PCI device. Below is my sleuthing which
> is probably not a surprise to anyone here, but please correct me where
> I am wrong.
>
> acpi_init() and pci_subsys_init() are both subsys_initcalls during
> boot up. I'm not sure if the ordering is dumb luck or not, but
> acpi_init() is called prior to pci_subsys_init(). The conflict error
> is spit out from pcibios_resource_survey() by way of pci_subsys_init()
> subsys_initcall. However, the PCI device scanning is kicked off prior
> to this through acpi_scan_init() by way of acpi_init()
> subsys_initcall. The conflict error occurs because there's already
> the child ACPI device in the resource tree. I'm not sure when/where
> those ACPI devices' resources are added, but clearly they are sitting
> in there since the conflict was found.
>
> Somewhere along the way a PCI device from a scan is linked with the
> ACPI device for that same PCI device in sysfs. This is with me
> putting a _HID and _CID in the PCI ACPI device.
> # readlink -f /sys/devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/INT5A92:00/physical_node
> /sys/devices/pci0000:00/0000:00:0d.0
> # readlink -f /sys/devices/pci0000\:00/0000\:00\:0d.0/firmware_node/
> /sys/devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/INT5A92:00
>
> So the hierarchy is known eventually, but it's clearly not honored
> when adding resources. The current ACPI support doesn't handle
> PciBarTarget which initially sounds (from ACPI spec) like the way to
> go for referencing a resource in a PCI device from an ACPI device. So
> that's out of the question currently, but maybe someone has a patch
> for that? I don't think reordering the acpi_init() and
> pci_subsys_init() would do anything different except change which
> device discovers the conflict.
>
> Is there a way to honor the ACPI device hierarchy during resource
> addition for the PCI devices? The conflict is found because of the
> presence of a child device claiming resources through _CRS.
> Alternatively, is there a good way to defer the probing of an ACPI
> device until one knows PCI resources have been added?
>
> Any insights would be very helpful. Thank you.
I stumbled upon the hierarchy connection. That's all handled with the
platform_notify() end of things when device_add() is done on the pci
device. I was thinking we could take advantage of this when adding
resources, but a struct resource has no struct device. It's just a
name description for the resource at hand. However, platform devices
are added when the ACPI tree is parsed along with adding the resources
associated with them (PciBarTarget would be helpful here) so those
resources are sitting in the resource tree when PCI BARs are added.
The following suggestion is sort of hacky, but it's the best I could
come up with provided the currently supported infrastructure. In
pci_claim_resource() do request_resource_conflict() as before. If it
fails do the following: 1. check if the device has an ACPI companion.
2. For any children hanging off the ACPI companion device. check if
that device's name matches the conflict resource's name. 3. If so,
insert_resource_conflict() to place the BAR within the tree itself.
Thoughts or suggestions are welcomed.
Thank you.
-Aaron
--
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
