Re: [PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and assigning resources.

On Fri, Oct 02, 2015 at 08:00:20PM -0700, Yinghai Lu wrote:

On Fri, Oct 2, 2015 at 4:38 PM, David Daney [off-list ref] wrote:

quoted

On 10/02/2015 04:14 PM, Yinghai Lu wrote:

quoted

https://patchwork.kernel.org/patch/7304371/
[v6,06/53] PCI: Claim fixed resource during remove/rescan path

This one is interesting, but I don't think it will work.

pci_claim_resource() calls pci_find_parent_resource(), which will fail in
important use cases.

It is perfectly legal for a bridge provisioned by EA to not specify any
resources.  In this case we must walk up the bus tree until we find
something that contains the device resource, and can thus be a parent.

That is a big part of what my patch is doing.

looks we need another resource flags for EA in addition to
FIXED as it could out side of bridge MMIO range.

I think I get what you are saying, but let me check.

If a bridge gets reconfigured, it needs to be able to tell if this resource is EA or not.
If it is a fixed EA resource, it doesn't need to include it in its MMIO range.
If it is fixed for some other reason,
	we should make sure the bridges MMIO range includes the fixed region.

You are suggesting that we should add a flag for EA, so if a bridge tries to move,
the bridge knows weather or not the resource is EA,
and therefore if it needs to worry about the resource when setting it's range?

Is that what you are trying to say?

-Sean