Re: [Qemu-devel] [Qemu-ppc] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support

[David Gibson]

On Wed, Aug 28, 2019 at 11:50:14AM +0200, Greg Kurz wrote:
> On Wed, 28 Aug 2019 15:57:40 +1000
> David Gibson <address@hidden> wrote:
> 
> > On Wed, Aug 28, 2019 at 01:27:35PM +1000, Alexey Kardashevskiy wrote:
> > > 
> > > 
> > > On 28/08/2019 12:12, David Gibson wrote:
> > > > On Wed, Aug 28, 2019 at 10:46:34AM +1000, David Gibson wrote:
> > > > > On Tue, Aug 27, 2019 at 04:56:50PM +1000, Alexey Kardashevskiy wrote:
> > > > > > The following changes since commit 
> > > > > > d6bb8b27204eaa58f1da948b65454e3a598ab2a4:
> > > > > > 
> > > > > >    pseries: Update SLOF firmware image (2019-08-27 16:47:04 +1000)
> > > > > > 
> > > > > > are available in the Git repository at:
> > > > > > 
> > > > > >    address@hidden:aik/qemu.git tags/qemu-slof-20190827
> > > > > > 
> > > > > > for you to fetch changes up to 
> > > > > > da9960a5aaa25d27c9798c3d94c7b4c2d8af31ac:
> > > > > > 
> > > > > >    spapr: Render full FDT on ibm,client-architecture-support 
> > > > > > (2019-08-27 16:47:46 +1000)
> > > > > > 
> > > > > > ----------------------------------------------------------------
> > > > > > Alexey Kardashevskiy (1):
> > > > > >        spapr: Render full FDT on ibm,client-architecture-support
> > > > > > 
> > > > > >   hw/ppc/spapr.c | 90 
> > > > > > +++++++---------------------------------------------------
> > > > > >   1 file changed, 10 insertions(+), 80 deletions(-)
> > > > > > 
> > > > > > 
> > > > > > *** Note: this is not for master, this is for pseries
> > > > > > 
> > > > > 
> > > > > Merged, thanks.
> > > > 
> > > > Urgh.  And the qemu change is now un-merged.  Alas, as soon as we had
> > > > a CAS reboot for XIVE the guest didn't boot on the second attempt.
> > > > Haven't had a chance to investigate yet.
> > > 
> > > QEMU command line, guest kernel version? I'd give it a try.
> > 
> > RHEL8.1 in guest and host, booting via GRUB into a XIVE capable
> > kernel.
> > 
> > I've now been able to investigate, dumping the dtb at H_UPDATE_DT
> > time.
> 
> FWIW I sent a patch to do just that some times ago:
> 
> https://patchwork.ozlabs.org/patch/1095705/

Yeah, I know.  I didn't like adding yet another option as a permanent
fixture.

-- 
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