On Friday 26 October 2007 11:59:39 am M. Warner Losh wrote: > In message: <[EMAIL PROTECTED]> > John Baldwin <[EMAIL PROTECTED]> writes: > : On Friday 26 October 2007 11:00:34 am Warner Losh wrote: > : > imp 2007-10-26 15:00:34 UTC > : > > : > FreeBSD src repository > : > > : > Modified files: > : > sys/boot/common ufsread.c > : > Log: > : > The arm boot code uses this function as well. Redefining cgbase() > : > saves about 500 bytes in the boot code. While the AT91RM9200 has 12k > : > of space for the boot loader, which is more than i386's 8k, the code > : > generated by gcc is a bit bigger. > : > > : > I've had this in p4 for about two years now. > : > > : > Revision Changes Path > : > 1.16 +1 -1 src/sys/boot/common/ufsread.c > : > > : > : We should probably change this to be #ifdef SMALL_CGBASE or some such then and > : just define it before ufsread.c in boot2.c and other files that include it. > : Does that sound ok to you (or do you have a better name?) UFS_SMALL_CGBASE > : maybe? > > I can't think of a better name, but that does sound good to me. It > would also allow finer granularity of control for these things.
Ok. > One thing I'm unsure about is the amd64 boot blocks. Do those get > build -D__i386__? Are they just the i386 boot blocks because amd64 > boxes boot in the same old mode than 32-bit x86 boxes boot? Yes and yes. -- John Baldwin _______________________________________________ [email protected] mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "[EMAIL PROTECTED]"
