From owner-freebsd-ppc@FreeBSD.ORG Sat Jan 29 23:36:35 2011 Return-Path: Delivered-To: powerpc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 23ACE106564A; Sat, 29 Jan 2011 23:36:35 +0000 (UTC) (envelope-from xcllnt@mac.com) Received: from asmtpout029.mac.com (asmtpout029.mac.com [17.148.16.104]) by mx1.freebsd.org (Postfix) with ESMTP id 08ACB8FC12; Sat, 29 Jan 2011 23:36:34 +0000 (UTC) MIME-version: 1.0 Content-transfer-encoding: 7BIT Content-type: text/plain; CHARSET=US-ASCII Received: from [192.168.2.13] (atm.xcllnt.net [70.36.220.6]) by asmtp029.mac.com (Oracle Communications Messaging Exchange Server 7u4-20.01 64bit (built Nov 21 2010)) with ESMTPSA id <0LFT00HP05KUND70@asmtp029.mac.com>; Sat, 29 Jan 2011 15:36:31 -0800 (PST) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.2.15,1.0.148,0.0.0000 definitions=2011-01-29_05:2011-01-28, 2011-01-29, 1970-01-01 signatures=0 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 ipscore=0 suspectscore=0 phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx engine=6.0.2-1012030000 definitions=main-1101290119 From: Marcel Moolenaar In-reply-to: <4D44A26F.1070701@freebsd.org> Date: Sat, 29 Jan 2011 15:36:29 -0800 Message-id: <81D12FF3-11D1-4625-ABE3-9EE30748E9E6@mac.com> References: <550814DC-3E80-401C-B725-BFCB391726DF@mac.com> <4D449504.7030209@freebsd.org> <5C0CFF6E-94C9-43F5-A22F-5A0F6ECC0448@mac.com> <4D4498FF.9050808@freebsd.org> <4D449D3B.5050008@freebsd.org> <4D44A26F.1070701@freebsd.org> To: Nathan Whitehorn X-Mailer: Apple Mail (2.1082) Cc: powerpc@freebsd.org Subject: Re: Beware of revision 218075 X-BeenThere: freebsd-ppc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting FreeBSD to the PowerPC List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 29 Jan 2011 23:36:35 -0000 On Jan 29, 2011, at 3:27 PM, Nathan Whitehorn wrote: >>> Index: intr_machdep.c >>> =================================================================== >>> --- intr_machdep.c (revision 218074) >>> +++ intr_machdep.c (revision 218075) >>> @@ -373,6 +426,9 @@ >>> i->pol != INTR_POLARITY_CONFORM) >>> PIC_CONFIG(i->pic, i->intline, i->trig, i->pol); >>> >>> + if (i != NULL&& i->pic == root_pic) >>> + PIC_BIND(i->pic, i->intline, i->cpu); >>> + >>> if (i->event != NULL) >>> PIC_ENABLE(i->pic, i->intline, vector); >>> } >> >> That can be reverted without harm AFAICT. One of my earlier >> changes removed the SYSINIT to do the binding and instead do >> it here. I partially reverted that (i.e. the SYSINIT is >> still there, but the change above remained), because I didn't >> know the consequence of such a change. > I've reverted it in r218081. Everything seems OK now. Thanks. I was just about to revert the change myself after realizing that I may have responded in a way that would put the burden on you. >>> The "max IRQ = 128" thing also likely breaks Cell systems, like the PS3, where the PIC has 256 interrupts, but I haven't looked into that in detail yet. >> This is where we need to implement multiple passes. I tried to >> do that in the same commit, but ended up with a lot more >> churn than I was comfortable handling. But ideally, we probe >> busses and interrupt controllers first, so that we don't have >> this problem -- all interrupt controllers will have registered >> before we need to map from PIC+pin to IRQ. >> >> There's probably a quick fix in case it's broken. If the pin >> is higher than the number of IRQs we have recorded for the PIC, >> we can simply bump the number of irqs the PIC has (i.e. extend >> the IRQ range assigned to the PIC). Unless we have other PICs >> recorded this can always be done -- since the PS3 only has 1 >> PIC, this should be a quick kluge to get it working again. > > Ah, OK. My PS3 seems to boot fine, so don't worry about it. Good luck navigating through the rest of this minefield! I actually think I've just passed through it. This commit was perceived to be the most dangerous and it seems I sustained nothing more than a bruise or scratch :-) -- Marcel Moolenaar xcllnt@mac.com