cvs commit: src/sys/alpha/include atomic.h
John Baldwin
jhb at FreeBSD.org
Fri Sep 10 12:09:44 PDT 2004
On Friday 10 September 2004 01:00 am, Marcel Moolenaar wrote:
> marcel 2004-09-10 05:00:27 UTC
>
> FreeBSD src repository
>
> Modified files:
> sys/alpha/include atomic.h
> Log:
> The previous commit, roughly one and a half years ago removed the
> branch prediction optimization for LINT, because the kernel was too
> large. This commit now removes it altogether since it causes build
> failures for GENERIC kernels and the various applicable trends are
> such that one can expect that it these failure will cause more
> problems than they're worth in the future. These trends include:
> 1. Alpha was demoted from tier 1 to tier 2 due to lack of active
> support. The number of people willing to fix build breakages
> is not likely to increase and those developers that do have the
> gumption to test MI changes on alpha are not likely to spend
> time fixing unexpected build failures first.
> 2. The kernel will only increase in size. Even though stripped-down
> kernels do link without problems now, compiler optimizations (like
> inlining) and new (non-optional) functionality will likely cause
> stripped-down kernels to break in the future as well.
>
> So, with my asbestos suit on, get rid of potential problems before
> they happen.
>
> MT5 candidate.
>
> Revision Changes Path
> 1.19 +14 -62 src/sys/alpha/include/atomic.h
Hmm, maybe leave it in but add 'MUTEX_NO_INLINE' to GENERIC on Alpha so that
GENERIC will build but people can take out the 'NO_INLINE' bit in custom
kernels if they want?
--
John Baldwin <jhb at FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve" = http://www.FreeBSD.org
More information about the cvs-src
mailing list