From owner-p4-projects@FreeBSD.ORG Mon Feb 18 02:11:15 2008 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 1208D16A420; Mon, 18 Feb 2008 02:11:15 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id A33EE16A417; Mon, 18 Feb 2008 02:11:14 +0000 (UTC) (envelope-from xcllnt@mac.com) Received: from smtpoutm.mac.com (smtpoutm.mac.com [17.148.16.69]) by mx1.freebsd.org (Postfix) with ESMTP id 8118213C457; Mon, 18 Feb 2008 02:11:14 +0000 (UTC) (envelope-from xcllnt@mac.com) Received: from mac.com (asmtp010-s [10.150.69.73]) by smtpoutm.mac.com (Xserve/smtpout006/MantshX 4.0) with ESMTP id m1I2BElN027177; Sun, 17 Feb 2008 18:11:14 -0800 (PST) Received: from [192.168.1.100] (209-128-86-226.bayarea.net [209.128.86.226]) (authenticated bits=0) by mac.com (Xserve/asmtp010/MantshX 4.0) with ESMTP id m1I2BCAG026716 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Sun, 17 Feb 2008 18:11:13 -0800 (PST) Message-Id: <755B845B-F43E-463D-8D87-7596A8C462F8@mac.com> From: Marcel Moolenaar To: Rafal Jaworowski In-Reply-To: <47B8A8CD.3040707@semihalf.com> Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes Content-Transfer-Encoding: 7bit Mime-Version: 1.0 (Apple Message framework v919.2) Date: Sun, 17 Feb 2008 18:11:12 -0800 References: <200802162141.m1GLfgkj048217@repoman.freebsd.org> <47B75EB3.2020001@semihalf.com> <504560A3-EABB-4896-8B3E-C7FC89F31EFB@mac.com> <47B84D9B.4010903@semihalf.com> <47B8A8CD.3040707@semihalf.com> X-Mailer: Apple Mail (2.919.2) Cc: Marcel Moolenaar , Perforce Change Reviews Subject: Re: PERFORCE change 135517 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 Feb 2008 02:11:15 -0000 On Feb 17, 2008, at 1:36 PM, Rafal Jaworowski wrote: > Another comment: please note that killing WITH_UBOOT introduces > another > problem: now both sys/boot/powerpc/uboot/loader and > sys/boot/powerpc/ofw/loader are built during buildworld, which may > be good > from one point of view, but there's no control over which of the > loader file > is taken during installworld: it seems the last one (u-boot's in > this case) > overwrites the previous... Oh, yeah. We need to rename the U-Boot loader... More comments: with 1GB, the API signature is not found. What we probably should do is have the platform-specific entry set a hint, which can be based on the address of the global data. In other words: on PowerPC in start.S, we save r2 and use that to control where we're going to search for the API signature. FYI, -- Marcel Moolenaar xcllnt@mac.com