All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Bader <stefan.bader@canonical.com>
To: kvm@vger.kernel.org
Cc: Anthoine Bourgeois <bourgeois@bertin.fr>,
	Paolo Bonzini <pbonzini@redhat.com>
Subject: nVMX regression v3.13+, bisected
Date: Wed, 26 Feb 2014 20:43:40 +0100	[thread overview]
Message-ID: <530E43EC.7000600@canonical.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1312 bytes --]

Hi,

I was looking at a bug report[1] about a regression on nested VMX that started
with kernel v3.13 (same issue still existed with v3.14-rc4). The problem shows
up when running a v3.13 kernel in L0 and then trying to launch a L2 (L1 was
either a v3.2 kernel or v3.13, so seemed to have no immediate influence). L2 is
trying to boot a iso image and hangs before the isolinux boot loader displays
anything. A preinstalled hd image fails to boot, too.

I bisected this and ended up on the following commit which, when reverted made
the launch work again:

Author: Anthoine Bourgeois <bourgeois@bertin.fr>
Date:   Wed Nov 13 11:45:37 2013 +0100

    kvm, vmx: Fix lazy FPU on nested guest

    If a nested guest does a NM fault but its CR0 doesn't contain the TS
    flag (because it was already cleared by the guest with L1 aid) then we
    have to activate FPU ourselves in L0 and then continue to L2. If TS flag
    is set then we fallback on the previous behavior, forward the fault to
    L1 if it asked for.

    Signed-off-by: Anthoine Bourgeois <bourgeois@bertin.fr>
    Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

The condition to exit to L0 seems to be according to what the description says.
Could it be that the handling in L0 is doing something wrong?

-Stefan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]

             reply	other threads:[~2014-02-26 19:43 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-26 19:43 Stefan Bader [this message]
2014-02-26 20:25 ` nVMX regression v3.13+, bisected Paolo Bonzini
2014-02-26 20:27   ` Stefan Bader
2014-02-26 20:44     ` Kashyap Chamarthy
2014-02-27 12:10       ` Kashyap Chamarthy
2014-02-27 15:55         ` Kashyap Chamarthy
2014-02-27 10:51     ` Paolo Bonzini
2014-02-27 13:41       ` anthoine.bourgeois
2014-02-27 17:01       ` anthoine.bourgeois
2014-02-27 16:58         ` Paolo Bonzini
2014-02-27 21:34           ` Kashyap Chamarthy

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=530E43EC.7000600@canonical.com \
    --to=stefan.bader@canonical.com \
    --cc=bourgeois@bertin.fr \
    --cc=kvm@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    /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.