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.