From owner-cvs-all Sun Jan 5 18:12: 5 2003 Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id BB52D37B401; Sun, 5 Jan 2003 18:12:03 -0800 (PST) Received: from kayak.xcllnt.net (209-128-86-226.bayarea.net [209.128.86.226]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0D8AE43EB2; Sun, 5 Jan 2003 18:12:03 -0800 (PST) (envelope-from marcel@xcllnt.net) Received: from athlon.pn.xcllnt.net (athlon.pn.xcllnt.net [192.168.4.3]) by kayak.xcllnt.net (8.12.6/8.12.6) with ESMTP id h062C2rT042244; Sun, 5 Jan 2003 18:12:02 -0800 (PST) (envelope-from marcel@kayak.pn.xcllnt.net) Received: from athlon.pn.xcllnt.net (localhost [127.0.0.1]) by athlon.pn.xcllnt.net (8.12.6/8.12.6) with ESMTP id h062C2es015472; Sun, 5 Jan 2003 18:12:02 -0800 (PST) (envelope-from marcel@athlon.pn.xcllnt.net) Received: (from marcel@localhost) by athlon.pn.xcllnt.net (8.12.6/8.12.6/Submit) id h062C2IV015471; Sun, 5 Jan 2003 18:12:02 -0800 (PST) (envelope-from marcel) Date: Sun, 5 Jan 2003 18:12:02 -0800 From: Marcel Moolenaar To: Peter Wemm Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/ia64/ia64 clock.c interrupt.c src/sys/ia64/include clock.h Message-ID: <20030106021202.GA15451@athlon.pn.xcllnt.net> References: <20030106012838.GB15219@athlon.pn.xcllnt.net> <20030106015507.367062A7EA@canning.wemm.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20030106015507.367062A7EA@canning.wemm.org> User-Agent: Mutt/1.5.1i Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Sun, Jan 05, 2003 at 05:55:07PM -0800, Peter Wemm wrote: > Marcel Moolenaar wrote: > > On Sun, Jan 05, 2003 at 05:18:26PM -0800, Peter Wemm wrote: > > > Marcel Moolenaar wrote: > > > > While here, replace (itc_frequency+hz/2)/hz with itm_reload for > > > > consistency. There's now a single place where we determine the > > > > ITM reload value. > > > > > > Actually it is for speed. This is a floating point divide.. (ia64 doesn't > > > have integer divide) > > > > Ah, yes. Good one! Forced commit to make a point? > > I moved the two duplicate reloads to a single copy a few lines earlier and > noted the divide issue in the commit message. Thanks! > I'm not sure why I didn't > get around to this in the original version in p4. Maybe because it's insignificant compared to all the other things that need to be done... :-) -- Marcel Moolenaar USPA: A-39004 marcel@xcllnt.net To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message