From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 0/9] in-kernel APIC v8 (kernel side) Date: Thu, 24 May 2007 18:06:52 +0300 Message-ID: <4655AA0C.9040108@qumranet.com> References: <20070524131917.11321.17746.stgit@novell1.haskins.net> <4655A63E.7000505@qumranet.com> <4655A80C.4000606@qumranet.com> <4655709E.BA47.005A.0@novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: Gregory Haskins Return-path: In-Reply-To: <4655709E.BA47.005A.0-Et1tbQHTxzrQT0dZR+AlfA@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 Gregory Haskins wrote: >>>> On Thu, May 24, 2007 at 10:58 AM, in message <4655A80C.4000606-atKUWr5tajBWk0Htik3J/w@public.gmane.org>, >>>> > Avi Kivity wrote: > >> Avi Kivity wrote: >> >>> Gregory Haskins wrote: >>> >>>> Passes the usual tests, and offers an 11% speedup. There is another >>>> enhancement not included here (coming shortly) which exposes another 12% >>>> speedup over this as well. Stay tuned >>>> >>>> >>>> >>> This fails on amd, locks up completely. Bisecting now. lapic.c needs >>> a #include to compile here. >>> >>> >> Unsurprisingly, this is >> >> KVM: Adds ability to preempt an executing VCPU >> >> I see nothing wrong there, will look further. >> > > > I bet it has something to do with the local_irq_disable vs clgi in the kvm_run() code. > clgi() overrides local irq, so I don't see how that happens. -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/