Re: CVS commit: src/sys/arch/zaurus/conf

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

> >> I don't see how as sys/arch/zaurus/conf/INSTALL doesn't define DDB.
> >
> > On zaurus GENERIC also has a size restriction (due to bootloader).
> 
> Yes, I understand that. What I don't understand is how my
> sys/arch/arm/conf/Makefile.arm change affects the zaurus INSTALL kernel
> size because INSTALL specifically disables DDB and my patch is
> conditional on DDB

As martin wrote, the problem is not INSTALL but GENERIC
because zaurus GENERIC is large (~5MB) enough.

> My patch allows COPTS to be overridden as
> 
> 
> http://src.illumos.org/source/xref/netbsd-src/sys/conf/Makefile.kern.inc#69
> 
> has
> 
> 69 DEFCOPTS?=	-O2
> 70 COPTS?=	${DEFCOPTS}
> 
> which I believe allows COPTS to be overridden by
> 
> makeoptions COPTS="..."

If it's users' responsibility to explicitly add "-mapcs-frame" in COPTS 
when they want to build a kernel with both COPTS="-Os" and options DDB,
it's okay for me.  (I'm afraid zaurus GENERIC needs -Os in near future)

---
Izumi Tsutsui