From owner-cvs-all Sun Jan 5 17:55: 8 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 8C18D37B401; Sun, 5 Jan 2003 17:55:07 -0800 (PST) Received: from canning.wemm.org (canning.wemm.org [192.203.228.65]) by mx1.FreeBSD.org (Postfix) with ESMTP id 4CA9443EC2; Sun, 5 Jan 2003 17:55:07 -0800 (PST) (envelope-from peter@wemm.org) Received: from wemm.org (localhost [127.0.0.1]) by canning.wemm.org (Postfix) with ESMTP id 367062A7EA; Sun, 5 Jan 2003 17:55:07 -0800 (PST) (envelope-from peter@wemm.org) X-Mailer: exmh version 2.5 07/13/2001 with nmh-1.0.4 To: Marcel Moolenaar 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 In-Reply-To: <20030106012838.GB15219@athlon.pn.xcllnt.net> Date: Sun, 05 Jan 2003 17:55:07 -0800 From: Peter Wemm Message-Id: <20030106015507.367062A7EA@canning.wemm.org> 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 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. I'm not sure why I didn't get around to this in the original version in p4. Cheers, -Peter -- Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com "All of this is for nothing if we don't go to the stars" - JMS/B5 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message