Subject: Re: CVS commit: src/sys
To: Jeff Rizzo <riz@NetBSD.org>
From: David Laight <firstname.lastname@example.org>
Date: 02/03/2006 18:16:13
On Thu, Feb 02, 2006 at 08:16:53AM -0800, Jeff Rizzo wrote:
> I'm not certain this is the specific commit that caused it, but amd64
> kernels which are not optimized (like, say, GENERIC) fail to link due to
> looking for bswap16 and bswap32. I looked things over, but am not sure
> of the correct way to fix this...
I'm not sure where I changed the effect of building non-optimised kernels.
I'm fairly sure the const-folding and __inline versions were both inside
#if __OPTIMISE__. I don't think it makes any sense for it to be otherwise.
libkern (or somewhere) may need to contain extern defns of bswapnn()
anyway - for cases where the #define doesn't apply (do we allow the
address of bswapnn() to be taken?)
It is also worth noting that the libc functions are __bswap16(), __bswap32()
and bswap64() on i386 and amd64, whereas the kernel ones never have the
the leading underscores.
I'll look at what happens to unoptimised code later.
David Laight: email@example.com