Subject: Re: CVS commit: src/sys/arch/i386/conf
To: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
From: Valeriy E. Ushakov <uwe@ptc.spbu.ru>
List: source-changes
Date: 11/27/2006 19:42:56
On Tue, Nov 28, 2006 at 01:14:53 +0900, Izumi Tsutsui wrote:
> > I don't see the problem. Your change hides the problem, and
> > prevents us from understanding what is going on, so I would back
> > it out.
>
> If anyone handled it, there was no problem for me.
> (I'm not in a position to resolve i386 specific issue anyway)
tls@ added -mfoo to COPTS (e.g. 1.794 of GENERIC). The fix is to add
-mfoo to CPUFLAGS instead (we have plenty of configs that do that),
not adding -O2 to COPTS in the config.
The real issue of course is that kernel makefile uses COPTS
incorrectly. bsd.README documents COPTS as "ADDITIONAL flags to the
compiler when creating C objects" (emphasis mine), so we can't really
blame Thor except for trusting the docs :).
SY, Uwe
--
uwe@ptc.spbu.ru | Zu Grunde kommen
http://snark.ptc.spbu.ru/~uwe/ | Ist zu Grunde gehen