From owner-freebsd-tinderbox@FreeBSD.ORG Fri Aug 20 22:56:23 2010 Return-Path: Delivered-To: tinderbox@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 03D1A10656AB; Fri, 20 Aug 2010 22:56:23 +0000 (UTC) (envelope-from nwhitehorn@freebsd.org) Received: from agogare.doit.wisc.edu (agogare.doit.wisc.edu [144.92.197.211]) by mx1.freebsd.org (Postfix) with ESMTP id C7A5E8FC19; Fri, 20 Aug 2010 22:56:22 +0000 (UTC) MIME-version: 1.0 Content-transfer-encoding: 8BIT Content-type: text/plain; charset=UTF-8; format=flowed Received: from avs-daemon.smtpauth2.wiscmail.wisc.edu by smtpauth2.wiscmail.wisc.edu (Sun Java(tm) System Messaging Server 7u2-7.05 32bit (built Jul 30 2009)) id <0L7H00L003PYFM00@smtpauth2.wiscmail.wisc.edu>; Fri, 20 Aug 2010 17:56:22 -0500 (CDT) Received: from comporellon.tachypleus.net ([unknown] [76.210.72.125]) by smtpauth2.wiscmail.wisc.edu (Sun Java(tm) System Messaging Server 7u2-7.05 32bit (built Jul 30 2009)) with ESMTPSA id <0L7H00DMN3PWST20@smtpauth2.wiscmail.wisc.edu>; Fri, 20 Aug 2010 17:56:21 -0500 (CDT) Date: Fri, 20 Aug 2010 17:56:19 -0500 From: Nathan Whitehorn In-reply-to: <86fwy9f5vj.fsf@ds4.des.no> To: =?UTF-8?B?RGFnLUVybGluZyBTbcO4cmdyYXY=?= Message-id: <4C6F0813.9030007@freebsd.org> X-Spam-Report: AuthenticatedSender=yes, SenderIP=76.210.72.125 X-Spam-PmxInfo: Server=avs-12, Version=5.6.0.2009776, Antispam-Engine: 2.7.2.376379, Antispam-Data: 2010.8.20.224815, SenderIP=76.210.72.125 References: <201008190304.o7J34Wa4089466@freebsd-current.sentex.ca> <86occzdmhg.fsf@ds4.des.no> <4C6D557E.6080406@freebsd.org> <86sk29ws6u.fsf@ds4.des.no> <4C6E825C.5060509@freebsd.org> <86fwy9f5vj.fsf@ds4.des.no> User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US; rv:1.9.1.11) Gecko/20100729 Thunderbird/3.0.6 Cc: powerpc@freebsd.org, FreeBSD Tinderbox , current@freebsd.org Subject: Re: [head tinderbox] failure on powerpc64/powerpc X-BeenThere: freebsd-tinderbox@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Tinderbox reports, responses, and meta-comments" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 20 Aug 2010 22:56:23 -0000 On 08/20/10 14:20, Dag-Erling Smørgrav wrote: > Nathan Whitehorn writes: > >> Next up: tinderbox needs to build GENERIC64, not GENERIC, on >> powerpc64. make buildkernel is programmed to do the right thing is you >> don't pass a KERNCONF, but tinderbox apparently passes GENERIC >> explicitly. This one's in your court! >> > There is no way to fix this with the current tinderbox code. > > MHO is that the entire powerpc64 thing is very poorly thought out. > I'm the first to admit that many of the config tricks involved in this port, and GENERIC64, are ugly hacks, largely because config(8) was not designed with such things in mind. But the alternatives were much worse: - A sys/powerpc64 that duplicated most of sys/powerpc, which would have been a maintenance nightmare (see sun4v). - A sys/powerpc64 that was mostly symlinks and stub includes, which is inelegant at best, and could cause kinds of fascinating problems for user programs including things in machine/. - Bunches of new make variables like TARGET_BIG_ENDIAN and TARGET_64BIT and such like mips and arm have, which is way more awful, as well as precluding tinderbox and make universe coverage in a reasonable way. powerpc64 is the first architecture trying out this new multi-architecture MACHINE approach, which Warner designed to kill off things like TARGET_BIG_ENDIAN, so there are going to be some interesting rough patches along the way, and I really appreciate your patience in sorting them out. To address the immediate problem, I think the best solution is to use the -m option to config to reject kernel configs for different architectures, the way the tinderbox scripts currently skip kernel configs that they are asked to build that don't exist. If you think this is a reasonable approach, I'm happy to code up a patch to tinderbox to do this over the weekend. You can feel free to disconnect powerpc64 from tinderbox in the interim. -Nathan