* [Qemu-devel] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support @ 2019-08-27 6:56 Alexey Kardashevskiy 2019-08-28 0:46 ` David Gibson 0 siblings, 1 reply; 7+ messages in thread From: Alexey Kardashevskiy @ 2019-08-27 6:56 UTC (permalink / raw) To: David Gibson; +Cc: Alexey Kardashevskiy, qemu-ppc, qemu-devel 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: git@github.com: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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support 2019-08-27 6:56 [Qemu-devel] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support Alexey Kardashevskiy @ 2019-08-28 0:46 ` David Gibson 2019-08-28 2:12 ` David Gibson 0 siblings, 1 reply; 7+ messages in thread From: David Gibson @ 2019-08-28 0:46 UTC (permalink / raw) To: Alexey Kardashevskiy; +Cc: qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 1100 bytes --] 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: > > git@github.com: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. -- 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 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support 2019-08-28 0:46 ` David Gibson @ 2019-08-28 2:12 ` David Gibson 2019-08-28 3:27 ` Alexey Kardashevskiy 0 siblings, 1 reply; 7+ messages in thread From: David Gibson @ 2019-08-28 2:12 UTC (permalink / raw) To: Alexey Kardashevskiy; +Cc: qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 1396 bytes --] 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: > > > > git@github.com: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. -- 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 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support 2019-08-28 2:12 ` David Gibson @ 2019-08-28 3:27 ` Alexey Kardashevskiy 2019-08-28 5:57 ` David Gibson 0 siblings, 1 reply; 7+ messages in thread From: Alexey Kardashevskiy @ 2019-08-28 3:27 UTC (permalink / raw) To: David Gibson; +Cc: qemu-ppc, qemu-devel 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: >>> >>> git@github.com: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. -- Alexey ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support 2019-08-28 3:27 ` Alexey Kardashevskiy @ 2019-08-28 5:57 ` David Gibson 2019-08-28 9:50 ` [Qemu-devel] [Qemu-ppc] " Greg Kurz 0 siblings, 1 reply; 7+ messages in thread From: David Gibson @ 2019-08-28 5:57 UTC (permalink / raw) To: Alexey Kardashevskiy; +Cc: qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 4074 bytes --] 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: > > > > > > > > git@github.com: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. Comparing before and after the fully-render-dt change, I get: @@ -24,13 +24,13 @@ }; chosen { - bootargs = "BOOT_IMAGE=/vmlinuz-4.18.0-137.el8.ppc64le root=/dev/mapper/rhel-root ro crashkernel=auto rd.lvm.lv=rhel/root rd.lvm.lv=rhel/swap"; + bootargs = [00]; bootpath = "/pci@800000020000000/scsi@0"; cpu = <0xe45ee80>; ibm,arch-vec-5-platform-support = <0x17801880 0x19001a40>; ibm,architecture-vec-5 = [19 00 20 00 00 80 05 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 80 40 40 00 40]; - linux,initrd-end = <0x00 0x588ed7d>; - linux,initrd-start = <0x00 0x3c00000>; + linux,initrd-end = <0x00>; + linux,initrd-start = <0x00>; linux,pci-probe-only = <0x00>; linux,stdout-package = <0xe733a70>; linux,stdout-path = "/vdevice/vty@71000001"; @@ -252,7 +252,7 @@ used-by-rtas; ethernet@1 { - assigned-addresses = <0x82000830 0x00 0x80040000 0x00 0x40000 0x81000810 0x00 0x100 0x00 0x20 0x82000814 0x00 0x80010000 0x00 0x1000 0xc3000820 0x2100 0x10000 0x00 0x4000>; + assigned-addresses; cache-line-size = <0x00>; class-code = <0x20000>; device-id = <0x1000>; @@ -274,7 +274,7 @@ }; scsi@0 { - assigned-addresses = <0x81000010 0x00 0x80 0x00 0x80 0x82000014 0x00 0x80000000 0x00 0x1000 0xc3000020 0x2100 0x00 0x00 0x4000>; + assigned-addresses; cache-line-size = <0x00>; class-code = <0x10000>; device-id = <0x1001>; So basically the bootargs, initrd params and assigned-addresses have been lost. For bootargs and initrd, I think what's happening is that because qemu isn't setting those (with no -kernel parameter) SLOF is creating them, but they're now being overwritten by qemu's post-CAS DT. I'm not sure what's going on with assigned-addresses. -- 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 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support 2019-08-28 5:57 ` David Gibson @ 2019-08-28 9:50 ` Greg Kurz 2019-08-29 0:00 ` David Gibson 0 siblings, 1 reply; 7+ messages in thread From: Greg Kurz @ 2019-08-28 9:50 UTC (permalink / raw) To: David Gibson; +Cc: Alexey Kardashevskiy, qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 4254 bytes --] On Wed, 28 Aug 2019 15:57:40 +1000 David Gibson <david@gibson.dropbear.id.au> 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: > > > > > > > > > > git@github.com: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/ > Comparing before and after the fully-render-dt change, I get: > > @@ -24,13 +24,13 @@ > }; > > chosen { > - bootargs = "BOOT_IMAGE=/vmlinuz-4.18.0-137.el8.ppc64le root=/dev/mapper/rhel-root ro crashkernel=auto rd.lvm.lv=rhel/root rd.lvm.lv=rhel/swap"; > + bootargs = [00]; > bootpath = "/pci@800000020000000/scsi@0"; > cpu = <0xe45ee80>; > ibm,arch-vec-5-platform-support = <0x17801880 0x19001a40>; > ibm,architecture-vec-5 = [19 00 20 00 00 80 05 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 80 40 40 00 40]; > - linux,initrd-end = <0x00 0x588ed7d>; > - linux,initrd-start = <0x00 0x3c00000>; > + linux,initrd-end = <0x00>; > + linux,initrd-start = <0x00>; > linux,pci-probe-only = <0x00>; > linux,stdout-package = <0xe733a70>; > linux,stdout-path = "/vdevice/vty@71000001"; > @@ -252,7 +252,7 @@ > used-by-rtas; > > ethernet@1 { > - assigned-addresses = <0x82000830 0x00 0x80040000 0x00 0x40000 0x81000810 0x00 0x100 0x00 0x20 0x82000814 0x00 0x80010000 0x00 0x1000 0xc3000820 0x2100 0x10000 0x00 0x4000>; > + assigned-addresses; > cache-line-size = <0x00>; > class-code = <0x20000>; > device-id = <0x1000>; > @@ -274,7 +274,7 @@ > }; > > scsi@0 { > - assigned-addresses = <0x81000010 0x00 0x80 0x00 0x80 0x82000014 0x00 0x80000000 0x00 0x1000 0xc3000020 0x2100 0x00 0x00 0x4000>; > + assigned-addresses; > cache-line-size = <0x00>; > class-code = <0x10000>; > device-id = <0x1001>; > > So basically the bootargs, initrd params and assigned-addresses have > been lost. > > For bootargs and initrd, I think what's happening is that because qemu > isn't setting those (with no -kernel parameter) SLOF is creating them, > but they're now being overwritten by qemu's post-CAS DT. > > I'm not sure what's going on with assigned-addresses. > [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support 2019-08-28 9:50 ` [Qemu-devel] [Qemu-ppc] " Greg Kurz @ 2019-08-29 0:00 ` David Gibson 0 siblings, 0 replies; 7+ messages in thread From: David Gibson @ 2019-08-29 0:00 UTC (permalink / raw) To: Greg Kurz; +Cc: Alexey Kardashevskiy, qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 2380 bytes --] On Wed, Aug 28, 2019 at 11:50:14AM +0200, Greg Kurz wrote: > On Wed, 28 Aug 2019 15:57:40 +1000 > David Gibson <david@gibson.dropbear.id.au> 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: > > > > > > > > > > > > git@github.com: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 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-08-29 1:17 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-08-27 6:56 [Qemu-devel] [GIT PULL for qemu-pseries] spapr: Render full FDT on ibm, client-architecture-support Alexey Kardashevskiy 2019-08-28 0:46 ` David Gibson 2019-08-28 2:12 ` David Gibson 2019-08-28 3:27 ` Alexey Kardashevskiy 2019-08-28 5:57 ` David Gibson 2019-08-28 9:50 ` [Qemu-devel] [Qemu-ppc] " Greg Kurz 2019-08-29 0:00 ` David Gibson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).