[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC for-2.7 07/11] pseries: Move adding of fdt reserve
Re: [Qemu-devel] [RFC for-2.7 07/11] pseries: Move adding of fdt reserve map entries
Thu, 21 Apr 2016 16:03:12 +1000
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2
On 04/21/2016 03:52 PM, David Gibson wrote:
On Thu, Apr 21, 2016 at 03:14:48PM +1000, Alexey Kardashevskiy wrote:
On 04/20/2016 12:33 PM, David Gibson wrote:
The flattened device tree passed to pseries guests contains a list of
reserved memory areas. Currently we construct this list early in
spapr_build_fdt() as we sequentially write out the fdt.
This will be inconvenient for upcoming cleanups, so this patch moves
the reserve map changes to the end of fdt construction. This changes
fdt_add_reservemap_entry() calls - which work when writing the fdt
sequentially to fdt_add_mem_rsv() calls used when altering the fdt in
random access mode.
Looks to me like the real reason for this move is that new qdt_setprop_xxx
API does not support memory reserve map yet. Will it, when?
Right, and it's not clear that it even should include reserve map
stuff. The reserve map isn't really part of the device tree, it's
just included in the fdt blob for historical and implementation
So I'd prefer to avoid managing a list of reserve entries in qdt -
instead I was thinking of just having a list of reserves passed
straight into qdt_flatten().
In the meantime, I'd prefer to defer that design decision.
In general, when
do you plan to get rid of _FDT()?
Once I've got rid of all the calls to libfdt functions that need error
I meant timeframe :) Like "2.7 release" or so.
[Qemu-devel] [RFC for-2.7 02/11] pseries: Split device tree construction from device tree load, David Gibson, 2016/04/19
[Qemu-devel] [RFC for-2.7 10/11] pseries: Consolidate construction of /rtas device tree node, David Gibson, 2016/04/19
[Qemu-devel] [RFC for-2.7 01/11] qdt: IEEE1275-style device tree utility code, David Gibson, 2016/04/19