Message ID | 1437134200-13086-2-git-send-email-drjones@redhat.com |
---|---|
State | New |
Headers | show |
On Fri, Jul 17, 2015 at 01:56:39PM +0200, Andrew Jones wrote: > Signed-off-by: Andrew Jones <drjones@redhat.com> Looks good to me, but I'd like an actual commit message: what's dumpdtb, how and why would you use it. > --- > hw/ppc/spapr.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c > index a6f19473cf278..c1cbf3387ae0c 100644 > --- a/hw/ppc/spapr.c > +++ b/hw/ppc/spapr.c > @@ -30,6 +30,7 @@ > #include "hw/fw-path-provider.h" > #include "elf.h" > #include "net/net.h" > +#include "sysemu/device_tree.h" > #include "sysemu/block-backend.h" > #include "sysemu/cpus.h" > #include "sysemu/kvm.h" > @@ -822,6 +823,7 @@ static void spapr_finalize_fdt(sPAPRMachineState *spapr, > exit(1); > } > > + qemu_fdt_dumpdtb(fdt, fdt_totalsize(fdt)); > cpu_physical_memory_write(fdt_addr, fdt, fdt_totalsize(fdt)); > > g_free(bootlist);
On Mon, Jul 20, 2015 at 02:02:33PM +1000, David Gibson wrote: > On Fri, Jul 17, 2015 at 01:56:39PM +0200, Andrew Jones wrote: > > Signed-off-by: Andrew Jones <drjones@redhat.com> > > Looks good to me, but I'd like an actual commit message: what's > dumpdtb, how and why would you use it. Ok, just sent it separately with a commit message and the RFC dropped. Thanks, drew > > > --- > > hw/ppc/spapr.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c > > index a6f19473cf278..c1cbf3387ae0c 100644 > > --- a/hw/ppc/spapr.c > > +++ b/hw/ppc/spapr.c > > @@ -30,6 +30,7 @@ > > #include "hw/fw-path-provider.h" > > #include "elf.h" > > #include "net/net.h" > > +#include "sysemu/device_tree.h" > > #include "sysemu/block-backend.h" > > #include "sysemu/cpus.h" > > #include "sysemu/kvm.h" > > @@ -822,6 +823,7 @@ static void spapr_finalize_fdt(sPAPRMachineState *spapr, > > exit(1); > > } > > > > + qemu_fdt_dumpdtb(fdt, fdt_totalsize(fdt)); > > cpu_physical_memory_write(fdt_addr, fdt, fdt_totalsize(fdt)); > > > > g_free(bootlist); > > -- > 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
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index a6f19473cf278..c1cbf3387ae0c 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -30,6 +30,7 @@ #include "hw/fw-path-provider.h" #include "elf.h" #include "net/net.h" +#include "sysemu/device_tree.h" #include "sysemu/block-backend.h" #include "sysemu/cpus.h" #include "sysemu/kvm.h" @@ -822,6 +823,7 @@ static void spapr_finalize_fdt(sPAPRMachineState *spapr, exit(1); } + qemu_fdt_dumpdtb(fdt, fdt_totalsize(fdt)); cpu_physical_memory_write(fdt_addr, fdt, fdt_totalsize(fdt)); g_free(bootlist);
Signed-off-by: Andrew Jones <drjones@redhat.com> --- hw/ppc/spapr.c | 2 ++ 1 file changed, 2 insertions(+)