All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Huth <thuth@redhat.com>
To: "Cédric Le Goater" <clg@kaod.org>,
	qemu-ppc@nongnu.org, "Anton Blanchard" <anton@ozlabs.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>,
	David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [Qemu-devel] [Qemu-ppc] KVM-PR is broken with current QEMU
Date: Wed, 21 Sep 2016 10:22:11 +0200	[thread overview]
Message-ID: <37465716-edd3-de38-2b29-44a3093cf312@redhat.com> (raw)
In-Reply-To: <07648f7a-24e2-9854-5d05-fe5dfb38c719@kaod.org>

On 20.09.2016 16:39, Cédric Le Goater wrote:
> On 09/20/2016 04:24 PM, Thomas Huth wrote:
>> On 20.09.2016 16:04, Cédric Le Goater wrote:
[...]
>>> There are other issues after in the guest (kernel crashing). But I think
>>> these are related to TM which is not supported in KVM-PR. I am not sure
>>> 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.
> 
> 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. 

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

  reply	other threads:[~2016-09-21  8:22 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-20 11:44 [Qemu-devel] KVM-PR is broken with current QEMU Thomas Huth
2016-09-20 12:24 ` Cédric Le Goater
2016-09-20 14:04   ` [Qemu-devel] [Qemu-ppc] " Cédric Le Goater
2016-09-20 14:24     ` Thomas Huth
2016-09-20 14:39       ` Cédric Le Goater
2016-09-21  8:22         ` Thomas Huth [this message]
2016-09-22  1:57           ` David Gibson
2016-09-22  5:30             ` Thomas Huth
2016-09-22  7:18               ` Thomas Huth
2016-09-22  9:46                 ` Cédric Le Goater
2016-09-22  9:57                 ` Anton Blanchard
2016-09-20 21:45 ` [Qemu-devel] " Benjamin Herrenschmidt
2016-09-21  7:45   ` Thomas Huth
2016-09-22  6:25   ` Thomas Huth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=37465716-edd3-de38-2b29-44a3093cf312@redhat.com \
    --to=thuth@redhat.com \
    --cc=anton@ozlabs.org \
    --cc=clg@kaod.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.