From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH] Proper use of VMX execution controls MSR. Date: Thu, 29 Mar 2007 10:11:58 +0100 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "Li, Xin B" , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On 29/3/07 09:37, "Li, Xin B" wrote: > if we have the feature support in Xen, we should set the input control > bit, and expect the adjusted value to be 1; > if we don't have the feature support in Xen, we should clear the input > control bit, and expect adjusted value to be 0; > That's meaning the adjusted output is determined by the input control. > So the expression should be "AND (msr_hi XOR msr_lo)". The subsequent AND with msr_hi and the OR with msr_lo has exactly the same effect. So the AND (msr_hi XOR msr_lo) is not incorrect, but it is redundant. -- Keir