[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] hw/mips/boston: Pack fdt in fdt filter
From: |
David Gibson |
Subject: |
Re: [PATCH 2/2] hw/mips/boston: Pack fdt in fdt filter |
Date: |
Thu, 18 Aug 2022 12:48:39 +1000 |
On Tue, Aug 16, 2022 at 12:46:46PM +0100, Jiaxun Yang wrote:
>
>
> > 2022年8月16日 01:44,Philippe Mathieu-Daudé <f4bug@amsat.org> 写道:
> >
> > On 13/8/22 18:27, Jiaxun Yang wrote:
> >> FDT can be awfully fat after series of modifications in fdt
> >> filter. Just pack it up before add to ram.
> >> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> >> ---
> >> hw/mips/boston.c | 1 +
> >> 1 file changed, 1 insertion(+)
> >> diff --git a/hw/mips/boston.c b/hw/mips/boston.c
> >> index 5145179951..a40f193f78 100644
> >> --- a/hw/mips/boston.c
> >> +++ b/hw/mips/boston.c
> >> @@ -400,6 +400,7 @@ static const void *boston_fdt_filter(void *opaque,
> >> const void *fdt_orig,
> >> 1, boston_memmap[BOSTON_HIGHDDR].base +
> >> ram_low_sz,
> >> 1, ram_high_sz);
> >> + fdt_pack(fdt);
> >> fdt = g_realloc(fdt, fdt_totalsize(fdt));
> >> qemu_fdt_dumpdtb(fdt, fdt_sz);
> >>
> >
> > Why not pack by default in qemu_fdt_dumpdtb()?
>
> qemu_fdt_dumpdtb() is explicitly a function for debugging purpose.
> Donno if it’s wise to hijack it.
Agreed. Having this modify the dtb sounds like a very bad idea.
--
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_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature