[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
Fri, 22 Apr 2016 14:22:37 +1000
On Thu, Apr 21, 2016 at 04:03:12PM +1000, Alexey Kardashevskiy wrote:
> 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.
Well.. it'd be nice to do this before the 2.7 release, but it really
depends how much time I have to do this cleanup stuff.
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
Description: PGP signature
[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