public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Alexander Graf <agraf@suse.de>
Cc: Avi Kivity <avi@redhat.com>, Kevin Wolf <kwolf@suse.de>,
	kvm@vger.kernel.org
Subject: Re: Linux in VirtualPC in KVM fails
Date: Fri, 09 Jan 2009 13:04:10 -0600	[thread overview]
Message-ID: <49679FAA.3000802@codemonkey.ws> (raw)
In-Reply-To: <BF37150A-0572-47D3-BDCA-26D0068E0A8C@suse.de>

Alexander Graf wrote:
>
> Shouldn't it be ok to push patches to linux-stable to use the CPUID 
> and MSR information and simply not expose the CPUID identification in 
> newer KVM versions? That way older guests on newer KVM don't use KVM 
> paravirt (which should still be ok), but everything else runs as 
> smoothly as possible.

I'd say that VirtualPC is broken and leave it at that.

All modern hypervisors use CPUID to expose themselves to the guests.  
VirtualPC is totally capable of rewriting CPUID instructions.  Shame on 
it for not doing so.  I don't think we should change anything.

Regards,

Anthony Liguori

> Alex
>
>> So I suggest:
>> - add a new cpuid bit, for 'paravirt verification'
>> - if the new bit is present, the guest verifies that paravirt is 
>> truly enabled by reading an msr (and checking a signature).
>>
>> Older guests or newer guests running on older hosts will still be 
>> broken, but there's a command line parameter for disabling paravirt 
>> on the guest, and we can add one for the host as well.
>>
>> -- 
>> I have a truly marvellous patch that fixes the bug which this
>> signature is too narrow to contain.
>>
>
> -- 
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


  reply	other threads:[~2009-01-09 19:04 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-09 15:43 Linux in VirtualPC in KVM fails Kevin Wolf
2009-01-09 17:50 ` Avi Kivity
2009-01-09 19:00   ` Alexander Graf
2009-01-09 19:04     ` Anthony Liguori [this message]
2009-01-09 19:32       ` Avi Kivity
2009-01-09 20:10         ` Anthony Liguori
2009-01-11  7:16           ` Avi Kivity
2009-01-09 19:34     ` Avi Kivity
2009-01-09 19:05   ` Anthony Liguori
2009-01-09 19:24     ` Alexander Graf
2009-01-09 20:09       ` Anthony Liguori
2009-01-11  7:17         ` Avi Kivity

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=49679FAA.3000802@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=agraf@suse.de \
    --cc=avi@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=kwolf@suse.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox