From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] simplify pdpte check Date: Thu, 01 Feb 2007 16:07:01 +0200 Message-ID: <45C1F405.5070400@qumranet.com> References: <20070201140138.GM2327@rhun.haifa.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel To: Muli Ben-Yehuda Return-path: In-Reply-To: <20070201140138.GM2327-WD1JZD8MxeCTrf4lBMg6DdBPR1lH4CV8@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: kvm.vger.kernel.org Muli Ben-Yehuda wrote: > Small cleanup: we check (pdpte & 1) && (pdpte & constant). Instead > just check (pdpte & (constant | 1)). > > Signed-off-by: Muli Ben-Yehuda > > Index: kernel/kvm_main.c > =================================================================== > --- kernel/kvm_main.c (revision 4379) > +++ kernel/kvm_main.c (working copy) > @@ -336,7 +336,7 @@ > ret = 1; > for (i = 0; i < 4; ++i) { > pdpte = pdpt[offset + i]; > - if ((pdpte & 1) && (pdpte & 0xfffffff0000001e6ull)) { > + if (pdpte & 0xfffffff0000001e7ull) { > ret = 0; > goto out; > } > > The code is not equivalent. For example, pdpte == 2 passes the check before the patch and fails it after. -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier. Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642