All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeroen Groenewegen van der Weyden <groen692@grosc.com>
To: Tim Deegan <Tim.Deegan@citrix.com>
Cc: "Christoph.Egger@amd.com" <Christoph.Egger@amd.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	"Dong, Eddie" <eddie.dong@intel.com>
Subject: Re: [PATCH 20 of 20] n2 MSR handling and capability exposure
Date: Tue, 26 Jul 2011 09:15:30 +0200	[thread overview]
Message-ID: <4E2E6992.9030407@grosc.com> (raw)
In-Reply-To: <20110725161657.GF8970@whitby.uk.xensource.com>

Hi Tim,

I think the behaviour is still the same,

1) cs23726
2) vvmc.c patched with attachment.
3) new compile

after a little while the domu becomes ir-responsive.

with xm dmesg I see a lot of these:
(XEN) vvmx.c:1182:d3 vmclear gpa 1f5a89000 not the same as current vmcs 
00000001f448f000
(XEN) vvmx.c:1182:d3 vmclear gpa 1f5a89000 not the same as current vmcs 
00000001f448f000


Note: I have to say, patching this on this level is not common practice 
for me. although I think I did it the right way. please keep in mind I 
can make mistakes on this level.

mfg,
Jeroen.

Op 25-7-2011 18:16, Tim Deegan schreef:
> Hi,
>
> At 15:08 +0100 on 25 Jul (1311606523), Tim Deegan wrote:
>> FWIW, I can reproduce this with a Debian 2.6.32-5-686 n1 guest on
>> current unstable tip.  Running two copies of 'kvm' inside that
>> (i.e. simple n2s without any disks) I see this on the n0 console:
>>
>> (XEN) vvmx.c:1181:d1 vmclear gpa 3661d000 not the same as current vmcs 0000000036459000
>> (XEN) vvmx.c:1181:d1 vmclear gpa 36459000 not the same as current vmcs 000000003661d000
>>
>> and the n1 guest locks up using 100% cpu on one of its vcpus.
> AFAICS when KVM has two VMs sharing a CPU, it just switches between them
> with VMPTRLD, rather than VMCLEARing the current one on every context
> switch.  When it migrates one of them away, it VMCLEARs it, even if it's
> not the most recently loaded VMCS.
>
> Xen's emulation of VMCLEAR doesn't clear the 'launched' bit in this
> case, though the SDM says it should.  The attached patch fixes the hang
> for me, but has had only very light testing (i.e. I haven't checked that
> proper OSes running inside the KVM VMs behave correctly).
>
> Eddie, does this look right to you?
>
> Jeroen, can you try it and see if it fixes your problems?
>
> Cheers,
>
> Tim.
>

  reply	other threads:[~2011-07-26  7:15 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-09  8:25 [PATCH 00 of 20] Rebased Nested VMX v2 Eddie Dong
2011-06-09  8:25 ` [PATCH 01 of 20] pre-cleanup1: Extend nhvm_vmcx_guest_intercepts_trap to include errcode to Eddie Dong
2011-06-09  9:46   ` Christoph Egger
2011-06-09  8:25 ` [PATCH 02 of 20] pre-cleanup2: Move IDT_VECTORING processing code out of intr_assist Eddie Dong
2011-06-09  8:25 ` [PATCH 03 of 20] Add data structure for nestedvmx Eddie Dong
2011-06-09  8:25 ` [PATCH 04 of 20] Add APIs for nestedhvm_ops Eddie Dong
2011-07-08  7:42   ` vickywill
2011-07-08  7:43     ` vickywill
2011-06-09  8:25 ` [PATCH 05 of 20] Emulation of guest VMXON/OFF instruction Eddie Dong
2011-06-09  8:25 ` [PATCH 06 of 20] Define structure and access APIs for virtual VMCS Eddie Dong
2011-06-09  8:25 ` [PATCH 07 of 20] Emulation of guest vmptrld Eddie Dong
2011-06-09  8:25 ` [PATCH 08 of 20] Emulation of guest VMPTRST Eddie Dong
2011-06-09  8:25 ` [PATCH 09 of 20] Emulation of guest VMCLEAR Eddie Dong
2011-06-09  8:25 ` [PATCH 10 of 20] Emulation of guest VMWRITE Eddie Dong
2011-06-09  8:25 ` [PATCH 11 of 20] Emulation of guest VMREAD Eddie Dong
2011-06-09  8:25 ` [PATCH 12 of 20] Add APIs to switch n1/n2 VMCS Eddie Dong
2011-06-09  8:25 ` [PATCH 13 of 20] Emulation of VMRESUME/VMLAUNCH Eddie Dong
2011-06-09  8:25 ` [PATCH 14 of 20] Extend VMCS control fields for n2 guest Eddie Dong
2011-06-09  8:25 ` [PATCH 15 of 20] Switch shadow/virtual VMCS between n1/n2 guests Eddie Dong
2011-06-09  8:25 ` [PATCH 16 of 20] interrupt/exception handling for n2 guest Eddie Dong
2011-06-09  8:25 ` [PATCH 17 of 20] VM exit handler of n2-guest Eddie Dong
2011-06-09  8:25 ` [PATCH 18 of 20] Lazy FPU for n2 guest Eddie Dong
2011-06-09  8:25 ` [PATCH 19 of 20] Add VMXE bits in virtual CR4 Eddie Dong
2011-06-09  8:25 ` [PATCH 20 of 20] n2 MSR handling and capability exposure Eddie Dong
2011-06-13  8:27   ` Tim Deegan
2011-06-14  2:29     ` Dong, Eddie
2011-06-15 12:45       ` Tim Deegan
2011-06-20  6:31         ` Jeroen Groenewegen van der Weyden
2011-06-30 14:14           ` Jeroen Groenewegen van der Weyden
2011-07-01  9:01             ` Tim Deegan
2011-07-01  9:54               ` Christoph Egger
2011-07-01 10:11                 ` Jeroen Groenewegen van der Weyden
2011-07-04  8:58                   ` Tim Deegan
2011-07-04  9:58                     ` Jeroen Groenewegen van der Weyden
2011-07-08  7:12                     ` Jeroen Groenewegen van der Weyden
2011-07-18 15:41                       ` Dong, Eddie
2011-07-19 13:59                         ` Jeroen Groenewegen van der Weyden
2011-07-20 13:44                           ` Dong, Eddie
2011-07-20 14:12                             ` Jeroen Groenewegen van der Weyden
2011-07-25 14:08                               ` Tim Deegan
2011-07-25 16:16                                 ` Tim Deegan
2011-07-26  7:15                                   ` Jeroen Groenewegen van der Weyden [this message]
2011-07-26 10:00                                     ` Tim Deegan
2011-07-26 10:11                                       ` Tim Deegan
2011-07-26 10:46                                         ` Jeroen Groenewegen van der Weyden
2011-07-26 11:42                                           ` Tim Deegan
2011-07-26 13:33                                             ` Jeroen Groenewegen van der Weyden
2011-07-26 14:21                                               ` Tim Deegan
2011-07-26 15:25                                                 ` Jeroen Groenewegen van der Weyden
2011-07-26 15:48                                                   ` Tim Deegan
2011-07-26 16:00                                                     ` Jeroen Groenewegen van der Weyden
2011-07-26 16:08                                                       ` Tim Deegan
2011-07-26 11:05                                       ` Jeroen Groenewegen van der Weyden
2011-06-09 10:13 ` [PATCH 00 of 20] Rebased Nested VMX v2 Christoph Egger
2011-06-13  9:24   ` Tim Deegan
2011-06-14  3:01     ` Dong, Eddie
2011-06-14  9:01       ` Tim Deegan
2011-06-14  7:43     ` Christoph Egger
2011-06-14 14:08       ` Dong, Eddie
  -- strict thread matches above, loose matches on Subject: below --
2011-06-02  8:57 [PATCH 00 of 20] NestedVMX support Eddie Dong
2011-06-02  8:57 ` [PATCH 20 of 20] n2 MSR handling and capability exposure Eddie Dong
2011-06-02 15:07   ` Tim Deegan
2011-06-02 15:11     ` Tim Deegan
2011-06-02 19:20       ` Keir Fraser
2011-06-03  8:39       ` Dong, Eddie
2011-06-03  8:25     ` Dong, Eddie

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=4E2E6992.9030407@grosc.com \
    --to=groen692@grosc.com \
    --cc=Christoph.Egger@amd.com \
    --cc=Tim.Deegan@citrix.com \
    --cc=eddie.dong@intel.com \
    --cc=xen-devel@lists.xensource.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.