From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49902) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bmcnJ-0004yS-4J for qemu-devel@nongnu.org; Wed, 21 Sep 2016 04:22:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bmcnI-0004pc-06 for qemu-devel@nongnu.org; Wed, 21 Sep 2016 04:22:17 -0400 References: <75632046-c527-8d5f-6ac0-a40c28239579@redhat.com> <71298df5-d5de-e09f-5abd-9a8dbeefb16c@kaod.org> <3faff7d8-88f2-08c9-2d1d-6478872f4569@redhat.com> <07648f7a-24e2-9854-5d05-fe5dfb38c719@kaod.org> From: Thomas Huth Message-ID: <37465716-edd3-de38-2b29-44a3093cf312@redhat.com> Date: Wed, 21 Sep 2016 10:22:11 +0200 MIME-Version: 1.0 In-Reply-To: <07648f7a-24e2-9854-5d05-fe5dfb38c719@kaod.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [Qemu-ppc] KVM-PR is broken with current QEMU List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?Q?C=c3=a9dric_Le_Goater?= , qemu-ppc@nongnu.org, Anton Blanchard Cc: QEMU Developers , David Gibson On 20.09.2016 16:39, C=C3=A9dric Le Goater wrote: > On 09/20/2016 04:24 PM, Thomas Huth wrote: >> On 20.09.2016 16:04, C=C3=A9dric Le Goater wrote: [...] >>> There are other issues after in the guest (kernel crashing). But I th= ink >>> these are related to TM which is not supported in KVM-PR. I am not su= re >>> where we are on that point. >> >> There was a patch some months ago: >> >> https://lists.gnu.org/archive/html/qemu-ppc/2016-04/msg00046.html >> >> ... but I think it has never been included, as far as I can see. >=20 > and with that patch, the guest fully boots. But David had some concerns > on the way it is done. It would be nice to put some cycle on this.=20 Looking at the mail thread, I think TM should be currently disabled for both, KVM-PR and TCG, i.e. only enabled for KVM-HV. The TM support in TCG is just fake, since TBEGIN always fails. Once we've got proper TM support in TCG, this can be easily changed within QEMU. And once we've got TM support in KVM-PR, I think we should also introduce a capability flag to KVM which can be used to inform QEMU about this. So I think Anton's patch currently just lacks the check for TCG. Anton, if you've got some spare minutes, could you maybe send an updated version of that patch? Thanks, Thomas