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

From	"Sean O. Stalley" <sean.stalley@intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and assigning resources.
Date	2015-10-06 00:50 +0200
Message-ID	<qgmJY-4k3-23@gated-at.bofh.it> (permalink)
References	<qfh9E-8tG-13@gated-at.bofh.it> <qfh9E-8tG-19@gated-at.bofh.it> <qfhMm-10f-13@gated-at.bofh.it> <qflmV-6d8-3@gated-at.bofh.it> <qflmV-6d8-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Fri, Oct 02, 2015 at 08:00:20PM -0700, Yinghai Lu wrote:
> On Fri, Oct 2, 2015 at 4:38 PM, David Daney <ddaney@caviumnetworks.com> wrote:
> > On 10/02/2015 04:14 PM, Yinghai Lu wrote:
> >>
> >> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and assigning resources. David Daney <ddaney.cavm@gmail.com> - 2015-10-03 00:40 +0200
  Re: [PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and  assigning resources. Yinghai Lu <yinghai@kernel.org> - 2015-10-03 01:20 +0200
    Re: [PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and  assigning resources. Yinghai Lu <yinghai@kernel.org> - 2015-10-03 05:10 +0200
      Re: [PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and  assigning resources. "Sean O. Stalley" <sean.stalley@intel.com> - 2015-10-06 00:50 +0200
  Re: [PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and  assigning resources. "Sean O. Stalley" <sean.stalley@intel.com> - 2015-10-06 00:30 +0200
    Re: [PATCH v4 3/5] PCI: Handle IORESOURCE_PCI_FIXED when sizing and  assigning resources. David Daney <ddaney.cavm@gmail.com> - 2015-10-06 23:00 +0200

csiph-web