From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 4/4] KVM: Add support for in-kernel LAPIC model Date: Mon, 07 May 2007 13:22:44 +0300 Message-ID: <463EFDF4.7090106@qumranet.com> References: <20070502212713.16738.8133.stgit@novell1.haskins.net> <20070502214330.16738.51436.stgit@novell1.haskins.net> <463EFCC5.8050408@qumranet.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: <463EFCC5.8050408-atKUWr5tajBWk0Htik3J/w@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 Avi Kivity wrote: > > >> + >> +/* >> + *----------------------------------------------------------------------- >> + * KERNEL-TIMERS >> + * >> + * Unfortunately we really need HRTIMERs to do this right, but they might >> + * not exist on olders kernels (sigh). So we roughly abstract this interface >> + * to support nanosecond resolution, and then emulate it as best we can if >> + * the real HRTIMERs arent available >> + *----------------------------------------------------------------------- >> + */ >> >> > > The correct way to use non-hrtimer kernels is to write the hrtimer api > in terms of the old api, in external-module-compat.h. > > An alternative is to just put a patch adding the #ifdef and the non-hrtimer case in the kernel/ directory, and to have 'make sync' apply the patch. This is annoying to maintain but easier to generate. -- 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/