Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 20 Aug 2010 17:56:19 -0500
From:      Nathan Whitehorn <nwhitehorn@freebsd.org>
To:        =?UTF-8?B?RGFnLUVybGluZyBTbcO4cmdyYXY=?= <des@des.no>
Cc:        powerpc@freebsd.org, FreeBSD Tinderbox <tinderbox@freebsd.org>, current@freebsd.org
Subject:   Re: [head tinderbox] failure on powerpc64/powerpc
Message-ID:  <4C6F0813.9030007@freebsd.org>
In-Reply-To: <86fwy9f5vj.fsf@ds4.des.no>
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>

next in thread | previous in thread | raw e-mail | index | archive | help
On 08/20/10 14:20, Dag-Erling Smørgrav wrote:
> Nathan Whitehorn<nwhitehorn@freebsd.org>  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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4C6F0813.9030007>