Thread (18 messages) 18 messages, 5 authors, 2019-06-04

Re: [PATCH kernel] prom_init: Fetch flatten device tree from the system firmware

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2019-06-03 21:20:05

On Mon, 2019-06-03 at 12:56 +1000, Alexey Kardashevskiy wrote:
quoted
That is all you need if you do not want to use OF at all.
? We also need OF drivers to boot grub and the system, and a default
console for early booting, but yes, we do not want to keep using slof
that much.
quoted
If you *do* want to keep having an Open Firmware, what we want or need
is a faster way to walk huge device trees.
Why? We do not need to _walk_ the tree at all (we _have_ to now and
while walking we do nothing but pack everything together into the fdt
blob) as slof can easily do this already.
I agree with Alexey. In a sense this can be thought as an extension of
"quiesce", which effectively kills OF.

Yes we could make property fetching faster but mostly by creating a new
bulk interface which is quite a bit of work, a new API, and will in
practice not be used for anything other than creating the FDT. In that
case, nothing will beat in performance having OF create the FDT itself
based on its current tree.
quoted
quoted
There is no use for the "fetch all properties" cases other than
building an FDT that any of us can think of, and it would create a more
complicated interface than just "fetch an FDT".
It is a simple way to speed up fetching the device tree enormously,
without needing big changes to either OF or the clients using it -- not
in the code, but importantly also not conceptually: everything works just
as before, just a lot faster.
I can safely presume though that this will never ever be used in
practice. And it will be still slower, a tiny bit. And require new code
in both slof and linux.
Correct.
I can rather see us getting rid of SLOF in the future which in turn will
require the fdt blob pointer in r5 (or whatever it is, forgot) when the
guest starts; so "fetch-all-props" won't be used there either.
quoted
quoted
So I go for the simple one and agree with Alexey's idea.
When dealing with a whole device tree you have to know about the various
dynamically generated nodes and props, and handle each appropriately.
The code I am changing fetches the device tree and build an fdt. What is
that special knowledge in this context you are talking about?
Ben.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help