From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kiszka Subject: Re: [PATCH] KVM: nVMX: nested MSR auto load/restore emulation. Date: Sat, 22 Nov 2014 08:31:00 +0100 Message-ID: <54703BB4.1040306@web.de> References: Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="PnTrIEkFrp5pO97DM6BmHWWUnuF717M5p" Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Bandan Das To: Wincy Van , Paolo Bonzini Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --PnTrIEkFrp5pO97DM6BmHWWUnuF717M5p Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 2014-11-22 05:24, Wincy Van wrote: > Some hypervisors need MSR auto load/restore feature. >=20 > We read MSRs from vm-entry MSR load area which specified by L1, > and load them via kvm_set_msr in the nested entry. > When nested exit occurs, we get MSRs via kvm_get_msr, writting > them to L1`s MSR store area. After this, we read MSRs from vm-exit > MSR load area, and load them via kvm_set_msr. >=20 > VirtualBox will work fine with this patch. Cool! This feature is long overdue. Patch is unfortunately misformatted which makes it very hard to read. Please check via linux/scripts/checkpatch.pl for the proper style. Could you also write a corresponding kvm-unit-test (see x86/vmx_tests.c)?= Jan --PnTrIEkFrp5pO97DM6BmHWWUnuF717M5p Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iEYEARECAAYFAlRwO7QACgkQitSsb3rl5xQwFgCfY0UXBsAiyB8Xefv1k/rbsJUx CkoAoIq4bRZd5lFrp2Iwkd+PCNj21Wm3 =dX3X -----END PGP SIGNATURE----- --PnTrIEkFrp5pO97DM6BmHWWUnuF717M5p--