Date: Tue, 15 Jul 2003 15:06:45 -0700
From: Andrew Morton <akpm@osdl.org>
To: george anzinger <george@mvista.com>
Cc: bernie@develer.com, linux-kernel@vger.kernel.org, rmk@arm.linux.org.uk,
       torvalds@osdl.org
Subject: Re: do_div64 generic
Message-Id: <20030715150645.4fa11de7.akpm@osdl.org>
In-Reply-To: <3F1477B2.6090106@mvista.com>
References: <3F1360F4.2040602@mvista.com>
	<200307150717.54981.bernie@develer.com>
	<20030714223805.4e5bee3f.akpm@osdl.org>
	<200307150823.01602.bernie@develer.com>
	<3F1477B2.6090106@mvista.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1127
Lines: 26

george anzinger <george@mvista.com> wrote:
>
> > George, do you agree? May I go on and post a patch killing
> > div_long_long_rem() everywhere?
> 
> The issue is that div is a very long instruction and the do_div() 
> thing uses 2 or three of them, while the div_long_long_rem() is just 
> 1.  Also, a lot of archs already have the required div by a different 
> name.  It all boils down to a performance thing.

It is only used in nanosleep(), and then only in the case where the sleep
terminated early.

If someone is calling nanosleep() so frequently for this to matter, the
time spent in divide is the least of their problems.  Unless you have some
real-worldish benchmarks to demonstrate otherwise?

You know what they say about premtur optmstns, and having to propagate
funky new divide primitives across N architectures is indeed evil.

Bernardo, can you do the patch please?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/