Subject: re: CVS commit: src/sys/kern
To: Juergen Hannken-Illjes <hannken@netbsd.org>
From: matthew green <mrg@eterna.com.au>
List: source-changes
Date: 08/13/2004 10:55:02
On Thu, Aug 12, 2004 at 11:42:43PM +1000, matthew green wrote:
>
>
> Module Name: src
> Committed By: hannken
> Date: Thu Aug 12 12:15:21 UTC 2004
>
> Modified Files:
> src/sys/kern: kern_microtime.c
>
> Log Message:
> Make it compile again for sparc64/DIAGNOSTIC.
>
> All archs should use `cpuid_t ci_cpuid' ...
>
>
> hmmm cpuid_t is a u_long -- why does sparc64 need to use a 64bit
> int for this instead of a 32 bit one? that seems wasteful. (it
> currently just uses 'int'.)
Because we dont need casts in printf's. If all archs use ci_cpuid
as an index, "typedef u_int cpuid_t" leaves space
for approx. 4.000.000.000 cpus.
if cpuid_t was changed to a u_int that would be fine but right
now it's a u_long....
.mrg.