From owner-freebsd-arch Thu Dec 13 14:32:16 2001 Delivered-To: freebsd-arch@freebsd.org Received: from falcon.prod.itd.earthlink.net (falcon.mail.pas.earthlink.net [207.217.120.74]) by hub.freebsd.org (Postfix) with ESMTP id BF43D37B416 for ; Thu, 13 Dec 2001 14:32:13 -0800 (PST) Received: from pool0211.cvx21-bradley.dialup.earthlink.net ([209.179.192.211] helo=mindspring.com) by falcon.prod.itd.earthlink.net with esmtp (Exim 3.33 #1) id 16EeOh-000241-00; Thu, 13 Dec 2001 14:32:11 -0800 Message-ID: <3C192C70.9A79B3C5@mindspring.com> Date: Thu, 13 Dec 2001 14:32:16 -0800 From: Terry Lambert X-Mailer: Mozilla 4.7 [en]C-CCK-MCD {Sony} (Win98; U) X-Accept-Language: en MIME-Version: 1.0 To: Thomas Moestl Cc: arch@FreeBSD.org Subject: Re: Please review: changes to MI bus code for sparc64 References: <20011213192033.A871@crow.dom2ip.de> <3C18F78D.C537D487@mindspring.com> <20011213201213.B871@crow.dom2ip.de> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Thomas Moestl wrote: > > I don't understand the, isa_init() parameter change, as the post-patch > > code does not appear to use "dev"? > > The sparc64 one does. OK; explains why it's not int he patches. > > The PCI_BROKEN_INTPIN/PCI_INTLINE_0_BAD seem to be the same thing; Still agree with Mike. [ ... PCI routines becoming global ... ] > I did this change to support the Sun APB bridge, which does not have > base and limit registers, but uses a different mechanism to set the > decoded ranges. So, to check resource allocations correctly, the apb > driver needs to implement it's own alloc_resource method. The pcib > functions which need not be 'overridden' can be directly used in the > method table. I still think that the proper way is to call a function to get the addresses of the static functions. Ideally, you would pass and/or break out the alloc_resource method, which would make the Sun APB brige "standard". > > Also in subr_rman.c, in int_rman_activate_resource(), the change from > > 32 to 31 is not obvious... should you also change "> 0" to ">= 0"? It's > > not clear to me what this change fixes. I'm probably just missing > > something... > > 1 << 32 is out of u_int32_t range, so it just would make no sense. In > addition, it will overflow i (which is an int) on most architectures. I was more concerned with the 1 << 0 not being hit at all. I agree on the overflow fix; I was just curious why the entire range was not shifted down 1, instead of just avoiding the overflow. -- Terry To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message