From owner-cvs-src@FreeBSD.ORG Fri Oct 26 16:00:01 2007 Return-Path: Delivered-To: cvs-src@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id DCDEA16A41A; Fri, 26 Oct 2007 16:00:01 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id 9838413C4BE; Fri, 26 Oct 2007 16:00:01 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.1/8.14.1) with ESMTP id l9QFx9sW062742; Fri, 26 Oct 2007 09:59:09 -0600 (MDT) (envelope-from imp@bsdimp.com) Date: Fri, 26 Oct 2007 09:59:39 -0600 (MDT) Message-Id: <20071026.095939.-432837659.imp@bsdimp.com> To: jhb@FreeBSD.org From: "M. Warner Losh" In-Reply-To: <200710261136.54938.jhb@freebsd.org> References: <200710261500.l9QF0ZJ0036161@repoman.freebsd.org> <200710261136.54938.jhb@freebsd.org> X-Mailer: Mew version 5.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/boot/common ufsread.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 26 Oct 2007 16:00:02 -0000 In message: <200710261136.54938.jhb@freebsd.org> John Baldwin 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. 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? Warner