From owner-cvs-src@FreeBSD.ORG Sat Jan 10 23:28:54 2004 Return-Path: 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 CCD7316A4CE; Sat, 10 Jan 2004 23:28:54 -0800 (PST) Received: from harmony.village.org (rover.bsdimp.com [204.144.255.66]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5D90343D4C; Sat, 10 Jan 2004 23:28:53 -0800 (PST) (envelope-from imp@bsdimp.com) Received: from localhost (warner@rover2.village.org [10.0.0.1]) by harmony.village.org (8.12.10/8.12.9) with ESMTP id i0B7SqET033602; Sun, 11 Jan 2004 00:28:52 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Sun, 11 Jan 2004 00:28:38 -0700 (MST) Message-Id: <20040111.002838.75200166.imp@bsdimp.com> To: nate@root.org From: "M. Warner Losh" In-Reply-To: <20040110232115.R49720@root.org> References: <20040111065246.651CF16A4F9@hub.freebsd.org> <20040110232115.R49720@root.org> X-Mailer: Mew version 3.3 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/pci pci_pci.c pcib_private.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 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: Sun, 11 Jan 2004 07:28:54 -0000 In message: <20040110232115.R49720@root.org> Nate Lawson writes: : On Sat, 10 Jan 2004, Warner Losh wrote: : > Modified files: : > sys/dev/pci pci_pci.c pcib_private.h : > Log: : > Add support for subtractive decoding bridges. These bridges pass all : > signals to addresses to the child busses. Typically, ProgIf of 1 : > means a subtractive bridge. However, Intel has a whole lot of ones : > with a ProgIf of 80 that are also subtractive. We cope with these : > bridges too. This eliminates hw.pci.allow_unsupported_io_range : > because that had almost the same effect as these patches (almost means : > 'buggy'). Remove the bogus checks for ISA bus locations: these cycles : > aren't special and are only passed by transparent bridges. : > : > We allow any range to succeed. If the range is a superset of the : > range that's decoded, trim the resource to that range. Otherwise, : > pass the range unchanged. This will change the location that PC Card : > and CardBus cards are attached. This might bogusly cause some : > overlapping allocation that wasn't present before, but the overlapping : > fixes need to be in the pci level. : > : > There's also a few formatting changes here. : > : > Revision Changes Path : > 1.29 +126 -159 src/sys/dev/pci/pci_pci.c : > 1.6 +2 -0 src/sys/dev/pci/pcib_private.h : : Woo hoo! Scary also. :) Tell me about it :-) I'll likely tease out the resource allocation stuff from the power stuff once I get it fixed to try to do allocations that are properly aligned. Warner