From mboxrd@z Thu Jan  1 00:00:00 1970
From: Sheng Yang <sheng@linux.intel.com>
Subject: Re: [PATCH] MSR related clean up
Date: Wed, 24 Jun 2009 16:50:22 +0800
Message-ID: <200906241650.22488.sheng@linux.intel.com>
References: <1245319026-28658-1-git-send-email-sheng@linux.intel.com>
Mime-Version: 1.0
Content-Type: Text/Plain;
  charset="iso-8859-15"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <1245319026-28658-1-git-send-email-sheng@linux.intel.com>
Content-Disposition: inline
List-Unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: xen-devel@lists.xensource.com
Cc: Eddie Dong <eddie.dong@intel.com>, Keir Fraser <keir.fraser@eu.citrix.com>
List-Id: xen-devel@lists.xenproject.org

On Thursday 18 June 2009 17:57:06 Sheng Yang wrote:
> rdmsr_safe() is used to access MSR unknown to Xen, and is not safe... I
> think it is legacy.
>
> Also let msr_write be parity with msr_read on MSR_IA32_MISC_ENABLE.
>
> CC: Eddie Dong <eddie.dong@intel.com>
> Signed-off-by: Sheng Yang <sheng@linux.intel.com>

Keir?

-- 
regards
Yang, Sheng

>
> diff -r c30742011bb8 -r 64dc4510484e xen/arch/x86/hvm/vmx/vmx.c
> --- a/xen/arch/x86/hvm/vmx/vmx.c	Thu Mar 12 18:48:09 2009 +0000
> +++ b/xen/arch/x86/hvm/vmx/vmx.c	Thu Jun 18 17:10:22 2009 +0800
> @@ -1836,8 +1836,7 @@
>          }
>
>          if ( rdmsr_viridian_regs(ecx, &eax, &edx) ||
> -             rdmsr_hypervisor_regs(ecx, &eax, &edx) ||
> -             rdmsr_safe(ecx, eax, edx) == 0 )
> +             rdmsr_hypervisor_regs(ecx, &eax, &edx) )
>          {
>              regs->eax = eax;
>              regs->edx = edx;
> @@ -2008,6 +2007,9 @@
>      }
>      case MSR_IA32_VMX_BASIC...MSR_IA32_VMX_PROCBASED_CTLS2:
>          goto gp_fault;
> +    /* Ignore writing to these MSRs */
> +    case MSR_IA32_MISC_ENABLE:
> +        break;
>      default:
>          if ( vpmu_do_wrmsr(regs) )
>              return X86EMUL_OKAY;
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel