On Monday, 3 September 2018 10:09:07 CEST Richard W.M. Jones wrote:
See: https://bugzilla.redhat.com/show_bug.cgi?id=1614948#c23
Thanks: David Gibson. --- lib/launch-direct.c | 3 +++ lib/launch-libvirt.c | 10 ++++++++++ 2 files changed, 13 insertions(+)
diff --git a/lib/launch-direct.c b/lib/launch-direct.c index 47e8f37de..94dd995c6 100644 --- a/lib/launch-direct.c +++ b/lib/launch-direct.c @@ -523,6 +523,9 @@ launch_direct (guestfs_h *g, void *datav, const char *arg) #ifdef __aarch64__ if (has_kvm && !force_tcg) append_list ("gic-version=host"); +#endif +#ifdef __powerpc64__ + append_list ("cap-htm=off"); #endif
Which version of qemu was this introduced in?
diff --git a/lib/launch-libvirt.c b/lib/launch-libvirt.c index 48404ef93..53284fc08 100644 --- a/lib/launch-libvirt.c +++ b/lib/launch-libvirt.c @@ -1181,6 +1181,16 @@ construct_libvirt_xml_boot (guestfs_h *g, } }
+#ifdef __powerpc64__ + start_element ("features") { + start_element ("pseries") { + start_element ("htm") { + attribute ("state", "on"); + } end_element (); + } end_element (); + } end_element (); +#endif
This generates an XML snippet like: <features> <pseries> <htm state="on"/> </pseries> </features> I see two problems: - there is no <pseries> subelement, <htm> is directly under <features> - shouldn't it be off, just like done in the direct backend? -- Pino Toscano