From owner-cvs-all Thu Oct 25 15:17:53 2001 Delivered-To: cvs-all@freebsd.org Received: from anchor-post-32.mail.demon.net (anchor-post-32.mail.demon.net [194.217.242.90]) by hub.freebsd.org (Postfix) with ESMTP id 3B70A37B403; Thu, 25 Oct 2001 15:17:45 -0700 (PDT) Received: from [62.49.251.130] (helo=herring.nlsystems.com) by anchor-post-32.mail.demon.net with esmtp (Exim 2.12 #1) id 15wsoq-000HgN-0W; Thu, 25 Oct 2001 23:17:44 +0100 Received: from salmon.nlsystems.com (salmon [10.0.0.3]) by herring.nlsystems.com (8.11.2/8.11.2) with ESMTP id f9PMGS708712; Thu, 25 Oct 2001 23:16:28 +0100 (BST) (envelope-from dfr@nlsystems.com) Date: Thu, 25 Oct 2001 23:14:27 +0100 (BST) From: Doug Rabson To: Luigi Rizzo Cc: John Baldwin , Jonathan Lemon , , Subject: Re: cvs commit: src/sys/dev/fxp if_fxp.c In-Reply-To: <20011025143748.C57392@iguana.aciri.org> Message-ID: <20011025231245.K549-100000@salmon.nlsystems.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII 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 Thu, 25 Oct 2001, Luigi Rizzo wrote: > On Thu, Oct 25, 2001 at 10:33:58PM +0100, Doug Rabson wrote: > >>> Aargh. I really shouldn't do late night commits. Remove a floating point > >>> multiply, and replace it with a close equivalent. 1.488 =~ 1.5 > >> > >> Heh. Floating point in the kernel is bad.. mmmkay? > > > > Its perfectly fine on ia64 though - integer multiplies are done with float > > registers... > > does it mean that integer multiply in the kernel is bad on the ia64 :) Not really - its reasonably quick. They just didn't bother to add any integer multipliers when there were two perfectly good floating point units going idle. Moving integers in and out of float registers is quick and easy so it actually makes sense. Divides are done with floating point too. -- Doug Rabson Mail: dfr@nlsystems.com Phone: +44 20 8348 6160 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message