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