Date: Fri, 10 Sep 2004 15:06:53 -0400 From: John Baldwin <jhb@FreeBSD.org> To: Marcel Moolenaar <marcel@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/alpha/include atomic.h Message-ID: <200409101506.53655.jhb@FreeBSD.org> In-Reply-To: <200409100500.i8A50R7U038632@repoman.freebsd.org> References: <200409100500.i8A50R7U038632@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve" = http://www.FreeBSD.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200409101506.53655.jhb>