From owner-svn-src-head@FreeBSD.ORG Sat Dec 18 17:30:43 2010 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 06176106566C; Sat, 18 Dec 2010 17:30:43 +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 AC2D08FC19; Sat, 18 Dec 2010 17:30:42 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.3/8.14.1) with ESMTP id oBIHQ6FK022006; Sat, 18 Dec 2010 10:26:07 -0700 (MST) (envelope-from imp@bsdimp.com) Message-ID: <4D0CEEAE.8070506@bsdimp.com> Date: Sat, 18 Dec 2010 10:26:06 -0700 From: Warner Losh User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US; rv:1.9.2.12) Gecko/20101029 Thunderbird/3.1.6 MIME-Version: 1.0 To: Nathan Whitehorn References: <201012160036.oBG0aAEh003539@svn.freebsd.org> <201012161225.31459.jhb@freebsd.org> <20101216175536.GA52462@dragon.NUXI.org> <4D0B801A.7050608@freebsd.org> <4D0BB4F0.5090908@bsdimp.com> <4D0CD47A.2060707@freebsd.org> In-Reply-To: <4D0CD47A.2060707@freebsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Robert Watson Subject: Re: svn commit: r216473 - head/sbin/geom/class/eli X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 18 Dec 2010 17:30:43 -0000 On 12/18/2010 08:34, Nathan Whitehorn wrote: > On 12/17/10 13:07, Warner Losh wrote: >> On 12/17/2010 08:22, Nathan Whitehorn wrote: >>> On 12/16/10 13:04, Robert Watson wrote: >>>> On Thu, 16 Dec 2010, David O'Brien wrote: >>>> >>>>>>> Log: >>>>>>> Bump WARNS to 6. >>>>>>> >>>>>>> Modified: >>>>>>> head/sbin/geom/class/eli/Makefile >>>>>> >>>>>> FYI, this broke the tinderbox on arm, ia64, mips, and sparc64. >>>>> >>>>> Errr. Reverted. I built it on the architectures I had access to... >>>> >>>> For WARNS-related changes, I generally use "make universe" to test >>>> across architectures. This builds all of our architectures world + >>>> all available kernels, and seems the most effective way to avoid >>>> the above situations. (I've fallen into exactly the same trap...) >>>> >>>> The one thing to be cautious about is that make universe won't fail >>>> if an individual build fails, so you need to check the logs to make >>>> sure everything actually succeeded. >>> >>> The trouble with make universe is that it has been broken for months >>> and months now. ARM and powerpc64 are disconnected from the build >>> entirely, as are big-endian and 64-bit MIPS, and an increasing >>> number of ARM and PowerPC kernels depend on FDT tools not built by >>> default, and so do not build. Build infrastructure changes also make >>> it appear that the PowerPC GENERIC64 kernel is broken when it is >>> not. This severely reduces the coverage of make universe for >>> problems like this. >>> >>> I have a patch at >>> http://people.freebsd.org/~nwhitehorn/universe.diff that fixes both >>> of these problems, by teaching the universe rule in src/Makefile >>> about MACHINEs with multiple MACHINE_ARCHs and by enabling the build >>> of the FDT tools by default, which adds about 300K to world. The way >>> these are done is probably not optimal, but it is a better than the >>> current situation and is a good stopgap. With the patch, all >>> architectures succeed except for the ARM AVILA kernel, which seems >>> genuinely broken, and the various 64-bit MIPS kernels, since 64-bit >>> MIPS is not hooked up to the build yet. If I don't hear any >>> objections, I would like to commit it on Wednesday the 22nd. >> That works for me. As far as I can tell, doing the MIPS64 stuff will >> just take about a day of patiently fixing some breakage... Maybe >> I'll get to it over the holidays. > > Thanks! With your imprimatur, I've committed the src/Makefile portion > of the patch early. The FDT piece will wait until Wednesday, as > announced. FDT should have always been built from the moment it went into the tree. It was a mistake to make it optional, since we have no good way to make host tools optional based on the target we're compiling... Warner