From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mukesh Rathor Subject: Re: [PATCH 09/17] PVH xen: create PVH vmcs, and also initialization Date: Tue, 30 Apr 2013 14:04:16 -0700 Message-ID: <20130430140416.6b2c7257@mantra.us.oracle.com> References: <1366752366-16594-1-git-send-email-mukesh.rathor@oracle.com> <1366752366-16594-10-git-send-email-mukesh.rathor@oracle.com> <5177A91902000078000D0383@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5177A91902000078000D0383@nat28.tlf.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: xen-devel , dexuan.cui@intel.com List-Id: xen-devel@lists.xenproject.org On Wed, 24 Apr 2013 08:42:49 +0100 "Jan Beulich" wrote: > >>> On 23.04.13 at 23:25, Mukesh Rathor > >>> wrote: > > > + guest_pat = MSR_IA32_CR_PAT_RESET; > > + __vmwrite(GUEST_PAT, guest_pat); > > What's the point of having the local variable "guest_pat" here? > copy/paste from existing code in construct_vmcs(): u64 host_pat, guest_pat; rdmsrl(MSR_IA32_CR_PAT, host_pat); guest_pat = MSR_IA32_CR_PAT_RESET; __vmwrite(HOST_PAT, host_pat); __vmwrite(GUEST_PAT, guest_pat); I'll remove it in mine. thanks mukesh