From owner-cvs-src@FreeBSD.ORG Tue Nov 22 22:53:42 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id AB58D16A434; Tue, 22 Nov 2005 22:53:42 +0000 (GMT) (envelope-from marius@newtrinity.zeist.de) Received: from newtrinity.zeist.de (newtrinity.zeist.de [217.24.217.8]) by mx1.FreeBSD.org (Postfix) with ESMTP id 96E4443D6B; Tue, 22 Nov 2005 22:53:23 +0000 (GMT) (envelope-from marius@newtrinity.zeist.de) Received: from newtrinity.zeist.de (localhost [127.0.0.1]) by newtrinity.zeist.de (8.12.11/8.12.11/ZEIST.DE) with ESMTP id jAMMrCpY036488; Tue, 22 Nov 2005 23:53:16 +0100 (CET) (envelope-from marius@newtrinity.zeist.de) Received: (from marius@localhost) by newtrinity.zeist.de (8.12.11/8.12.10/Submit) id jAMMr7BP036487; Tue, 22 Nov 2005 23:53:07 +0100 (CET) (envelope-from marius) Date: Tue, 22 Nov 2005 23:53:07 +0100 From: Marius Strobl To: Scott Long Message-ID: <20051122235307.H16812@newtrinity.zeist.de> References: <200511222232.jAMMWo6u088484@repoman.freebsd.org> <43839E0D.2080108@samsco.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <43839E0D.2080108@samsco.org>; from scottl@samsco.org on Tue, Nov 22, 2005 at 03:39:09PM -0700 X-AntiVirus-modified: yes X-AntiVirus: checked by AntiVir Milter (version: 1.1.1-9; AVE: 6.32.0.58; VDF: 6.32.0.206; host: newtrinity.zeist.de) Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/sparc64/pci psycho.c psychovar.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Nov 2005 22:53:43 -0000 On Tue, Nov 22, 2005 at 03:39:09PM -0700, Scott Long wrote: > > Marius Strobl wrote: > > > marius 2005-11-22 22:32:50 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/sparc64/pci psycho.c psychovar.h > > Log: > > - Add a workaround (change the interrupt map mask to compare the full > > INO) for incorrect interrupt map entries on E250 machines. These > > incorrect entries caused the INO of the on-board HME to be also > > assigned to the second on-board NS16550 and to the on-board printer > > port controller. Further down the road caused hme(4) to fail to attach > > to the on-board HME in FreeBSD 5 and 6 as INTR_FAST and non-INTR_FAST > > handlers can't share the same IRQ there (it's unknown what whould > > happen in -CURRENT now that INTR_FAST and non-INTR_FAST handlers can > > share an IRQ but I'd expect funny problems with uart(4)). > > - Make sure there are exactly 4 PCI ranges instead of just checking > > that the bridge has a 'ranges' property in the OFW device tree at all. > > Besides the fact that currently the 64bit memory range isn't used by > > this driver it we can't really work with less than 4 ranges and don't > > have memory for more than 4 bus handles for the ranges in the softc. > > - Remove sc_range and sc_nrange from softc; for the bridges supported > > by this driver we no longer need to know the ranges besides the bus > > handles obtained from them once this driver is attached. That way we > > also can free the memory allocated for sc_range during attach again. > > - Remove sc_dvmabase from the softc and pass it to psycho_iommu_init() > > via an additional argument as we no longer need to know the DVMA base > > in this driver once the IOMMU is initialized. > > - Remove sc_dmatag from the softc, there isn't much sense in keeping > > the nexus dma tag around locally. > > > > It's been a TODO item forever to merge busdma tag management with > newbus, so that a driver can request the tag of its newbus's parent > instead of just guessing what constraints the parent allows. > Uhm, was this a FYI or do you mean I shouldn't have removed the nexus DMA tag from the psycho(4) softc (I fail to see why this would have to e.g. be reintroduced when busdma tag management is newbus'ified)? Marius -- This mail was scanned by AntiVir Milter. This product is licensed for non-commercial use. See www.antivir.de for details.