From: Bjorn Helgaas <helgaas@kernel.org>
To: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Sinan Kaya <okaya@codeaurora.org>,
linux-pci@vger.kernel.org, Zhou Wang <wangzhou1@hisilicon.com>,
Phil Edworthy <phil.edworthy@renesas.com>,
Bjorn Helgaas <bhelgaas@google.com>
Subject: Re: [RFC] ARM/ARM64 PCI_PROBE_ONLY platforms
Date: Mon, 1 Feb 2016 15:12:58 -0600 [thread overview]
Message-ID: <20160201211258.GD4861@localhost> (raw)
In-Reply-To: <20160201152507.GA30869@red-moon>
On Mon, Feb 01, 2016 at 03:25:07PM +0000, Lorenzo Pieralisi wrote:
> On Sat, Jan 30, 2016 at 07:30:27AM -0600, Bjorn Helgaas wrote:
>
> [...]
>
> > > Most non-UEFI firmwares I have seen on ARM rely on device specific
> > > driver like synopsys etc. to do the device initialization and ask
> > > kernel to do the enumeration.
> > >
> > > ACPI systems on the other hand handle the resource assignment before
> > > the OS starts.
> >
> > My guess is that this is more of a tradition than anything actually
> > required by the spec.
>
> I share your opinion, and that tradition on ARM64 should be built
> on top of existing DT based systems where the bootloader assigns
> *NOTHING* in 90% of designs.
>
> That's why I want to see resource claiming carried out by default
> on ACPI on ARM64, this would foster the tradition :), hopefully.
>
> > The bottom line is that Linux can't rely on much consistency across
> > the universe of architectures and firmwares. I think the only thing
> > that really makes sense for us to do is:
> >
> > - Read whatever assignments the firmware may have made
> > - Keep them unchanged if they seem sensible
>
> Here I take "sensible" as "it can be successfully claimed" - ie the resource
> is allocated in a valid way, though it may not be optimal (eg bridge window
> apertures).
Yes. But even if platforms assign resources in a way they can be
successfully claimed, I don't think platforms should rely on that
assignment being unchanged. For example, even if the boot-time
configuration is valid, the kernel may need to move things around to
deal with hotplug. I know we don't really do that today, but I think
we should be able to do it in principle.
Bjorn
prev parent reply other threads:[~2016-02-01 21:13 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-20 16:04 [RFC] ARM/ARM64 PCI_PROBE_ONLY platforms Lorenzo Pieralisi
2016-01-20 16:13 ` Sinan Kaya
2016-01-20 18:10 ` Lorenzo Pieralisi
2016-01-20 18:15 ` Sinan Kaya
2016-01-29 23:26 ` Bjorn Helgaas
2016-01-22 16:28 ` Phil Edworthy
2016-01-25 17:51 ` Lorenzo Pieralisi
2016-01-28 17:27 ` Lorenzo Pieralisi
2016-01-29 12:02 ` Gabriele Paoloni
2016-01-29 6:32 ` Pratyush Anand
2016-01-29 23:25 ` Bjorn Helgaas
2016-02-01 16:28 ` Lorenzo Pieralisi
2016-02-01 21:19 ` Bjorn Helgaas
2016-01-29 23:06 ` Bjorn Helgaas
2016-01-30 0:14 ` Sinan Kaya
2016-01-30 13:30 ` Bjorn Helgaas
2016-01-30 17:51 ` Okaya
2016-02-01 15:25 ` Lorenzo Pieralisi
2016-02-01 21:12 ` Bjorn Helgaas [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160201211258.GD4861@localhost \
--to=helgaas@kernel.org \
--cc=bhelgaas@google.com \
--cc=linux-pci@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=okaya@codeaurora.org \
--cc=phil.edworthy@renesas.com \
--cc=wangzhou1@hisilicon.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.